Correct padding and focus ring for frameless constrained window dialog

chrome/browser/ui/constrained_window.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_CONSTRAINED_WINDOW_H_
 #define CHROME_BROWSER_UI_CONSTRAINED_WINDOW_H_
 
 #include "build/build_config.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/native_widget_types.h"
 
 ///////////////////////////////////////////////////////////////////////////////
 // ConstrainedWindow
 //
 //  This interface represents a window that is constrained to a
 //  WebContentsView's bounds.
 //
 class ConstrainedWindow {
  public:
-  static const int kVerticalPadding = 14; // top/bottom padding.
-  static const int kHorizontalPadding = 17; // left/right padding.
-  static const int kRowPadding = 20;  // Vertical margin between dialog rows.
-  static const int kBorderRadius = 2;  // Border radius for dialog corners.
+  static const int kTitleTopPadding = 15; // Padding above the title.

[Ben Goodger (Google), 2012/10/12 16:48:59]

see my comment in another CL. I think you should move this and all the other
shared style hints to a constrained_window_constants.h file, since it's
unrelated to the interface of creating/showing a constrained window and is in
fact just a shared implementation detail between platforms.

[please use gerrit instead, 2012/10/12 18:56:08]

Done.

+  static const int kHorizontalPadding = 20; // Left and right padding.
+  static const int kClientTopPadding = 18; // Padding above the client view.
+  static const int kClientBottomPadding = 20; // Padding below the client view.
+  static const int kCloseButtonPadding = 7; // Padding around the close button.
+  static const int kBorderRadius = 2; // Border radius for dialog corners.
+
+#if defined(OS_MACOSX)
+  static const int kRowPadding = 20; // Padding between rows of text.
+#endif
 
   // Font style for dialog text.
   static const ui::ResourceBundle::FontStyle kTextFontStyle =
       ui::ResourceBundle::BaseFont;
   // Font style for bold dialog text.
   static const ui::ResourceBundle::FontStyle kBoldTextFontStyle =
       ui::ResourceBundle::BoldFont;
   // Font style for dialog title.
   static const ui::ResourceBundle::FontStyle kTitleFontStyle =
       ui::ResourceBundle::MediumFont;
@@ skipping 21 matching lines @@
   virtual bool CanShowConstrainedWindow();
 
   // Returns the native window of the constrained window.
   virtual gfx::NativeWindow GetNativeWindow();
 
  protected:
   virtual ~ConstrainedWindow() {}
 };
 
 #endif  // CHROME_BROWSER_UI_CONSTRAINED_WINDOW_H_