fix GridLayout::CreatePanel for new style dialogs

chrome/browser/ui/views/confirm_bubble_views.cc

Index: chrome/browser/ui/views/confirm_bubble_views.cc
diff --git a/chrome/browser/ui/views/confirm_bubble_views.cc b/chrome/browser/ui/views/confirm_bubble_views.cc
index 976659997cb81d3bea82492faf4c66dc3efb2bd6..e0ee172517f99c4732b7bf2461649c6f50c8b4d5 100644
--- a/chrome/browser/ui/views/confirm_bubble_views.cc
+++ b/chrome/browser/ui/views/confirm_bubble_views.cc
@@ -15,13 +15,7 @@
 ConfirmBubbleViews::ConfirmBubbleViews(ConfirmBubbleModel* model)
     : model_(model),
       link_(NULL) {
-  views::GridLayout* layout = new views::GridLayout(this);
-  // TODO(msw): Use layout constants and fix the new-style sizing.
-  layout->SetInsets(UseNewStyle() ? gfx::Insets(0, 0, 40, 0) :
-      gfx::Insets(views::kUnrelatedControlVerticalSpacing,
-                  views::kUnrelatedControlHorizontalSpacing,
-                  views::kUnrelatedControlVerticalSpacing,
-                  views::kUnrelatedControlHorizontalSpacing));
+  views::GridLayout* layout = views::GridLayout::CreatePanel(this);

[Evan Stade, 2013/04/16 01:39:39]

this is a behavioral change, but I think it's an improvement.

[msw, 2013/04/16 01:58:24]

A screenshot would help :)

[Evan Stade, 2013/04/16 19:58:25]

Done.

   SetLayoutManager(layout);
 
   // Use a fixed maximum message width, so longer messages will wrap.