Remove a new usage of LabelButton::SetStyle (in ChromeCleanerDialog).

chrome/browser/ui/views/chrome_cleaner_dialog.cc

 // Copyright 2017 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_cleaner_dialog.h"
 
 #include "base/strings/string16.h"
 #include "chrome/browser/safe_browsing/chrome_cleaner_dialog_controller.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "components/constrained_window/constrained_window_views.h"
 #include "ui/base/ui_base_types.h"
 #include "ui/events/event.h"
 #include "ui/gfx/native_widget_types.h"
 #include "ui/gfx/text_constants.h"
+#include "ui/views/controls/button/md_text_button.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/layout_constants.h"
 #include "ui/views/widget/widget.h"
 
 namespace chrome {
 
 void ShowChromeCleanerPrompt(
     Browser* browser,
     safe_browsing::ChromeCleanerDialogController* controller) {
   ChromeCleanerDialog* dialog = new ChromeCleanerDialog(controller);
   dialog->Show(browser);
 }
 
 }  // namespace chrome
 
 namespace {
 constexpr int kDialogWidth = 448;
 }  // namespace
 
 ////////////////////////////////////////////////////////////////////////////////
 // ChromeCleanerDialog
 
 ChromeCleanerDialog::ChromeCleanerDialog(
     safe_browsing::ChromeCleanerDialogController* controller)
     : browser_(nullptr),
       controller_(controller),
-      advanced_button_(
-          new views::LabelButton(this, controller_->GetAdvancedButtonLabel())) {
+      advanced_button_(MdTextButton::CreateSecondaryUiButton(

[tapted, 2017/05/03 01:36:02]

optional: the member isn't really needed..
ChromeCleanerDialog::CreateExtraView() could just `return
MdTextButton::CreateSecondaryUiButton(..)` (|advanced_button_| is only used for
a DCHECK otherwise)

[Evan Stade, 2017/05/03 15:10:39]

I'm on the fence because I think that DCHECK is mildly useful, but done.

+          this,
+          controller_->GetAdvancedButtonLabel())) {
   DCHECK(controller_);
 
   SetLayoutManager(new views::BoxLayout(
       /*orientation=*/views::BoxLayout::kVertical,
       /*inside_border_horizontal_spacing=*/views::kButtonHEdgeMarginNew,
       /*inside_border_vertical_spacing=*/views::kPanelVertMargin,
       /*between_child_spacing=*/0));
   views::Label* label = new views::Label(controller_->GetMainText());
   label->SetMultiLine(true);
   label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
   AddChildView(label);
-
-  advanced_button_->SetStyle(views::Button::STYLE_BUTTON);
 }
 
 ChromeCleanerDialog::~ChromeCleanerDialog() {
   // Make sure the controller is correctly notified in case the dialog widget is
   // closed by some other means than the dialog buttons.
   if (controller_)
     controller_->Cancel();
 }
 
 void ChromeCleanerDialog::Show(Browser* browser) {
@@ skipping 79 matching lines @@
   DCHECK(browser_);
 
   // TODO(alito): Navigate to the webui version of the Chrome Cleaner UI when
   // that is implemented.
   if (controller_) {
     controller_->AdvancedButtonClicked();
     controller_ = nullptr;
   }
   GetWidget()->Close();
 }