Converts most of WorkspaceLayoutManager tests to use common code

ash/common/test/ash_test.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ASH_COMMON_TEST_ASH_TEST_H_
 #define ASH_COMMON_TEST_ASH_TEST_H_
 
 #include <memory>
 #include <string>
 
 #include "ash/common/shell_window_ids.h"
 #include "base/macros.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/display/manager/display_layout.h"
 #include "ui/gfx/geometry/rect.h"
+#include "ui/views/widget/widget.h"
 #include "ui/wm/public/window_types.h"
 
 namespace display {
 class Display;
 }
 
 namespace ash {
 
 class AshTestImpl;
+class WmShelf;
 class WmWindow;
 
 // Wraps a WmWindow calling WmWindow::Destroy() from the destructor. WmWindow is
 // owned by the corresponding window implementation. The only way to delete
 // WmWindow is to call WmWindow::Destroy(), which deletes the corresponding
 // window, then the WmWindow. This class calls WmWindow::Destroy() from its
 // destructor.
 class WindowOwner {
  public:
   explicit WindowOwner(WmWindow* window);
@@ skipping 12 matching lines @@
 // ash/common and run in both mus and aura.
 //
 // The implementation of AshTestImpl that is used depends upon gn targets. To
 // use the aura backend depend on "//ash:ash_with_aura_test_support." The mus
 // backend is not provided as a separate link target.
 class AshTest : public testing::Test {
  public:
   AshTest();
   ~AshTest() override;
 
+  // Cover functions for WmShell::Get()->IsRunningInMash().

[msw, 2016/10/06 20:18:24]

nit: "Cover functions"? maybe just "Calls WmShell::IsRunningInMash()."?
(it's also short enough to just inline everywhere [+wm_shell.h include])

[sky, 2016/10/06 21:05:58]

Done.

+  static bool IsRunningInMash();
+
+  // Returns the WmShelf for the primary display.
+  static WmShelf* GetPrimaryShelf();
+
   bool SupportsMultipleDisplays() const;
 
   // Update the display configuration as given in |display_spec|.
   // See test::DisplayManagerTestApi::UpdateDisplay for more details.
   void UpdateDisplay(const std::string& display_spec);
 
   // Creates a top level visible window in the appropriate container. If

[msw, 2016/10/06 20:18:24]

I see that James already asked about this, but I need to ask. Does this not
actually/always create a top-level window? Otherwise, what's the practical
difference for users between this and CreateToplevelTestWindow? Should the aura
impl of CreateTestWindow just always provide a valid delegate for
ui::wm::WINDOW_TYPE_NORMAL? Should the function comment here mention the
conditions in which the window will be top-level?
also nit: 'top-level' to be consistent.

[sky, 2016/10/06 21:05:58]

I removed the mention to top-level here. I would rather keep these two functions
separate as it'll make the porting easier. To do otherwise means more time
porting each test, which is already painful.

   // |bounds_in_screen| is empty the window is added to the primary root window,
   // otherwise the window is added to the display matching |bounds_in_screen|.
   // |shell_window_id| is the shell window id to give to the new window.
   std::unique_ptr<WindowOwner> CreateTestWindow(
       const gfx::Rect& bounds_in_screen = gfx::Rect(),
       ui::wm::WindowType type = ui::wm::WINDOW_TYPE_NORMAL,
       int shell_window_id = kShellWindowId_Invalid);
 
+  // Creates a visible top-level window. For aura a top-level window is a Window
+  // that has a delegate, see aura::Window::GetToplevelWindow() for more
+  // details.
+  std::unique_ptr<WindowOwner> CreateToplevelTestWindow(
+      const gfx::Rect& bounds_in_screen = gfx::Rect(),
+      int shell_window_id = kShellWindowId_Invalid);
+
   // Creates a visible window parented to |parent| with the specified bounds and
   // id.
   std::unique_ptr<WindowOwner> CreateChildWindow(
       WmWindow* parent,
       const gfx::Rect& bounds = gfx::Rect(),
       int shell_window_id = kShellWindowId_Invalid);
 
   // Returns the Display for the secondary display. It's assumed there are two
   // displays.
   display::Display GetSecondaryDisplay();
 
   // Sets the placement of the secondary display. Returns true if the secondary
   // display can be moved, false otherwise. The false return value is temporary
   // until mus fully supports this.
   bool SetSecondaryDisplayPlacement(
       display::DisplayPlacement::Position position,
       int offset);
 
+  // Configures |init_params| so that the widget will be created on the same
+  // display as |window|.
+  void ConfigureWidgetInitParamsForDisplay(
+      WmWindow* window,
+      views::Widget::InitParams* init_params);
+
+  // Adds |window| to the appropriate container in the primary root window.
+  void ParentWindowInPrimaryRootWindow(WmWindow* window);
+
+  // Adds |window| as as a transient child of |parent|.
+  void AddTransientChild(WmWindow* parent, WmWindow* window);
+
+  // Helper for calling TestSessionStateDelegate::SetCanLockScreen(). Controls
+  // the return value from SessionStateDelegate::CanLockScreen().
+  void SetCanLockScreen(bool can_lock_screen);

[msw, 2016/10/06 20:18:24]

This seems a little unnecessary as an AshTest function; maybe it would be more
fitting as a static function on TestSessionStateDelegate, in some ash_test_utils
file or similar, if not simply inlined in the few places that need it.

[sky, 2016/10/06 21:05:58]

Agreed. Done.

+
+  void RunAllPendingInMessageLoop();
+
  protected:
   // testing::Test:
   void SetUp() override;
   void TearDown() override;
 
  private:
   std::unique_ptr<AshTestImpl> test_impl_;
 
   DISALLOW_COPY_AND_ASSIGN(AshTest);
 };
 
 }  // namespace ash
 
 #endif  // ASH_COMMON_TEST_ASH_TEST_H_