Annotate scroll* methods in chrome.automation as optional

R=anastasi@chromium.org

AX-Relnotes: n/a
Bug: 1100666
Change-Id: Id7020d13b9a7aa7cec33782d1513aa2bef8cae28
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2277870Reviewed-by: Anastasia Helfinstein <anastasi@google.com>
Commit-Queue: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#785820}

extensions/common/api/automation.idl

@@ -1167,22 +1167,22 @@ enum EventMoveDirectionType {
     static void suspendMedia();
 
     // Scrolls this scrollable container backward.
-    static void scrollBackward(PerformActionCallback callback);
+    static void scrollBackward(optional PerformActionCallback callback);
 
     // Scrolls this scrollable container forward.
-    static void scrollForward(PerformActionCallback callback);
+    static void scrollForward(optional PerformActionCallback callback);
 
     // Scrolls this scrollable container up.
-    static void scrollUp(PerformActionCallback callback);
+    static void scrollUp(optional PerformActionCallback callback);
 
     // Scrolls this scrollable container down.
-    static void scrollDown(PerformActionCallback callback);
+    static void scrollDown(optional PerformActionCallback callback);
 
     // Scrolls this scrollable container left.
-    static void scrollLeft(PerformActionCallback callback);
+    static void scrollLeft(optional PerformActionCallback callback);
 
     // Scrolls this scrollable container right.
-    static void scrollRight(PerformActionCallback callback);
+    static void scrollRight(optional PerformActionCallback callback);
 
     // Adds a listener for the given event type and event phase.
     static void addEventListener(

third_party/closure_compiler/externs/automation.js

@@ -512,9 +512,9 @@ chrome.automation.FindParams;
 
 /**
  * @typedef {{
- *   anchorObject: chrome.automation.AutomationNode,
+ *   anchorObject: AutomationNode,
  *   anchorOffset: number,
- *   focusObject: chrome.automation.AutomationNode,
+ *   focusObject: AutomationNode,
  *   focusOffset: number
  * }}
  * @see https://developer.chrome.com/extensions/automation#type-SetDocumentSelectionParams
@@ -1720,7 +1720,7 @@ chrome.automation.AutomationNode.prototype.suspendMedia = function() {};
 
 /**
  * Scrolls this scrollable container backward.
- * @param {function(boolean): void} callback Callback called for actions with a
+ * @param {function(boolean): void=} callback Callback called for actions with a
  *     response.
  * @see https://developer.chrome.com/extensions/automation#method-scrollBackward
  */
@@ -1728,7 +1728,7 @@ chrome.automation.AutomationNode.prototype.scrollBackward = function(callback) {
 
 /**
  * Scrolls this scrollable container forward.
- * @param {function(boolean): void} callback Callback called for actions with a
+ * @param {function(boolean): void=} callback Callback called for actions with a
  *     response.
  * @see https://developer.chrome.com/extensions/automation#method-scrollForward
  */
@@ -1736,7 +1736,7 @@ chrome.automation.AutomationNode.prototype.scrollForward = function(callback) {}
 
 /**
  * Scrolls this scrollable container up.
- * @param {function(boolean): void} callback Callback called for actions with a
+ * @param {function(boolean): void=} callback Callback called for actions with a
  *     response.
  * @see https://developer.chrome.com/extensions/automation#method-scrollUp
  */
@@ -1744,7 +1744,7 @@ chrome.automation.AutomationNode.prototype.scrollUp = function(callback) {};
 
 /**
  * Scrolls this scrollable container down.
- * @param {function(boolean): void} callback Callback called for actions with a
+ * @param {function(boolean): void=} callback Callback called for actions with a
  *     response.
  * @see https://developer.chrome.com/extensions/automation#method-scrollDown
  */
@@ -1752,7 +1752,7 @@ chrome.automation.AutomationNode.prototype.scrollDown = function(callback) {};
 
 /**
  * Scrolls this scrollable container left.
- * @param {function(boolean): void} callback Callback called for actions with a
+ * @param {function(boolean): void=} callback Callback called for actions with a
  *     response.
  * @see https://developer.chrome.com/extensions/automation#method-scrollLeft
  */
@@ -1760,7 +1760,7 @@ chrome.automation.AutomationNode.prototype.scrollLeft = function(callback) {};
 
 /**
  * Scrolls this scrollable container right.
- * @param {function(boolean): void} callback Callback called for actions with a
+ * @param {function(boolean): void=} callback Callback called for actions with a
  *     response.
  * @see https://developer.chrome.com/extensions/automation#method-scrollRight
  */