Fixed various typos in a lot of files

Closes #49478

TEST=Manual

GitOrigin-RevId: f4c9c68
Change-Id: I134a97caed4eec59d70e9cbca16b7e9a472cf2c1
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/251902
Reviewed-by: Michael Thomsen <[email protected]>
Commit-Queue: Alexander Thomas <[email protected]>
Reviewed-by: Aske Simon Christensen <[email protected]>
Reviewed-by: Kevin Chisholm <[email protected]>
Reviewed-by: Alexander Thomas <[email protected]>

Author: anisalibegic

.github/ISSUE_TEMPLATE/1_cherry_pick.yml

@@ -27,7 +27,7 @@ body:
     id: issue_description
     attributes:
       label: Issue Description
-      description: Brief description of the issue.  What is the issue? What platforms are the problems occuring on?
+      description: Brief description of the issue.  What is the issue? What platforms are the problems occurring on?
     validations:
       required: true
   - type: textarea

.mailmap

@@ -1,4 +1,4 @@
-# This file contains name to mail address mappings that are usefull
+# This file contains name to mail address mappings that are useful
 # for old svn based commits (to get nice names instead of mail in blame and
 # other tools)
 Adam Klein <[email protected]>

benchmarks/BigIntParsePrint/dart/BigIntParsePrint.dart

@@ -12,7 +12,7 @@ import 'package:fixnum/fixnum.dart';
 import 'native_version_dummy.dart'
     if (dart.library.js) 'native_version_javascript.dart';
 
-// Benckmark BigInt and Int64 formatting and parsing.
+// Benchmark BigInt and Int64 formatting and parsing.
 
 // A global sink that is used in the [check] method ensures that the results are
 // not optimized.
