fix GridLayout::CreatePanel for new style dialogs

ui/views/layout/grid_layout.cc

Index: ui/views/layout/grid_layout.cc
diff --git a/ui/views/layout/grid_layout.cc b/ui/views/layout/grid_layout.cc
index ae8ba02e46ec9c0e7c167e2d5366d014b47ec82a..a5f308bcfab9d9e6e44cb99e4c53e24c2a16988b 100644
--- a/ui/views/layout/grid_layout.cc
+++ b/ui/views/layout/grid_layout.cc
@@ -11,6 +11,7 @@
 #include "ui/gfx/insets.h"
 #include "ui/views/layout/layout_constants.h"
 #include "ui/views/view.h"
+#include "ui/views/window/dialog_delegate.h"
 
 namespace views {
 
@@ -675,8 +676,12 @@ GridLayout::~GridLayout() {
 // static
 GridLayout* GridLayout::CreatePanel(View* host) {
   GridLayout* layout = new GridLayout(host);
-  layout->SetInsets(kPanelVertMargin, kPanelHorizMargin,
-                    kPanelVertMargin, kPanelHorizMargin);
+
+  int horizontal_margin = DialogDelegate::UseNewStyle() ?

[msw, 2013/04/15 22:48:37]

I'm don't think we should change the default GridLayout margin. Other individual
host Views should probably call SetInsets() with their desired values.

[Evan Stade, 2013/04/15 23:52:53]

this is not for all GridLayouts. It's only for "panels", i.e. GridLayouts
created via CreatePanel. Only dialogs should be using this function.

[msw, 2013/04/16 00:53:21]

Ah, I didn't realize that. Then this seems reasonable.

+      kPanelHorizMarginNewStyle : kPanelHorizMargin;
+
+  layout->SetInsets(kPanelVertMargin, horizontal_margin,
+                    kPanelVertMargin, horizontal_margin);
   return layout;
 }