Initial constants for new ConstrainedWindow style.

chrome/browser/ui/constrained_window.h

Index: chrome/browser/ui/constrained_window.h
diff --git a/chrome/browser/ui/constrained_window.h b/chrome/browser/ui/constrained_window.h
index 720ccb897a812652b0bf83fd206bcae7cddac4aa..794adb92d25118207661459bd529f72869802d54 100644
--- a/chrome/browser/ui/constrained_window.h
+++ b/chrome/browser/ui/constrained_window.h
@@ -6,6 +6,8 @@
 #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"
 
 ///////////////////////////////////////////////////////////////////////////////
@@ -16,6 +18,21 @@
 //
 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.
+
+  // Font style for dialog text.
+  static const ui::ResourceBundle::FontStyle kTextFontStyle =
+      ui::ResourceBundle::BaseFont;

[Mike Wittman, 2012/08/14 00:14:23]

On Windows, the specified size of the text font appears to be 1 point larger
than BaseFont.  I don't know if it's the same relationship on Mac or ChromeOS.

While testing this in Views out I created a function:
gfx::Font ConstrainedWindowViews::GetBodyFont()
{
  return ui::ResourceBundle::GetSharedInstance().GetFont(
      ui::ResourceBundle::BaseFont).DeriveFont(1);
}

I *think* this is similar to what Ben was describing in his comment.

[groby-ooo-7-16, 2012/08/14 00:20:33]

Clarifying w/ UI leads. Last word was "use BaseFont/MediumFont"

On 2012/08/14 00:14:23, Mike Wittman wrote:

+  // Font style for dialog title.
+  static const ui::ResourceBundle::FontStyle kTitleFontStyle =
+      ui::ResourceBundle::MediumFont;

[Mike Wittman, 2012/08/14 00:14:23]

Similarly, on Windows the title font is 1 point larger than MediumFont.

+
+   SkColor GetBackgroundColor();  // Dialog background color.
+   SkColor GetTextColor();  // Dialog text color.
+
   // Makes the Constrained Window visible. Only one Constrained Window is shown
   // at a time per tab.
   virtual void ShowConstrainedWindow() = 0;
@@ -33,4 +50,8 @@ class ConstrainedWindow {
   virtual ~ConstrainedWindow() {}
 };
 
+namespace ConstrainedWindowStyle {

[Mike Wittman, 2012/08/14 00:14:23]

Delete this namespace?

[groby-ooo-7-16, 2012/08/14 00:20:33]

Done.

+
+}
+
 #endif  // CHROME_BROWSER_UI_CONSTRAINED_WINDOW_H_