@@ -49,7 +49,7 @@ class Benchmark extends BenchmarkBase {
       if (b.bitLength < bits) {
         restartDelta += seed >> 20;
         restartDelta += BigInt.one;
-        // Restart from a slighly reduced seed to generate different numbers.
+        // Restart from a slightly reduced seed to generate different numbers.
         b = seed - restartDelta;
       }
       var string = b.toString();
@@ -310,7 +310,7 @@ void main() {
     selectFormatNativeBigIntBenchmark('JsBigInt.toString.4096.bits', 4096),
   ];
 
-  // Warm up all benchmarks to ensure consistent behavious of shared code.
+  // Warm up all benchmarks to ensure consistent behavior of shared code.
   benchmarks.forEach((bm) => bm()
     ..setup()
     ..run()

benchmarks/Richards/dart/Richards.dart

@@ -414,7 +414,7 @@ class HandlerTask extends Task {
 }
 
 /// A simple package of data that is manipulated by the tasks.  The exact layout
-/// of the payload data carried by a packet is not importaint, and neither is
+/// of the payload data carried by a packet is not important, and neither is
 /// the nature of the work performed on packets by the tasks. Besides carrying
 /// data, packets form linked lists and are hence used both as data and
 /// worklists.

benchmarks/Richards/dart2/Richards.dart

@@ -418,7 +418,7 @@ class HandlerTask extends Task {
 }
 
 /// A simple package of data that is manipulated by the tasks.  The exact layout
-/// of the payload data carried by a packet is not importaint, and neither is
+/// of the payload data carried by a packet is not important, and neither is
 /// the nature of the work performed on packets by the tasks. Besides carrying
 /// data, packets form linked lists and are hence used both as data and
 /// worklists.

benchmarks/Richards/javascript/Richards.js

@@ -497,7 +497,7 @@ var DATA_SIZE = 4;
 
 /**
  * A simple package of data that is manipulated by the tasks.  The exact layout
- * of the payload data carried by a packet is not importaint, and neither is the
+ * of the payload data carried by a packet is not important, and neither is the
  * nature of the work performed on packets by the tasks.
  *
  * Besides carrying data, packets form linked lists and are hence used both as

build/mac/change_mach_o_flags.py

@@ -121,7 +121,7 @@ def CheckedRead(file, count):
 
 
 def ReadUInt32(file, endian):
-    """Reads an unsinged 32-bit integer from the file-like |file| object,
+    """Reads an unsigned 32-bit integer from the file-like |file| object,
   treating it as having endianness specified by |endian| (per the |struct|
   module), and returns it as a number. Raises a MachOError if the proper
   length of data can't be read from |file|."""
@@ -160,7 +160,7 @@ def ReadFatArch(file):
 
 
 def WriteUInt32(file, uint32, endian):
-    """Writes |uint32| as an unsinged 32-bit integer to the file-like |file|
+    """Writes |uint32| as an unsigned 32-bit integer to the file-like |file|
   object, treating it as having endianness specified by |endian| (per the
   |struct| module)."""
 

pkg/_fe_analyzer_shared/test/macros/executor/simple_macro.dart

@@ -8,7 +8,7 @@ import 'package:_fe_analyzer_shared/src/macros/api.dart';
 
 /// A very simple macro that augments any declaration it is given, usually
 /// adding print statements and inlining values from the declaration object
-/// for comparision with expected values in tests.
+/// for comparison with expected values in tests.
 ///
 /// When applied to [MethodDeclaration]s there is some extra work that happens
 /// to validate the introspection APIs work as expected.

pkg/analysis_server/lib/lsp_protocol/protocol_generated.dart

@@ -306,7 +306,7 @@ typedef TextDocumentSignatureHelpResult = SignatureHelp?;
 
 /// Result for a request to resolve the type definition locations of a symbol at
 /// a given text document position. The request's parameter is of type
-/// TextDocumentPositioParams the response is of type Definition or a Thenable
+/// TextDocumentPositionParams the response is of type Definition or a Thenable
 /// that resolves to such.
 typedef TextDocumentTypeDefinitionResult
     = Either2<Definition, List<DefinitionLink>>?;
@@ -21955,7 +21955,7 @@ class PreviousResultId implements ToJsonable {
     );
   }
 
-  /// The URI for which the client knowns a result id.
+  /// The URI for which the client knows a result id.
   final DocumentUri uri;
 
   /// The value of the previous result id.

pkg/analyzer/lib/src/summary2/ast_binary_writer.dart

@@ -434,7 +434,7 @@ class AstBinaryWriter extends ThrowingAstVisitor<void> {
       }
     }
 
-    // TODO(scheglov) Dont write type, AKA separate true `int` and `double`?
+    // TODO(scheglov) Don't write type, AKA separate true `int` and `double`?
     _storeExpression(node);
   }
 

pkg/analyzer/lib/src/workspace/bazel_watcher.dart

@@ -432,7 +432,7 @@ abstract class PollTrigger {
 ///
 /// The idea here is to detect when Bazel finished running and use that to
 /// trigger polling. To detect that we use the `command.log` file that bazel
-/// contiuously updates as the build progresses. We find that file based on [1]:
+/// continuously updates as the build progresses. We find that file based on [1]:
 ///
 /// - In the workspace directory there should be a `bazel-out` symlink whose
 ///   target should be of the form:

pkg/async_helper/lib/async_helper.dart

@@ -89,7 +89,7 @@ Future<void> asyncTest(f()) {
   return f().then(asyncSuccess);
 }
 
-/// Verifies that the asyncronous [result] throws a [T].
+/// Verifies that the asynchronous [result] throws a [T].
 ///
 /// Fails if [result] completes with a value, or it completes with
 /// an error which is not a [T].

pkg/compiler/lib/src/elements/types.dart

@@ -2269,7 +2269,7 @@ abstract class DartTypes {
         type is FunctionTypeVariable && canAssignGenericFunctionTo(type.bound);
   }
 
-  /// Returns `true` if [type] occuring in a program with no sound null safety
+  /// Returns `true` if [type] occurring in a program with no sound null safety
   /// cannot accept `null` under sound rules.
   bool isNonNullableIfSound(DartType type) {
     if (type is DynamicType ||

pkg/compiler/lib/src/inferrer/powersets/powerset_bits.dart

@@ -523,7 +523,7 @@ class PowersetBitsDomain {
         // type and is maximal (i.e. instantiated to bounds), the typemask,
         // which is based on the class element, is still precise. We check
         // against Top for the parameter arguments since we don't have a
-        // convenient check for instantation to bounds.
+        // convenient check for instantiation to bounds.
         //
         // TODO(sra): Check arguments against bounds.
         // TODO(sra): Handle other variances.

pkg/compiler/lib/src/inferrer/type_system.dart

@@ -569,7 +569,7 @@ class TypeSystem {
     return _addPhi(node, variable, inputType, isTry);
   }
 
-  /// Simplies the phi representing [element] and of the type
+  /// Simplifies the phi representing [element] and of the type
   /// [phiType]. For example, if this phi has one incoming input, an
   /// implementation of this method could just return that incoming
   /// input type.

pkg/compiler/lib/src/inferrer/typemasks/masks.dart

@@ -352,7 +352,7 @@ class CommonMasks implements AbstractValueDomain {
         // type and is maximal (i.e. instantiated to bounds), the typemask,
         // which is based on the class element, is still precise. We check
         // against Top for the parameter arguments since we don't have a
-        // convenient check for instantation to bounds.
+        // convenient check for instantiation to bounds.
         //
         // TODO(sra): Check arguments against bounds.
         // TODO(sra): Handle other variances.

pkg/compiler/lib/src/io/position_information.dart

@@ -841,10 +841,10 @@ enum StepKind {
 
 /// Listener for the [JavaScriptTracer].
 abstract class TraceListener {
-  /// Called before [root] node is procesed by the [JavaScriptTracer].
+  /// Called before [root] node is processed by the [JavaScriptTracer].
   void onStart(js.Node root) {}
 
-  /// Called after [root] node has been procesed by the [JavaScriptTracer].
+  /// Called after [root] node has been processed by the [JavaScriptTracer].
   void onEnd(js.Node root) {}
 
   /// Called when a branch of the given [kind] is started. [value] is provided

pkg/compiler/lib/src/io/source_information.dart

@@ -66,7 +66,7 @@ abstract class SourceInformation extends JavaScriptNodeSourceInformation {
 /// Context information about inlined calls.
 ///
 /// This is associated with SourceInformation objects to be able to emit
-/// precise data about inlining that can then be used by defobuscation tools
+/// precise data about inlining that can then be used by deobfuscation tools
 /// when reconstructing a source stack from a production stack trace.
 class FrameContext {
   static const String tag = 'frame-context';

pkg/compiler/lib/src/ir/scope_visitor.dart

@@ -118,7 +118,7 @@ class ScopeModelBuilder extends ir.Visitor<EvaluationComplexity>
       assert(node is ir.Procedure || node is ir.Constructor);
       if (!(node is ir.Procedure && node.isRedirectingFactory)) {
         // Skip redirecting factories: they contain invalid expressions only
-        // used to suppport internal CFE modular compilation.
+        // used to support internal CFE modular compilation.
         node.accept(this);
       }
     }

pkg/compiler/lib/src/js/placeholder_safety.dart

@@ -10,7 +10,7 @@ typedef PositionPredicate = bool Function(int position);
 
 /// PlaceholderSafetyAnalysis determines which placeholders in a JavaScript
 /// template may be replaced with an arbitrary expression. Placeholders may be
-/// replaced with an arbitrary expression providied the template ensures the
+/// replaced with an arbitrary expression provided the template ensures the
 /// placeholders are evaluated in the same left-to-right order with no
 /// additional effects interleaved.
 ///

pkg/compiler/lib/src/js_backend/frequency_assignment.dart

@@ -176,7 +176,7 @@ class _Pool {
 /// entities are identified by integers.
 class _Cohort {
   _Cohort? next; // Next cohort in decreasing frequency.
-  final int count; // This is the cohort of items occuring [count] times.
+  final int count; // This is the cohort of items occurring [count] times.
   Set<int> unassigned = Set();
 
   _Cohort(this.count);

pkg/compiler/lib/src/js_backend/string_reference.dart

@@ -10,7 +10,7 @@
 ///     foo1() => 'A very long string';
 ///
 /// might be compiled to something like the following, where StringReference1 is
-/// assocatied with the string `'A very long string'`.
+/// associated with the string `'A very long string'`.
 ///
 ///     foo1: function() {
 ///       return StringReference1;
@@ -77,7 +77,7 @@ class StringReferencePolicy {
   // (hot, cold, execute-once, etc).
   static const int shortestSharedLength = 40;
 
-  // TODO(sra): Add policy for huge non-shared strings, strings occuring in
+  // TODO(sra): Add policy for huge non-shared strings, strings occurring in
   // run-once code, etc. Maybe make policy settings assignable for testing or
   // command-line configuration.
 }

pkg/compiler/lib/src/js_backend/type_reference.dart

@@ -390,7 +390,7 @@ class TypeReferenceFinalizerImpl implements TypeReferenceFinalizer {
       }
 
       // TODO(sra): There are other contexts that would be beneficial, e.g. a
-      // type reference occuring only in a throw expression.
+      // type reference occurring only in a throw expression.
 
       String suggestedName = _RecipeToIdentifier().run(referenceSet.recipe);
       if (usedNames.contains(suggestedName)) {

pkg/compiler/lib/src/js_model/type_recipe.dart

@@ -559,7 +559,7 @@ class _Substitution extends DartTypeSubstitutionVisitor<Null> {
     int count = _counts[variable] = (_counts[variable] ?? 0) + 1;
     if (count > 1) {
       // If the replacement is 'big', duplicating it can grow the term, perhaps
-      // exponentially given a sufficently pathological input.
+      // exponentially given a sufficiently pathological input.
       // TODO(sra): Fix exponential terms by encoding a DAG in recipes to avoid
       // linearization.
       if (replacement is FunctionType ||

pkg/compiler/tool/kernel_visitor/dart_html_metrics_visitor.dart

@@ -69,7 +69,7 @@ class MetricsVisitor extends RecursiveVisitor {
 
   @override
   void visitClass(Class node) {
-    // Dont want to add duplicate info.
+    // Don't want to add duplicate info.
     // When mixed, anonymous mixin class generated so we want to ignore.
     if (!node.isAnonymousMixin) {
       currentClass = node.name;

pkg/dartdev/lib/src/sdk.dart

@@ -114,7 +114,7 @@ class Runtime {
 
   static Runtime _createSingleton() {
     var versionString = Platform.version;
-    // Exepcted format: "version (channel) ..."
+    // Expected format: "version (channel) ..."
     var version = versionString;
     String? channel;
     var versionEnd = versionString.indexOf(' ');

pkg/frontend_server/lib/src/strong_components.dart

@@ -34,7 +34,7 @@ class StrongComponents {
   /// The Component that is being compiled.
   ///
   /// On incremental compiles, this will only contain the invalidated
-  /// lbraries.
+  /// libraries.
   final Component component;
 
   /// The libraries loaded from a dill file that should not be processed.

pkg/js/proposal.md

@@ -1370,7 +1370,7 @@ class MyDartClass extends TheirJSClass {
   final int x = 1;
   int y, z;
   factory MyDartClass(int y, int z) {
-    // super constructpr parameters passed here?
+    // super constructor parameters passed here?
     var self = createJS<MyDartClass>(); 
     self.y = y;
     self.z = z;

pkg/kernel/lib/transformations/track_widget_constructor_locations.dart

@@ -433,7 +433,7 @@ class WidgetCreatorTracker {
   ///
   /// Upon transformation the [changedStructureNotifier] (if provided) is used
   /// to notify the listener that  that class hierarchy of certain classes has
-  /// changed. This is neccesary for instance when doing an incremental
+  /// changed. This is necessary for instance when doing an incremental
   /// compilation where the class hierarchy is kept between compiles and thus
   /// has to be kept up to date.
   void transform(Component module, List<Library> libraries,

pkg/modular_test/lib/src/pipeline.dart

@@ -4,7 +4,7 @@
 
 /// Abstraction for a compilation pipeline.
 ///
-/// A pipeline defines how modular steps are excuted and ensures that a step
+/// A pipeline defines how modular steps are executed and ensures that a step
 /// only has access to the data it declares.
 ///
 /// The abstract implementation validates how the data is declared, and the
@@ -42,7 +42,7 @@ class ModularStep {
   /// Whether this step is only executed on the main module.
   final bool onlyOnMain;
 
-  /// Whether this step is only exceuted on the SDK.
+  /// Whether this step is only executed on the SDK.
   final bool onlyOnSdk;
 
   /// Whether this step is not executed on the SDK.

pkg/nnbd_migration/test/api_test.dart

@@ -111,7 +111,7 @@ abstract class _ProvisionalApiTestBase extends AbstractContextTest {
     }
   }
 
-  /// Verifies that migraiton of the single file with the given [content]
+  /// Verifies that migration of the single file with the given [content]
   /// produces the [expected] output.
   ///
   /// Optional parameter [removeViaComments] indicates whether dead code should

pkg/test_runner/lib/src/process_queue.dart

@@ -149,7 +149,7 @@ class ProcessQueue {
 
         eventFinishedTestCase(finishedTestCase);
       }, onDone: () {
-        // Wait until the commandQueue/exectutor is done (it may need to stop
+        // Wait until the commandQueue/executor is done (it may need to stop
         // batch runners, browser controllers, ....)
         commandQueue.done.then((_) {
           cancelDebugTimer();

pkg/test_runner/test/skipping_dart2js_compilations_test.dart

@@ -29,7 +29,7 @@ import 'package:test_runner/src/test_case.dart';
 
 import 'utils.dart';
 
-/// This class is reponsible for setting up the files necessary for this test
+/// This class is responsible for setting up the files necessary for this test
 /// as well as touching a file.
 class FileUtils {
   late Directory tempDir;

pkg/vm_service/lib/src/vm_service.dart

@@ -2442,7 +2442,7 @@ class RPCError implements Exception {
 
   String? get details => data == null ? null : data!['details'];
 
-  /// Return a map representation of this error suitable for converstion to
+  /// Return a map representation of this error suitable for conversion to
   /// json.
   Map<String, dynamic> toMap() {
     Map<String, dynamic> map = {

pkg/vm_service/tool/dart/generate_dart.dart

@@ -292,7 +292,7 @@ class RPCError implements Exception {
 
   String? get details => data == null ? null : data!['details'];
 
-  /// Return a map representation of this error suitable for converstion to
+  /// Return a map representation of this error suitable for conversion to
   /// json.
   Map<String, dynamic> toMap() {
     Map<String, dynamic> map = {

runtime/include/dart_api.h

@@ -1615,7 +1615,7 @@ DART_EXPORT DART_WARN_UNUSED_RESULT Dart_Handle Dart_RunLoop(void);
  * \param error A non-NULL pointer which will hold an error message if the call
  *   fails. The error has to be free()ed by the caller.
  *
- * \return If successfull the VM takes owernship of the isolate and takes care
+ * \return If successful the VM takes owernship of the isolate and takes care
  *   of its message loop. If not successful the caller retains owernship of the
  *   isolate.
  */

runtime/observatory/lib/src/elements/heap_snapshot.dart

@@ -418,7 +418,7 @@ class MergedDominatorDiff {
     for (var i = 0; i < worklist.length; i++) {
       worklist[i]._computeChildren(worklist);
     }
-    // Compute area botton-up.
+    // Compute area bottom-up.
     for (var i = worklist.length - 1; i >= 0; i--) {
       worklist[i]._computeArea();
     }

runtime/observatory/lib/src/elements/process_snapshot.dart

@@ -113,7 +113,7 @@ class ProcessItemDiff {
     for (var i = 0; i < worklist.length; i++) {
       worklist[i]._computeChildren(worklist);
     }
-    // Compute area botton-up.
+    // Compute area bottom-up.
     for (var i = worklist.length - 1; i >= 0; i--) {
       worklist[i]._computeArea();
     }