Chromium Code Reviews| Index: ui/views/window/dialog_client_view.cc |
| diff --git a/ui/views/window/dialog_client_view.cc b/ui/views/window/dialog_client_view.cc |
| index a9414fb419fdfd6e68bf511993ba4fda84e86a7d..0e4a4c3415af6a765b7d4ebe7eb65868bde23ab6 100644 |
| --- a/ui/views/window/dialog_client_view.cc |
| +++ b/ui/views/window/dialog_client_view.cc |
| @@ -282,13 +282,8 @@ void DialogClientView::UpdateDialogButton(LabelButton** member, |
| (type != ui::DIALOG_BUTTON_CANCEL || |
| PlatformStyle::kDialogDefaultButtonCanBeCancel); |
| - // The default button is always blue in Harmony. |
| - if (is_default && (ui::MaterialDesignController::IsSecondaryUiMaterial() || |
|
sky
2017/06/06 00:00:49
I'm assuming we don't care about ui::MaterialDesig
afakhry
2017/06/06 00:20:35
Right, in this instance we don't, as we want the d
|
| - delegate->ShouldDefaultButtonBeBlue())) { |
| - button = MdTextButton::CreateSecondaryUiBlueButton(this, title); |
| - } else { |
| - button = MdTextButton::CreateSecondaryUiButton(this, title); |
| - } |
| + button = is_default ? MdTextButton::CreateSecondaryUiBlueButton(this, title) |
| + : MdTextButton::CreateSecondaryUiButton(this, title); |
| const int minimum_width = LayoutProvider::Get()->GetDistanceMetric( |
| views::DISTANCE_DIALOG_BUTTON_MINIMUM_WIDTH); |