Place newly created app windows on screen.

chrome/browser/extensions/platform_app_browsertest_util.cc

Index: chrome/browser/extensions/platform_app_browsertest_util.cc
diff --git a/chrome/browser/extensions/platform_app_browsertest_util.cc b/chrome/browser/extensions/platform_app_browsertest_util.cc
index a815326e36a2a8dbb5f759157e82ed08fdebb69b..f94537d020e9e5b023804f05137ae6228d80794c 100644
--- a/chrome/browser/extensions/platform_app_browsertest_util.cc
+++ b/chrome/browser/extensions/platform_app_browsertest_util.cc
@@ -163,6 +163,20 @@ void PlatformAppBrowserTest::CloseShellWindow(ShellWindow* window) {
   destroyed_observer.Wait();
 }
 
+void PlatformAppBrowserTest::CallAdjustBoundsToBeVisibleOnScreenForShellWinow(
+    ShellWindow* window,
+    const gfx::Rect& cached_bounds,
+    const gfx::Rect& cached_screen_bounds,
+    const gfx::Rect& current_screen_bounds,
+    const gfx::Size& minimum_size,
+    gfx::Rect* bounds) {
+  window->AdjustBoundsToBeVisibleOnScreen(cached_bounds,

[benwells, 2013/06/22 00:56:02]

nit: why do you need this function in the utility class? Can't you just call
window->Adjust....(...) in the call sites instead?

If you feel it is much nicer like this, consider making it static. 

[scheib, 2013/06/23 04:43:37]

Done.

[scheib, 2013/06/23 05:38:06]

Ah, needed to access private function, using PlatformAppBrowserTest's friend
status. I'm leaving it as is, but fixing the name.

+                                          cached_screen_bounds,
+                                          current_screen_bounds,
+                                          minimum_size,
+                                          bounds);
+}
+
 void ExperimentalPlatformAppBrowserTest::SetUpCommandLine(
     CommandLine* command_line) {
   PlatformAppBrowserTest::SetUpCommandLine(command_line);