Harmony: Apply the upper bound on equal-sized button widths in DialogClientView.

ui/views/window/dialog_client_view.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 "ui/views/window/dialog_client_view.h"
 
 #include <algorithm>
 
 #include "build/build_config.h"
 #include "ui/base/material_design/material_design_controller.h"
@@ skipping 320 matching lines @@
   // when hidden it's not included in the button row (it can't influence layout)
   // and it can't be added to |button_row_container_| (GridLayout complains).
   // So add it, hidden, to |this| so it can be observed.
   if (extra_view_ && !views[0])
     AddChildView(extra_view_);
 
   if (std::count(views.begin(), views.end(), nullptr) == kNumButtons)
     return;
 
   gfx::Insets insets = button_row_insets_;
+  LayoutProvider* const layout_provider = LayoutProvider::Get();
   // Support dialogs that clear |button_row_insets_| to do their own layout.
   // They expect GetDialogRelatedControlVerticalSpacing() in this case.
   // TODO(tapted): Remove this under Harmony.
   if (insets.top() == 0) {
-    const int top = LayoutProvider::Get()->GetDistanceMetric(
-        views::DISTANCE_RELATED_CONTROL_VERTICAL);
+    const int top =
+        layout_provider->GetDistanceMetric(DISTANCE_RELATED_CONTROL_VERTICAL);
     insets.Set(top, insets.left(), insets.bottom(), insets.right());
   }
 
   // The |resize_percent| constants. There's only one stretchy column (padding
   // to the left of ok/cancel buttons).
   constexpr float kFixed = 0.f;
   constexpr float kStretchy = 1.f;
 
   // Button row is [ extra <pad+stretchy> second <pad> third ]. Ensure the <pad>
   // column is zero width if there isn't a button on either side.
   // GetExtraViewSpacing() handles <pad+stretchy>.
-  const int button_spacing =
-      (ok_button_ && cancel_button_)
-          ? LayoutProvider::Get()->GetDistanceMetric(
-                views::DISTANCE_RELATED_BUTTON_HORIZONTAL)
-          : 0;
+  const int button_spacing = (ok_button_ && cancel_button_)
+                                 ? layout_provider->GetDistanceMetric(
+                                       DISTANCE_RELATED_BUTTON_HORIZONTAL)
+                                 : 0;
 
   constexpr int kButtonRowId = 0;
   ColumnSet* column_set = layout->AddColumnSet(kButtonRowId);
 
   // Rather than giving |button_row_container_| a Border, incorporate the insets
   // into the layout. This simplifies min/max size calculations.
   column_set->AddPaddingColumn(kFixed, insets.left());
   column_set->AddColumn(GridLayout::FILL, GridLayout::FILL, kFixed,
                         GridLayout::USE_PREF, 0, 0);
   column_set->AddPaddingColumn(kStretchy, GetExtraViewSpacing());
@@ skipping 12 matching lines @@
   layout->StartRowWithPadding(kFixed, kButtonRowId, kFixed, insets.top());
   for (size_t view_index = 0; view_index < kNumButtons; ++view_index) {
     if (views[view_index]) {
       layout->AddView(views[view_index]);
       link[link_index++] = kViewToColumnIndex[view_index];
     } else {
       layout->SkipColumns(1);
     }
   }
 
-  if (ui::MaterialDesignController::IsSecondaryUiMaterial()) {
-    // If |views[0]| is non-null, it is a visible |extra_view_| and its column
-    // will be in |link[0]|. Skip that if it is not a button, or if it is a
-    // Checkbox (which extends LabelButton). Otherwise, link everything.
-    bool skip_first_link =
-        views[0] && (!CustomButton::AsCustomButton(views[0]) ||
-                     views[0]->GetClassName() == Checkbox::kViewClassName);
-    if (skip_first_link)
-      column_set->LinkColumnSizes(link[1], link[2], -1);
-    else
-      column_set->LinkColumnSizes(link[0], link[1], link[2], -1);
-  }
+  column_set->set_linked_column_size_limit(
+      layout_provider->GetDistanceMetric(DISTANCE_BUTTON_MAX_LINKABLE_WIDTH));
+
+  // If |views[0]| is non-null, it is a visible |extra_view_| and its column
+  // will be in |link[0]|. Skip that if it is not a button, or if it is a
+  // Checkbox (which extends LabelButton). Otherwise, link everything.
+  bool skip_first_link =
+      views[0] && (!CustomButton::AsCustomButton(views[0]) ||
+                   views[0]->GetClassName() == Checkbox::kViewClassName);
+  if (skip_first_link)
+    column_set->LinkColumnSizes(link[1], link[2], -1);
+  else
+    column_set->LinkColumnSizes(link[0], link[1], link[2], -1);
+
   layout->AddPaddingRow(kFixed, insets.bottom());
 }
 
 void DialogClientView::SetupViews() {
   button_row_container_->RemoveAllChildViews(false /* delete children */);
   // If SetupLayout() "stored" a hidden |extra_view_| in |this|, ensure it can
   // be re-added to the layout when becoming visible.
   if (extra_view_)
     RemoveChildView(extra_view_);
 
   UpdateDialogButton(&ok_button_, ui::DIALOG_BUTTON_OK);
   UpdateDialogButton(&cancel_button_, ui::DIALOG_BUTTON_CANCEL);
 
   if (extra_view_)
     return;
 
   extra_view_ = GetDialogDelegate()->CreateExtraView();
   if (extra_view_)
     extra_view_->SetGroup(kButtonGroup);
 }
 
 }  // namespace views