Refactor ViewsDelegate and MD-ify the icon-to-text spacing for checkbox and radiobutton

chrome/browser/ui/views/chrome_views_delegate.cc

 // 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.
 
 #include "chrome/browser/ui/views/chrome_views_delegate.h"
 
 #include <memory>
 
 #include "base/logging.h"
 #include "base/single_thread_task_runner.h"
@@ skipping 39 matching lines @@
 // multi-profile environment.
 PrefService* GetPrefsForWindow(const views::Widget* window) {
   Profile* profile = GetProfileForWindow(window);
   if (!profile) {
     // Use local state for windows that have no explicit profile.
     return g_browser_process->local_state();
   }
   return profile->GetPrefs();
 }
 
+#if !defined(OS_WIN)
+ChromeViewsDelegate* views_delegate = nullptr;
+#endif
+
 }  // namespace
 
 
 // ChromeViewsDelegate --------------------------------------------------------
 
 #if !defined(OS_WIN)
-ChromeViewsDelegate::ChromeViewsDelegate() {}
-#endif
+ChromeViewsDelegate::ChromeViewsDelegate() {
+  DCHECK(!views_delegate);
+  views_delegate = this;
+}
 
 ChromeViewsDelegate::~ChromeViewsDelegate() {
   DCHECK_EQ(0u, ref_count_);
+
+  DCHECK_EQ(this, views_delegate);
+  views_delegate = nullptr;
 }
 
