Harmony - convert hung renderer dialog.

chrome/browser/ui/views/harmony/harmony_layout_delegate.cc

Index: chrome/browser/ui/views/harmony/harmony_layout_delegate.cc
diff --git a/chrome/browser/ui/views/harmony/harmony_layout_delegate.cc b/chrome/browser/ui/views/harmony/harmony_layout_delegate.cc
index eb1744312dc81b417ddc686c76ef3fcb861ada54..45cb24b059f216a32cfa5b124dbefd104427b4c7 100644
--- a/chrome/browser/ui/views/harmony/harmony_layout_delegate.cc
+++ b/chrome/browser/ui/views/harmony/harmony_layout_delegate.cc
@@ -17,8 +17,14 @@ HarmonyLayoutDelegate* HarmonyLayoutDelegate::Get() {
 
 int HarmonyLayoutDelegate::GetMetric(Metric metric) const {
   switch (metric) {
+    case Metric::BUTTON_HORIZONTAL_PADDING:
+      return kHarmonyLayoutUnit;
     case Metric::DIALOG_BUTTON_MARGIN:
       return kHarmonyLayoutUnit;
+    case Metric::DIALOG_BUTTON_MINIMUM_WIDTH:
+      // Minimum label size plus padding.

[Peter Kasting, 2017/02/16 01:14:47]

Maybe we should just expose the minimum label width directly?  I dunno

[Bret, 2017/02/16 01:31:04]

I think this is okay.

+      return 2 * kHarmonyLayoutUnit +
+             2 * GetMetric(Metric::BUTTON_HORIZONTAL_PADDING);
     case Metric::DIALOG_BUTTON_TOP_SPACING:
       return kHarmonyLayoutUnit;
     case Metric::DIALOG_CLOSE_BUTTON_MARGIN:
@@ -37,6 +43,10 @@ int HarmonyLayoutDelegate::GetMetric(Metric metric) const {
       return kHarmonyLayoutUnit;
     case Metric::SUBSECTION_HORIZONTAL_INDENT:
       return 0;
+    case Metric::UNRELATED_CONTROL_HORIZONTAL_SPACING:
+      return kHarmonyLayoutUnit;
+    case Metric::UNRELATED_CONTROL_HORIZONTAL_SPACING_LARGE:
+      return kHarmonyLayoutUnit;
     case Metric::UNRELATED_CONTROL_VERTICAL_SPACING:
       return kHarmonyLayoutUnit;
     case Metric::UNRELATED_CONTROL_VERTICAL_SPACING_LARGE: