Add autotestPrivate.swapWindowsInSplitView

Based on the discussion in https://crrev.com/c/1743869, we decided to
add autotestPrivate.swapWindowsInSplitView() instead of using automation
API in Tast.

Tast side CL: https://crrev.com/c/1716679

TEST=arc.SplitView
BUG=135150701

Change-Id: I9e96b01daf0c8c84dc77363727dd4e38244562c4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1758037Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Reviewed-by: Mitsuru Oshima (slow) <oshima@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#689581}

ash/public/cpp/split_view_test_api.h

@@ -26,6 +26,9 @@ class ASH_EXPORT SplitViewTestApi {
   // Snaps the window to left/right in the split view.
   void SnapWindow(aura::Window* window, SnapPosition snap_position);
 
+  // Swaps left and right windows in the split view.
+  void SwapWindows();
+
  private:
   DISALLOW_COPY_AND_ASSIGN(SplitViewTestApi);
 };

ash/wm/splitview/split_view_test_api.cc

@@ -31,4 +31,8 @@ void SplitViewTestApi::SnapWindow(
   Shell::Get()->split_view_controller()->SnapWindow(window, position);
 }
 
+void SplitViewTestApi::SwapWindows() {
+  Shell::Get()->split_view_controller()->SwapWindows();
+}
+
 }  // namespace ash

chrome/browser/chromeos/extensions/autotest_private/autotest_private_api.cc

@@ -2283,6 +2283,21 @@ AutotestPrivateGetArcAppWindowStateFunction::Run() {
           arc_window->GetProperty(ash::kWindowStateTypeKey))))));
 }
 
+///////////////////////////////////////////////////////////////////////////////
+// AutotestPrivateSwapWindowsInSplitViewFunction
+///////////////////////////////////////////////////////////////////////////////
+
+AutotestPrivateSwapWindowsInSplitViewFunction::
+    AutotestPrivateSwapWindowsInSplitViewFunction() = default;
+AutotestPrivateSwapWindowsInSplitViewFunction::
+    ~AutotestPrivateSwapWindowsInSplitViewFunction() = default;
+
+ExtensionFunction::ResponseAction
+AutotestPrivateSwapWindowsInSplitViewFunction::Run() {
+  ash::SplitViewTestApi().SwapWindows();
+  return RespondNow(NoArguments());
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 // AutotestPrivateAPI
 ///////////////////////////////////////////////////////////////////////////////

chrome/browser/chromeos/extensions/autotest_private/autotest_private_api.h

@@ -743,6 +743,17 @@ class AutotestPrivateGetArcAppWindowInfoFunction : public ExtensionFunction {
   ResponseAction Run() override;
 };
 
+class AutotestPrivateSwapWindowsInSplitViewFunction : public ExtensionFunction {
+ public:
+  AutotestPrivateSwapWindowsInSplitViewFunction();
+  DECLARE_EXTENSION_FUNCTION("autotestPrivate.swapWindowsInSplitView",
+                             AUTOTESTPRIVATE_SWAPWINDOWSINSPLITVIEW)
+
+ private:
+  ~AutotestPrivateSwapWindowsInSplitViewFunction() override;
+  ResponseAction Run() override;
+};
+
 template <>
 KeyedService*
 BrowserContextKeyedAPIFactory<AutotestPrivateAPI>::BuildServiceInstanceFor(

chrome/common/extensions/api/autotest_private.idl

@@ -531,5 +531,9 @@ namespace autotestPrivate {
     // |callback|: called when the operation has completed.
     static void getArcAppWindowInfo(DOMString packageName,
                                     GetArcAppWindowInfoCallback callback);
+
+    // Swap the windows in the split view.
+    // |callback|: Called when the operation has completed.
+    static void swapWindowsInSplitView(VoidCallback callback);
   };
 };

extensions/browser/extension_function_histogram_value.h

@@ -1437,6 +1437,7 @@ enum HistogramValue {
   AUTOTESTPRIVATE_GETARCAPPWINDOWSTATE = 1374,
   AUTOTESTPRIVATE_GETARCAPPWINDOWINFO = 1375,
   AUTOTESTPRIVATE_GETALLINSTALLEDAPPS = 1376,
+  AUTOTESTPRIVATE_SWAPWINDOWSINSPLITVIEW = 1377,
   // Last entry: Add new entries above, then run:
   // python tools/metrics/histograms/update_extension_histograms.py
   ENUM_BOUNDARY

tools/metrics/histograms/enums.xml

@@ -20356,6 +20356,7 @@ Called by update_net_error_codes.py.-->
   <int value="1374" label="AUTOTESTPRIVATE_GETARCAPPWINDOWSTATE"/>
   <int value="1375" label="AUTOTESTPRIVATE_GETARCAPPWINDOWINFO"/>
   <int value="1376" label="AUTOTESTPRIVATE_GETALLINSTALLEDAPPS"/>
+  <int value="1377" label="AUTOTESTPRIVATE_SWAPWINDOWSINSPLITVIEW"/>
 </enum>
 
 <enum name="ExtensionIconState">