views: Migrate the cases of "new .*NativeTextButton(" to LabelButton.

chrome/browser/ui/views/download/download_item_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 "chrome/browser/ui/views/download/download_item_view.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 20 matching lines @@
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/base/animation/slide_animation.h"
 #include "ui/base/events/event.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/base/text/text_elider.h"
 #include "ui/base/theme_provider.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/gfx/image/image.h"
-#include "ui/views/controls/button/text_button.h"
+#include "ui/views/controls/button/label_button.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/widget/root_view.h"
 #include "ui/views/widget/widget.h"
 
 // TODO(paulg): These may need to be adjusted when download progress
 //              animation is added, and also possibly to take into account
 //              different screen resolutions.
 static const int kTextWidth = 140;            // Pixels
 static const int kDangerousTextWidth = 200;   // Pixels
 static const int kHorizontalTextPadding = 2;  // Pixels
@@ skipping 985 matching lines @@
   TooltipTextChanged();
 }
 
 void DownloadItemView::ShowWarningDialog() {
   DCHECK(mode_ != DANGEROUS_MODE && mode_ != MALICIOUS_MODE);
   mode_ = ((model_.IsMalicious()) ? MALICIOUS_MODE : DANGEROUS_MODE);
 
   body_state_ = NORMAL;
   drop_down_state_ = NORMAL;
   if (mode_ == DANGEROUS_MODE) {
-    save_button_ = new views::NativeTextButton(
+    save_button_ = new views::LabelButton(
         this, model_.GetWarningConfirmButtonText());
-    save_button_->set_ignore_minimum_size(true);
+    save_button_->SetStyle(views::Button::STYLE_NATIVE_TEXTBUTTON);
+    save_button_->set_min_size(gfx::Size());

[msw, 2013/04/04 02:31:03]

Interestingly, TextButton::set_ignore_minimum_size would set a local member that
was never used :p Remove the set_min_size call altogether, since it's initialize
to an empty size anyway.

[tfarina, 2013/04/04 22:36:52]

Done.

     AddChildView(save_button_);
   }
-  discard_button_ = new views::NativeTextButton(
+  discard_button_ = new views::LabelButton(
       this, l10n_util::GetStringUTF16(IDS_DISCARD_DOWNLOAD));
-  discard_button_->set_ignore_minimum_size(true);
+  discard_button_->SetStyle(views::Button::STYLE_NATIVE_TEXTBUTTON);
+  discard_button_->set_min_size(gfx::Size());

[msw, 2013/04/04 02:31:03]

Remove set_min_size as per my earlier comment.

[tfarina, 2013/04/04 22:36:52]

Done.

   AddChildView(discard_button_);
 
   ui::ResourceBundle& rb = ui::ResourceBundle::GetSharedInstance();
   // The dangerous download label text and icon are different under
   // different cases.
   if (mode_ == MALICIOUS_MODE) {
     warning_icon_ = rb.GetImageSkiaNamed(IDR_SAFEBROWSING_WARNING);
   } else {
     // The download file has dangerous file type (e.g.: an executable).
     warning_icon_ = rb.GetImageSkiaNamed(IDR_WARNING);
@@ skipping 151 matching lines @@
 void DownloadItemView::AnimateStateTransition(State from, State to,
                                               ui::SlideAnimation* animation) {
   if (from == NORMAL && to == HOT) {
     animation->Show();
   } else if (from == HOT && to == NORMAL) {
     animation->Hide();
   } else if (from != to) {
     animation->Reset((to == HOT) ? 1.0 : 0.0);
   }
 }