+ChromeViewsDelegate* ChromeViewsDelegate::GetInstance() {
+  return views_delegate;
+}
+#endif
+
 void ChromeViewsDelegate::SaveWindowPlacement(const views::Widget* window,
                                               const std::string& window_name,
                                               const gfx::Rect& bounds,
                                               ui::WindowShowState show_state) {
   PrefService* prefs = GetPrefsForWindow(window);
   if (!prefs)
     return;
 
   std::unique_ptr<DictionaryPrefUpdate> pref_update =
       chrome::GetWindowPlacementDictionaryReadWrite(window_name, prefs);
@@ skipping 106 matching lines @@
 
 std::string ChromeViewsDelegate::GetApplicationName() {
   return version_info::GetProductName();
 }
 
 scoped_refptr<base::TaskRunner>
 ChromeViewsDelegate::GetBlockingPoolTaskRunner() {
   return content::BrowserThread::GetBlockingPool();
 }
 
-gfx::Insets ChromeViewsDelegate::GetDialogButtonInsets() const {
+gfx::Insets ChromeViewsDelegate::GetInsetsMetric(
+    views::InsetsMetric metric) const {
   const LayoutDelegate* layout_delegate = LayoutDelegate::Get();
-  const int top = layout_delegate->GetMetric(
-      LayoutDelegate::Metric::DIALOG_BUTTON_TOP_SPACING);
-  const int margin = layout_delegate->GetMetric(
-      LayoutDelegate::Metric::DIALOG_BUTTON_MARGIN);
-  return gfx::Insets(top, margin, margin, margin);
+  switch (metric) {
+    case views::InsetsMetric::DIALOG_BUTTON: {
+      const int top = layout_delegate->GetMetric(
+          LayoutDelegate::Metric::DIALOG_BUTTON_TOP_SPACING);
+      const int margin = layout_delegate->GetMetric(
+          LayoutDelegate::Metric::DIALOG_BUTTON_MARGIN);
+      return gfx::Insets(top, margin, margin, margin);
+    }
+    case views::InsetsMetric::DIALOG_FRAME_VIEW: {
+      const int top = layout_delegate->GetMetric(
+          LayoutDelegate::Metric::PANEL_CONTENT_MARGIN);
+      const int side = layout_delegate->GetMetric(
+          LayoutDelegate::Metric::DIALOG_BUTTON_MARGIN);
+      // Titles are inset at the top and sides, but not at the bottom.
+      return gfx::Insets(top, side, 0, side);
+    }
+    case views::InsetsMetric::BUBBLE_DIALOG:
+      return gfx::Insets(layout_delegate->GetMetric(
+          LayoutDelegate::Metric::PANEL_CONTENT_MARGIN));
+  }
+  NOTREACHED();
+  return gfx::Insets();
 }
 
-int ChromeViewsDelegate::GetDialogCloseButtonMargin() const {
-  return LayoutDelegate::Get()->GetMetric(
-      LayoutDelegate::Metric::DIALOG_CLOSE_BUTTON_MARGIN);
+int ChromeViewsDelegate::GetDistanceMetric(views::DistanceMetric metric) const {
+  switch (metric) {
+    case views::DistanceMetric::CLOSE_BUTTON_MARGIN:
+      return LayoutDelegate::Get()->GetMetric(
+          LayoutDelegate::Metric::DIALOG_CLOSE_BUTTON_MARGIN);
+    case views::DistanceMetric::RELATED_BUTTON_HORIZONTAL:
+      return LayoutDelegate::Get()->GetMetric(
+          LayoutDelegate::Metric::RELATED_BUTTON_HORIZONTAL_SPACING);
+    case views::DistanceMetric::RELATED_CONTROL_HORIZONTAL:
+      return LayoutDelegate::Get()->GetMetric(
+          LayoutDelegate::Metric::RELATED_CONTROL_HORIZONTAL_SPACING);
+    case views::DistanceMetric::RELATED_CONTROL_VERTICAL:
+      return LayoutDelegate::Get()->GetMetric(
+          LayoutDelegate::Metric::RELATED_CONTROL_VERTICAL_SPACING);
+    case views::DistanceMetric::DIALOG_BUTTON_MINIMUM_WIDTH:
+      return LayoutDelegate::Get()->GetMetric(
+          LayoutDelegate::Metric::DIALOG_BUTTON_MINIMUM_WIDTH);
+    case views::DistanceMetric::BUTTON_HORIZONTAL_PADDING:
+      return LayoutDelegate::Get()->GetMetric(
+          LayoutDelegate::Metric::BUTTON_HORIZONTAL_PADDING);
+  }
+  NOTREACHED();
+  return 0;
 }
 
-int ChromeViewsDelegate::GetDialogRelatedButtonHorizontalSpacing() const {
-  return LayoutDelegate::Get()->GetMetric(
-      LayoutDelegate::Metric::RELATED_BUTTON_HORIZONTAL_SPACING);
-}
-
-int ChromeViewsDelegate::GetDialogRelatedControlVerticalSpacing() const {
-  return LayoutDelegate::Get()->GetMetric(
-      LayoutDelegate::Metric::RELATED_CONTROL_VERTICAL_SPACING);
-}
-
-gfx::Insets ChromeViewsDelegate::GetDialogFrameViewInsets() const {
-  const LayoutDelegate* layout_delegate = LayoutDelegate::Get();
-  const int top = layout_delegate->GetMetric(
-      LayoutDelegate::Metric::PANEL_CONTENT_MARGIN);
-  const int side = layout_delegate->GetMetric(
-      LayoutDelegate::Metric::DIALOG_BUTTON_MARGIN);
-  // Titles are inset at the top and sides, but not at the bottom.
-  return gfx::Insets(top, side, 0, side);
-}
-
-gfx::Insets ChromeViewsDelegate::GetBubbleDialogMargins() const {
-  return gfx::Insets(LayoutDelegate::Get()->GetMetric(
-      LayoutDelegate::Metric::PANEL_CONTENT_MARGIN));
-}
-
-int ChromeViewsDelegate::GetButtonMinimumWidth() const {
-  return LayoutDelegate::Get()->GetMetric(
-      LayoutDelegate::Metric::BUTTON_MINIMUM_WIDTH);
-}
-
-int ChromeViewsDelegate::GetDialogButtonMinimumWidth() const {
-  return LayoutDelegate::Get()->GetMetric(
-      LayoutDelegate::Metric::DIALOG_BUTTON_MINIMUM_WIDTH);
-}
-
-int ChromeViewsDelegate::GetButtonHorizontalPadding() const {
-  return LayoutDelegate::Get()->GetMetric(
-      LayoutDelegate::Metric::BUTTON_HORIZONTAL_PADDING);
+int ChromeViewsDelegate::GetDefaultDistanceMetric(
+    views::DistanceMetric metric) const {
+  return views::ViewsDelegate::GetDistanceMetric(metric);
 }
 
 #if !defined(OS_CHROMEOS)
 views::Widget::InitParams::WindowOpacity
 ChromeViewsDelegate::GetOpacityForInitParams(
     const views::Widget::InitParams& params) {
   return views::Widget::InitParams::OPAQUE_WINDOW;
 }
 #endif