cros: Disable the auto management logic for v1app browser window

chrome/browser/ui/browser_browsertest.cc

 // Copyright 2012 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.
 
 #include "chrome/browser/ui/browser.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <memory>
@@ skipping 106 matching lines @@
 #if defined(OS_MACOSX)
 #include "base/mac/scoped_nsautorelease_pool.h"
 #include "chrome/browser/ui/cocoa/test/run_loop_testing.h"
 #endif
 
 #if defined(OS_WIN)
 #include "base/i18n/rtl.h"
 #include "chrome/browser/browser_process.h"
 #endif
 
+#if defined(OS_CHROMEOS)
+#include "ui/aura/window.h"
+#include "ui/display/display.h"
+#include "ui/display/screen.h"
+#endif
+
 using app_modal::AppModalDialog;
 using app_modal::AppModalDialogQueue;
 using app_modal::JavaScriptAppModalDialog;
 using base::ASCIIToUTF16;
 using content::InterstitialPage;
 using content::HostZoomMap;
 using content::NavigationController;
 using content::NavigationEntry;
 using content::OpenURLParams;
 using content::Referrer;
@@ skipping 2798 matching lines @@
     Browser* browser = new Browser(params);
     gfx::Rect bounds = browser->window()->GetBounds();
 
     // Should be EXPECT_EQ, but this width is inconsistent across platforms.
     // See https://crbug.com/567925.
     EXPECT_GE(bounds.width(), 100);
     EXPECT_EQ(122, bounds.height());
     browser->window()->Close();
   }
 }
+
+#if defined(OS_CHROMEOS)
+namespace {
+
+class BrowserTestParam : public InProcessBrowserTest,
+                         public testing::WithParamInterface<bool> {
+ public:
+  bool TestApp() { return GetParam(); }
+};
+
+}  // namespace
+
+// Test that in Chrome OS, for app browser (v1app) windows, the auto management

[sky, 2017/02/13 23:35:51]

We try to avoid platform specific code in c/b/ui. Platform specific code goes in
the appropriate subdirectories. So, how about putting this in
browser_frame_ash_browsertest?

[Qiang(Joe) Xu, 2017/02/14 00:00:20]

sure, updated.

+// logic is disabled while for tabbed browser windows, it is enabled.
+IN_PROC_BROWSER_TEST_P(BrowserTestParam,
+                       TabbedOrAppBrowserWindowAutoManagementTest) {
+  // Default |browser()| is not used by this test.
+  browser()->window()->Close();
+
+  // Open a new browser window (app or tabbed depending on a parameter).
+  bool test_app = TestApp();
+  Browser::CreateParams params =
+      test_app ? Browser::CreateParams::CreateForApp(
+                     "test_browser_app", true /* trusted_source */, gfx::Rect(),
+                     browser()->profile())
+               : Browser::CreateParams(browser()->profile());
+  params.initial_show_state = ui::SHOW_STATE_DEFAULT;
+  Browser* browser = new Browser(params);
+  gfx::NativeWindow window = browser->window()->GetNativeWindow();
+  gfx::Rect original_bounds(gfx::Rect(150, 250, 400, 100));
+  window->SetBounds(original_bounds);
+  window->Show();
+
+  // For tabbed browser window, it will be centered to work area by auto window
+  // mangement logic; for app browser window, it will remain the given bounds.
+  gfx::Rect work_area = display::Screen::GetScreen()
+                            ->GetDisplayNearestPoint(window->bounds().origin())
+                            .work_area();
+  gfx::Rect tabbed_expected_bounds = work_area;
+  tabbed_expected_bounds.ClampToCenteredSize(original_bounds.size());
+  tabbed_expected_bounds.set_y(original_bounds.y());
+  if (test_app)
+    EXPECT_EQ(original_bounds.ToString(), window->bounds().ToString());
+  else
+    EXPECT_EQ(tabbed_expected_bounds.ToString(), window->bounds().ToString());
+
+  // Close the browser and re-create the browser window with the same app name.
+  browser->window()->Close();
+  browser = new Browser(params);
+  browser->window()->Show();
+
+  // The newly created browser window should remain the same bounds for each.
+  window = browser->window()->GetNativeWindow();
+  if (test_app)
+    EXPECT_EQ(original_bounds.ToString(), window->bounds().ToString());
+  else
+    EXPECT_EQ(tabbed_expected_bounds.ToString(), window->bounds().ToString());
+}
+
+INSTANTIATE_TEST_CASE_P(BrowserTestTabbedOrApp,
+                        BrowserTestParam,
+                        testing::Bool());
+
+#endif  // defined(OS_CHROMEOS)