fix some typos in code and documentation (#11009)

asolntsev · web-flow · commit c06370910c80 · 2022-09-06T19:41:33.000-05:00
diff --git a/README.md b/README.md
@@ -122,7 +122,7 @@ bazel test //java/... --test_size_filters=small,medium,large --test_tag_filters=
 ```
 
 The `test_size_filters` argument takes small, medium, large. Small are akin to unit tests,
-medium is akin to integration tests, and large is akin to end to end tests.
+medium is akin to integration tests, and large is akin to end-to-end tests.
 
 The `test_tag_filters` allow us to pass in browser names and a few different tags that we can
 find in the code base.
diff --git a/common/devtools/browser_protocol.json b/common/devtools/browser_protocol.json
@@ -6461,11 +6461,11 @@
                 },
                 {
                     "name": "clearDeviceMetricsOverride",
-                    "description": "Clears the overriden device metrics."
+                    "description": "Clears the overridden device metrics."
                 },
                 {
                     "name": "clearGeolocationOverride",
-                    "description": "Clears the overriden Geolocation Position and Error."
+                    "description": "Clears the overridden Geolocation Position and Error."
                 },
                 {
                     "name": "resetPageScaleFactor",
@@ -6779,7 +6779,7 @@
                         },
                         {
                             "name": "initialVirtualTime",
-                            "description": "If set, base::Time::Now will be overriden to initially return this value.",
+                            "description": "If set, base::Time::Now will be overridden to initially return this value.",
                             "optional": true,
                             "$ref": "Network.TimeSinceEpoch"
                         }
@@ -12383,7 +12383,7 @@
                 },
                 {
                     "name": "clearDeviceMetricsOverride",
-                    "description": "Clears the overriden device metrics.",
+                    "description": "Clears the overridden device metrics.",
                     "experimental": true,
                     "deprecated": true,
                     "redirect": "Emulation"
@@ -12397,7 +12397,7 @@
                 },
                 {
                     "name": "clearGeolocationOverride",
-                    "description": "Clears the overriden Geolocation Position and Error.",
+                    "description": "Clears the overridden Geolocation Position and Error.",
                     "deprecated": true,
                     "redirect": "Emulation"
                 },
@@ -17189,4 +17189,4 @@
             ]
         }
     ]
-}
+}
diff --git a/common/devtools/chromium/v85/browser_protocol.pdl b/common/devtools/chromium/v85/browser_protocol.pdl
@@ -2969,10 +2969,10 @@ domain Emulation
       # True if emulation is supported.
       boolean result
 
-  # Clears the overriden device metrics.
+  # Clears the overridden device metrics.
   command clearDeviceMetricsOverride
 
-  # Clears the overriden Geolocation Position and Error.
+  # Clears the overridden Geolocation Position and Error.
   command clearGeolocationOverride
 
   # Requests that page scale factor is reset to initial values.
@@ -3120,7 +3120,7 @@ domain Emulation
       # If set the virtual time policy change should be deferred until any frame starts navigating.
       # Note any previous deferred policy change is superseded.
       optional boolean waitForNavigation
-      # If set, base::Time::Now will be overriden to initially return this value.
+      # If set, base::Time::Now will be overridden to initially return this value.
       optional Network.TimeSinceEpoch initialVirtualTime
     returns
       # Absolute timestamp at which virtual time was first enabled (up time in milliseconds).
@@ -5776,7 +5776,7 @@ domain Page
       # Serialized page data.
       string data
 
-  # Clears the overriden device metrics.
+  # Clears the overridden device metrics.
   experimental deprecated command clearDeviceMetricsOverride
     # Use 'Emulation.clearDeviceMetricsOverride' instead
     redirect Emulation
@@ -5786,7 +5786,7 @@ domain Page
     # Use 'DeviceOrientation.clearDeviceOrientationOverride' instead
     redirect DeviceOrientation
 
-  # Clears the overriden Geolocation Position and Error.
+  # Clears the overridden Geolocation Position and Error.
   deprecated command clearGeolocationOverride
     # Use 'Emulation.clearGeolocationOverride' instead
     redirect Emulation
diff --git a/common/src/web/rc/tests/TestGet.html b/common/src/web/rc/tests/TestGet.html
@@ -28,7 +28,7 @@
       </td>
     </tr>
     <!-- test API getters -->
-    
+
     <tr>
       <td>open</td>
       <td>../tests/html/test_get.html</td>
@@ -37,13 +37,13 @@
 
     <!-- IE uppercases the property names of the style.
          Firefox lowercases the property names of the style.
-         IE omits the trailing semi-colon. -->
+         IE omits the trailing semicolon. -->
     <tr>
       <td>verifyAttribute</td>
       <td>//img[@alt='banner']@style</td>
       <td>regexp:(width|WIDTH): 644px; (height|HEIGHT): 41px(;?)</td>
     </tr>
-    
+
     <!-- This asserts on the current behavior of selArrayToString(). Commas and
          backslashes are escaped in array values. Backslash-craziness!! -->
     <tr>
diff --git a/java/src/org/openqa/selenium/WebElement.java b/java/src/org/openqa/selenium/WebElement.java
@@ -140,7 +140,7 @@ default String getDomAttribute(String name) {
    * neither exists, null is returned.
    * <p>
    * The "style" attribute is converted as best can be to a text representation with a trailing
-   * semi-colon.
+   * semicolon.
    * <p>
    * The following are deemed to be "boolean" attributes, and will return either "true" or null:
    * <p>
diff --git a/java/src/org/openqa/selenium/grid/data/SessionRequest.java b/java/src/org/openqa/selenium/grid/data/SessionRequest.java
@@ -84,7 +84,7 @@ public SessionRequest(
     Map<String, Object> metadata,
     Map<String, String> traceHeaders) {
     this.requestId = Require.nonNull("Request ID", requestId);
-    this.enqueued = Require.nonNull("Enqueud time", enqueued);
+    this.enqueued = Require.nonNull("Enqueued time", enqueued);
     this.downstreamDialects = unmodifiableSet(
       new HashSet<>(Require.nonNull("Downstream dialects", downstreamDialects)));
     this.desiredCapabilities = unmodifiableSet(
diff --git a/java/src/org/openqa/selenium/grid/sessionmap/jdbc/JdbcBackedSessionMap.java b/java/src/org/openqa/selenium/grid/sessionmap/jdbc/JdbcBackedSessionMap.java
@@ -117,7 +117,7 @@ public static SessionMap create(Config config) {
   public boolean isReady() {
     try {
       return !connection.isClosed();
-    } catch (SQLException throwables) {
+    } catch (SQLException throwable) {
       return false;
     }
   }
diff --git a/java/src/org/openqa/selenium/remote/http/jdk/JdkHttpClient.java b/java/src/org/openqa/selenium/remote/http/jdk/JdkHttpClient.java
@@ -168,7 +168,7 @@ public WebSocket send(Message message) {
           CloseMessage closeMessage = (CloseMessage) message;
           makeCall = () -> underlyingSocket.sendClose(closeMessage.code(), closeMessage.reason());
         } else {
-          throw new IllegalArgumentException("Unsupport message type: " + message);
+          throw new IllegalArgumentException("Unsupported message type: " + message);
         }
 
         synchronized (underlyingSocket) {
diff --git a/java/src/org/openqa/selenium/support/ui/ExpectedConditions.java b/java/src/org/openqa/selenium/support/ui/ExpectedConditions.java
@@ -1230,7 +1230,7 @@ public String toString() {
    *
    * @param element     used as parent element. For example table with locator By.xpath("//table")
    * @param childLocator used to find child element. For example td By.xpath("./tr/td")
-   * @return visible subelement
+   * @return visible sub-element
    */
   public static ExpectedCondition<List<WebElement>> visibilityOfNestedElementsLocatedBy(
     final WebElement element, final By childLocator) {
@@ -1261,7 +1261,7 @@ public String toString() {
    * @param locator     used to check parent element. For example table with locator
    *                    By.xpath("//table")
    * @param childLocator used to find child element. For example td By.xpath("./tr/td")
-   * @return subelement
+   * @return sub-element
    */
   public static ExpectedCondition<WebElement> presenceOfNestedElementLocatedBy(
     final By locator, final By childLocator)
@@ -1285,7 +1285,7 @@ public String toString() {
    *
    * @param element     used as parent element
    * @param childLocator used to find child element. For example td By.xpath("./tr/td")
-   * @return subelement
+   * @return sub-element
    */
   public static ExpectedCondition<WebElement> presenceOfNestedElementLocatedBy(
     final WebElement element, final By childLocator)
@@ -1311,7 +1311,7 @@ public String toString() {
    * @param parent     used to check parent element. For example table with locator
    *                    By.xpath("//table")
    * @param childLocator used to find child element. For example td By.xpath("./tr/td")
-   * @return subelement
+   * @return sub-element
    */
   public static ExpectedCondition<List<WebElement>> presenceOfNestedElementsLocatedBy(
     final By parent, final By childLocator)
diff --git a/java/test/com/thoughtworks/selenium/corebased/TestGet.java b/java/test/com/thoughtworks/selenium/corebased/TestGet.java
@@ -31,7 +31,7 @@ public void testGet() {
     selenium.open("test_get.html");
     // IE uppercases the property names of the style.
     // Firefox lowercases the property names of the style.
-    // IE omits the trailing semi-colon.
+    // IE omits the trailing semicolon.
     verifyTrue(Pattern.compile("(width|WIDTH): 644px; (height|HEIGHT): 41px(;?)")
         .matcher(selenium.getAttribute("//img[@alt='banner']@style")).find());
     // This asserts on the current behavior of selArrayToString(). Commas and
diff --git a/java/test/org/openqa/selenium/TextHandlingTest.java b/java/test/org/openqa/selenium/TextHandlingTest.java
@@ -171,7 +171,7 @@ public void testShouldRetainTheFormattingOfTextWithinAPreElement() {
 
   @Test
   @NotYetImplemented(value = SAFARI, reason = "getText does not normalize spaces")
-  public void testShouldRetainTheFormatingOfTextWithinAPreElementThatIsWithinARegularBlock() {
+  public void testShouldRetainTheFormattingOfTextWithinAPreElementThatIsWithinARegularBlock() {
     driver.get(pages.simpleTestPage);
     String text = driver.findElement(By.id("div-with-pre")).getText();
 
diff --git a/java/test/org/openqa/selenium/json/JsonTest.java b/java/test/org/openqa/selenium/json/JsonTest.java
@@ -336,7 +336,7 @@ public void shouldConvertAnArrayBackIntoAnArray() {
   }
 
   @Test
-  public void sShouldBeAbleToReconsituteASessionId() {
+  public void sShouldBeAbleToReconstituteASessionId() {
     String json = new Json().toJson(new SessionId("id"));
     SessionId sessionId = new Json().toType(json, SessionId.class);
 
diff --git a/javascript/atoms/domcore.js b/javascript/atoms/domcore.js
@@ -48,7 +48,7 @@ goog.require('goog.dom.TagName');
  * of a boolean attribute that is present will often be the empty string.
  *
  * <p>For the style attribute, it standardizes the value by lower-casing the
- * property names and always including a trailing semi-colon.
+ * property names and always including a trailing semicolon.
  *
  * @param {!Element} element The element to use.
  * @param {string} attributeName The name of the attribute to return.
@@ -107,7 +107,7 @@ bot.dom.core.SPLIT_STYLE_ATTRIBUTE_ON_SEMICOLONS_REGEXP_ =
 /**
  * Standardize a style attribute value, which includes:
  *  (1) converting all property names lowercase
- *  (2) ensuring it ends in a trailing semi-colon
+ *  (2) ensuring it ends in a trailing semicolon
  * @param {string} value The style attribute value.
  * @return {string} The identical value, with the formatting rules described
  *     above applied.
diff --git a/javascript/atoms/test/click_submit_test.html b/javascript/atoms/test/click_submit_test.html
@@ -92,7 +92,7 @@
       var targetHref = buildAbsoluteUrl('/testdata/click_destination.html');
 
       return new goog.Promise(function(done) {
-        // As the iframe loads asyncronously we need to wait to check out
+        // As the iframe loads asynchronously we need to wait to check out
         // if the url matches the expected one.
         goog.events.listenOnce(iframe, 'load', done);
         action(clickTarget);
@@ -141,7 +141,7 @@
       var clickTarget = findElement({'id': 'submitForm'}, iframeDoc);
 
       return new goog.Promise(function(done) {
-        // As the iframe loads asyncronously we need to wait to check out
+        // As the iframe loads asynchronously we need to wait to check out
         // if the url matches the expected one.
         goog.events.listenOnce(iframe, 'load', done);
         action(clickTarget);
diff --git a/javascript/atoms/test/text_test.html b/javascript/atoms/test/text_test.html
@@ -232,13 +232,13 @@
       assertEquals("Some text", text);
     }
 
-    function testShouldRetainTheFormatingOfTextWithinAPreElement() {
+    function testShouldRetainTheFormattingOfTextWithinAPreElement() {
       var text = getVisibleTextByElementId("preformatted");
 
       assertEquals("   This section has a preformatted\n    text block    \n  split in four lines\n         ", text);
     }
 
-    function testShouldRetainTheFormatingOfTextWithinAPreElementThatIsWithinARegularBlock() {
+    function testShouldRetainTheFormattingOfTextWithinAPreElementThatIsWithinARegularBlock() {
       var text = getVisibleTextByElementId("div-with-pre");
 
       assertEquals("before pre\n   This section has a preformatted\n    text block    \n  split in four lines\n         \nafter pre", text);
@@ -263,7 +263,7 @@
       assertEquals("hello, world! bla-bla-bla", text);
       text = getVisibleTextByElementId("uppercased");
       assertEquals("HELLO, WORLD! BLA-BLA-BLA", text);
-      
+
       text = getVisibleTextByElementId("capitalized-1");
       assertEquals("Äåìî", text);
       text = getVisibleTextByElementId("capitalized-2");
@@ -362,7 +362,7 @@
   <div id="div-with-pre">
     <p>before pre</p>
     <pre id="preformatted">   This section has a preformatted
-    text block    
+    text block
   split in four lines
          </pre>
     <p>after pre</p>
diff --git a/javascript/node/selenium-webdriver/lib/webdriver.js b/javascript/node/selenium-webdriver/lib/webdriver.js
@@ -2707,7 +2707,7 @@ class WebElement {
    * the value of the property with the same name is returned. If neither value
    * is set, null is returned (for example, the "value" property of a textarea
    * element). The "style" attribute is converted as best can be to a
-   * text representation with a trailing semi-colon. The following are deemed to
+   * text representation with a trailing semicolon. The following are deemed to
    * be "boolean" attributes and will return either "true" or null:
    *
    * async, autofocus, autoplay, checked, compact, complete, controls, declare,
diff --git a/rb/lib/selenium/webdriver/common/element.rb b/rb/lib/selenium/webdriver/common/element.rb
@@ -101,7 +101,7 @@ def tag_name
       # if it exists. If it does not, then the value of the attribute with the given name is returned.
       # If neither exists, null is returned.
       #
-      # The "style" attribute is converted as best can be to a text representation with a trailing semi-colon.
+      # The "style" attribute is converted as best can be to a text representation with a trailing semicolon.
       #
       # The following are deemed to be "boolean" attributes, and will return either "true" or "false":
       #

Original file line number	Diff line number	Diff line change
`@@ -6461,11 +6461,11 @@`
`6461`	`6461`	`},`
`6462`	`6462`	`{`
`6463`	`6463`	`"name": "clearDeviceMetricsOverride",`
`6464`		`- "description": "Clears the overriden device metrics."`
	`6464`	`+ "description": "Clears the overridden device metrics."`
`6465`	`6465`	`},`
`6466`	`6466`	`{`
`6467`	`6467`	`"name": "clearGeolocationOverride",`
`6468`		`- "description": "Clears the overriden Geolocation Position and Error."`
	`6468`	`+ "description": "Clears the overridden Geolocation Position and Error."`
`6469`	`6469`	`},`
`6470`	`6470`	`{`
`6471`	`6471`	`"name": "resetPageScaleFactor",`
`@@ -6779,7 +6779,7 @@`
`6779`	`6779`	`},`
`6780`	`6780`	`{`
`6781`	`6781`	`"name": "initialVirtualTime",`
`6782`		`- "description": "If set, base::Time::Now will be overriden to initially return this value.",`
	`6782`	`+ "description": "If set, base::Time::Now will be overridden to initially return this value.",`
`6783`	`6783`	`"optional": true,`
`6784`	`6784`	`"$ref": "Network.TimeSinceEpoch"`
`6785`	`6785`	`}`
`@@ -12383,7 +12383,7 @@`
`12383`	`12383`	`},`
`12384`	`12384`	`{`
`12385`	`12385`	`"name": "clearDeviceMetricsOverride",`
`12386`		`- "description": "Clears the overriden device metrics.",`
	`12386`	`+ "description": "Clears the overridden device metrics.",`
`12387`	`12387`	`"experimental": true,`
`12388`	`12388`	`"deprecated": true,`
`12389`	`12389`	`"redirect": "Emulation"`
`@@ -12397,7 +12397,7 @@`
`12397`	`12397`	`},`
`12398`	`12398`	`{`
`12399`	`12399`	`"name": "clearGeolocationOverride",`
`12400`		`- "description": "Clears the overriden Geolocation Position and Error.",`
	`12400`	`+ "description": "Clears the overridden Geolocation Position and Error.",`
`12401`	`12401`	`"deprecated": true,`
`12402`	`12402`	`"redirect": "Emulation"`
`12403`	`12403`	`},`
`@@ -17189,4 +17189,4 @@`
`17189`	`17189`	`]`
`17190`	`17190`	`}`
`17191`	`17191`	`]`
`17192`		`-}`
	`17192`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ public static SessionMap create(Config config) {`
`117`	`117`	`public boolean isReady() {`
`118`	`118`	`try {`
`119`	`119`	`return !connection.isClosed();`
`120`		`- } catch (SQLException throwables) {`
	`120`	`+ } catch (SQLException throwable) {`
`121`	`121`	`return false;`
`122`	`122`	`}`
`123`	`123`	`}`
Original file line number	Diff line number	Diff line change
`@@ -168,7 +168,7 @@ public WebSocket send(Message message) {`
`168`	`168`	`CloseMessage closeMessage = (CloseMessage) message;`
`169`	`169`	`makeCall = () -> underlyingSocket.sendClose(closeMessage.code(), closeMessage.reason());`
`170`	`170`	`} else {`
`171`		`- throw new IllegalArgumentException("Unsupport message type: " + message);`
	`171`	`+ throw new IllegalArgumentException("Unsupported message type: " + message);`
`172`	`172`	`}`
`173`	`173`
`174`	`174`	`synchronized (underlyingSocket) {`