Implement MD specs for non-tabbed windows in Ash

chrome/browser/ui/browser_browsertest.cc

Index: chrome/browser/ui/browser_browsertest.cc
diff --git a/chrome/browser/ui/browser_browsertest.cc b/chrome/browser/ui/browser_browsertest.cc
index 2519d1540be478a9cbdffda58137f82f2b478b60..75389c7a5b894bd1e2808317ba675ccb4a538e3c 100644
--- a/chrome/browser/ui/browser_browsertest.cc
+++ b/chrome/browser/ui/browser_browsertest.cc
@@ -3213,11 +3213,19 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, DISABLED_ChangeDisplayMode) {
 // Test to ensure the bounds of popup, devtool, and app windows are properly
 // restored.
 IN_PROC_BROWSER_TEST_F(BrowserTest, TestPopupBounds) {
+  // TODO(tdanderson|pkasting): Change the first test to verify that the

[Peter Kasting, 2016/02/10 23:46:21]

Nit: "the first test" -> "this", since there are actually three EXPECTs below
and we'd need to change both first and second.

[tdanderson, 2016/02/11 16:10:21]

Done.

+  //                            contents bounds set by params.initial_bounds

[Peter Kasting, 2016/02/10 23:46:21]

Nit: Don't indent the text on these lines

[tdanderson, 2016/02/11 16:10:21]

Done.

+  //                            are the same as the contents bounds in the
+  //                            initialized window. See crbug.com/585856.
   {
     // Minimum size that a popup window should have appended to its height when
     // drawn (popup window bounds are for the content, not the window). This is
     // the size of the toolbar on views platforms.
+#if defined(OS_CHROMEOS)

[Peter Kasting, 2016/02/10 23:46:21]

Nit: I would eliminate the ifdef here, use the CrOS value in all cases (since
the 29 case is already only applicable to Windows, not Mac, so it's already not
a tight bound), and change the comment above -- which is inaccurate anyway -- to
read:

Minimum height a popup window should have added to the supplied content bounds
when drawn.  This accommodates the browser toolbar.

[tdanderson, 2016/02/11 16:10:21]

Done.

+    const int minimum_popup_padding = 27;
+#else
     const int minimum_popup_padding = 29;
+#endif  // defined(OS_CHROMEOS)
 
     // Creates an untrusted popup window and asserts that the eventual height is
     // padded with the toolbar and title bar height (initial height is content