GTK implementation of remembering platform app window geometry.

chrome/browser/extensions/platform_app_browsertest.cc

 // Copyright (c) 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 "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/automation/automation_util.h"
 #include "chrome/browser/tab_contents/render_view_context_menu.h"
 #include "chrome/browser/extensions/extension_test_message_listener.h"
 #include "chrome/browser/extensions/platform_app_browsertest_util.h"
 #include "chrome/browser/extensions/shell_window_registry.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/extensions/application_launch.h"
 #include "chrome/browser/ui/extensions/shell_window.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/render_process_host.h"
+#include "base/threading/platform_thread.h"
 
 using content::WebContents;
 
 namespace extensions {
 
 namespace {
 // Non-abstract RenderViewContextMenu class.
 class PlatformAppContextMenu : public RenderViewContextMenu {
  public:
   PlatformAppContextMenu(WebContents* web_contents,
@@ skipping 374 matching lines @@
       content::Source<content::WebContentsDelegate>(browser()));
   LoadAndLaunchPlatformApp("open_link");
   observer.Wait();
   ASSERT_EQ(2, browser()->tab_count());
 }
 
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, MutationEventsDisabled) {
   ASSERT_TRUE(RunPlatformAppTest("platform_apps/mutation_events")) << message_;
 }
 
+// Only implemented in GTK so far.
+#if defined(TOOLKIT_GTK)
+// Test that windows created with an id will remember and restore their
+// geometry when opening new windows.
+IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, ShellWindowRestorePosition) {
+  ExtensionTestMessageListener page2_listener("WaitForPage2", true);
+  ExtensionTestMessageListener page3_listener("WaitForPage3", true);
+  ExtensionTestMessageListener done_listener("Done1", false);
+  ExtensionTestMessageListener done2_listener("Done2", false);
+  ExtensionTestMessageListener done3_listener("Done3", false);
+
+  ASSERT_TRUE(LoadAndLaunchPlatformApp("geometry"));
+
+  // Wait for the app to be launched (although this is mostly to have a
+  // message to reply to to let the script know it should create its second
+  // window.
+  ASSERT_TRUE(page2_listener.WaitUntilSatisfied());
+
+  // Wait for the first window to verify its geometry was correctly set
+  // from the default* attributes passed to the create function.
+  ASSERT_TRUE(done_listener.WaitUntilSatisfied());
+
+  // Programatically move and resize the window.
+  ShellWindow* window = GetFirstShellWindow();
+  ASSERT_TRUE(window);
+  gfx::Rect bounds(137, 143, 203, 187);
+  window->GetBaseWindow()->SetBounds(bounds);
+
+#if defined(TOOLKIT_GTK)
+  // TODO(mek): On GTK we have to wait for a roundtrip to the X server before
+  // a resize actually happens:
+  // "if you call gtk_window_resize() then immediately call
+  //  gtk_window_get_size(), the size won't have taken effect yet. After the
+  //  window manager processes the resize request, GTK+ receives notification
+  //  that the size has changed via a configure event, and the size of the
+  //  window gets updated."
+  // Because of this we have to wait for an unknown time for the resize to
+  // actually take effect. So wait upto 10 seconds or until the resize got
+  // handled.
+  while (bounds != window->GetBaseWindow()->GetBounds()) {
+    content::RunAllPendingInMessageLoop();
+  }
+
+  // In the GTK ShellWindow implementation there also is a delay between
+  // getting the correct bounds and it calling SaveWindowPosition, so call that
+  // method explicitly to make sure the value was stored.
+  window->SaveWindowPosition();

[Marijn Kruisselbrink, 2012/08/30 23:26:35]

alternatively I could make the test a friend of ShellWindowGtk and do a
static_cast<ShellWindowGtk*>(window->GetBaseWindow())->OnDebouncedBoundsChanged()
but that seems even uglier.

[Evan Stade, 2012/08/30 23:55:38]

meh, it's ok, it's a test.

+#endif  // defined(TOOLKIT_GTK)
+
+  // Make sure the window was properly moved&resized.
+  ASSERT_EQ(bounds, window->GetBaseWindow()->GetBounds());
+
+  // Tell javascript to open a second window.
+  page2_listener.Reply("continue");
+
+  // Wait for javascript to verify that the second window got the updated
+  // coordinates, ignoring the default coordinates passed to the create method.
+  ASSERT_TRUE(done2_listener.WaitUntilSatisfied());
+
+  // Tell javascript to open a third window.
+  page3_listener.Reply("continue");
+
+  // Wait for javascript to verify that the third window got the restored size
+  // and explicitly specified coordinates.
+  ASSERT_TRUE(done3_listener.WaitUntilSatisfied());
+}
+#endif  // defined(TOOLKIT_GTK)
+
 }  // namespace extensions