Misc. cleanup

chrome/browser/ui/views/location_bar/content_setting_image_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/location_bar/content_setting_image_view.h"
 
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/themes/theme_properties.h"
 #include "chrome/browser/ui/content_settings/content_setting_bubble_model.h"
 #include "chrome/browser/ui/content_settings/content_setting_image_model.h"
@@ skipping 88 matching lines @@
   if (string_id && !label()->visible()) {
     ink_drop_delegate_->OnAction(views::InkDropState::HIDDEN);
     SetLabel(l10n_util::GetStringUTF16(string_id));
     label()->SetVisible(true);
     slide_animator_.Show();
   }
 
   content_setting_image_model_->SetAnimationHasRun(web_contents);
 }
 
-SkColor ContentSettingImageView::GetTextColor() const {
-  return GetNativeTheme()->GetSystemColor(
-      ui::NativeTheme::kColorId_TextfieldDefaultColor);
-}
-
-SkColor ContentSettingImageView::GetBorderColor() const {
-  return gfx::kGoogleYellow700;
-}
-
-bool ContentSettingImageView::ShouldShowBackground() const {
-  return (!IsShrinking() || label()->width() > 0) &&
-         (slide_animator_.is_animating() || pause_animation_);
-}
-
-double ContentSettingImageView::WidthMultiplier() const {
-  double state = pause_animation_ ? pause_animation_state_
-                                  : slide_animator_.GetCurrentValue();
-  // The fraction of the animation we'll spend animating the string into view,
-  // which is also the fraction we'll spend animating it closed; total
-  // animation (slide out, show, then slide in) is 1.0.
-  const double kOpenFraction =
-      static_cast<double>(kOpenTimeMS) / kAnimationDurationMS;
-  double size_fraction = 1.0;
-  if (state < kOpenFraction)
-    size_fraction = state / kOpenFraction;
-  if (state > (1.0 - kOpenFraction))
-    size_fraction = (1.0 - state) / kOpenFraction;
-  return size_fraction;
-}
-
-bool ContentSettingImageView::IsShrinking() const {
-  const double kOpenFraction =
-      static_cast<double>(kOpenTimeMS) / kAnimationDurationMS;
-  return (!pause_animation_ && slide_animator_.is_animating() &&
-          slide_animator_.GetCurrentValue() > (1.0 - kOpenFraction));
-}
-
-void ContentSettingImageView::AnimationEnded(const gfx::Animation* animation) {
-  slide_animator_.Reset();
-  if (!pause_animation_) {
-    label()->SetVisible(false);
-    parent_->Layout();
-    parent_->SchedulePaint();
-  }
-}
-
-void ContentSettingImageView::AnimationProgressed(
-    const gfx::Animation* animation) {
-  if (!pause_animation_) {
-    parent_->Layout();
-    parent_->SchedulePaint();
-  }
-}
-
-void ContentSettingImageView::AnimationCanceled(
-    const gfx::Animation* animation) {
-  AnimationEnded(animation);
-}
-
 const char* ContentSettingImageView::GetClassName() const {
   return "ContentSettingsImageView";
 }
 
 void ContentSettingImageView::OnBoundsChanged(
     const gfx::Rect& previous_bounds) {
   if (bubble_view_)
     bubble_view_->OnAnchorBoundsChanged();
 }
 
 bool ContentSettingImageView::OnMousePressed(const ui::MouseEvent& event) {
   // If the bubble is showing then don't reshow it when the mouse is released.
-  suppress_mouse_released_action_ = IsBubbleShowing();
+  suppress_mouse_released_action_ = bubble_view_ != nullptr;
   if (!suppress_mouse_released_action_ && !label()->visible())
     ink_drop_delegate_->OnAction(views::InkDropState::ACTION_PENDING);
 
   // We want to show the bubble on mouse release; that is the standard behavior
   // for buttons.
   return true;
 }
 
 void ContentSettingImageView::OnMouseReleased(const ui::MouseEvent& event) {
   // If this is the second click on this view then the bubble was showing on the
@@ skipping 27 matching lines @@
 }
 
 void ContentSettingImageView::OnNativeThemeChanged(
     const ui::NativeTheme* native_theme) {
   if (ui::MaterialDesignController::IsModeMaterial())
     UpdateImage();
 
   IconLabelBubbleView::OnNativeThemeChanged(native_theme);
 }
 
+SkColor ContentSettingImageView::GetTextColor() const {
+  return GetNativeTheme()->GetSystemColor(
+      ui::NativeTheme::kColorId_TextfieldDefaultColor);
+}
+
+SkColor ContentSettingImageView::GetBorderColor() const {
+  return gfx::kGoogleYellow700;
+}
+
+bool ContentSettingImageView::ShouldShowBackground() const {
+  return (!IsShrinking() || label()->width() > 0) &&
+         (slide_animator_.is_animating() || pause_animation_);
+}
+
+double ContentSettingImageView::WidthMultiplier() const {
+  double state = pause_animation_ ? pause_animation_state_
+                                  : slide_animator_.GetCurrentValue();
+  // The fraction of the animation we'll spend animating the string into view,
+  // which is also the fraction we'll spend animating it closed; total
+  // animation (slide out, show, then slide in) is 1.0.
+  const double kOpenFraction =
+      static_cast<double>(kOpenTimeMS) / kAnimationDurationMS;
+  double size_fraction = 1.0;
+  if (state < kOpenFraction)
+    size_fraction = state / kOpenFraction;
+  if (state > (1.0 - kOpenFraction))
+    size_fraction = (1.0 - state) / kOpenFraction;
+  return size_fraction;
+}
+
+bool ContentSettingImageView::IsShrinking() const {
+  const double kOpenFraction =
+      static_cast<double>(kOpenTimeMS) / kAnimationDurationMS;
+  return (!pause_animation_ && slide_animator_.is_animating() &&
+          slide_animator_.GetCurrentValue() > (1.0 - kOpenFraction));
+}
+
+void ContentSettingImageView::AnimationEnded(const gfx::Animation* animation) {
+  slide_animator_.Reset();
+  if (!pause_animation_) {
+    label()->SetVisible(false);
+    parent_->Layout();
+    parent_->SchedulePaint();
+  }
+}
+
+void ContentSettingImageView::AnimationProgressed(
+    const gfx::Animation* animation) {
+  if (!pause_animation_) {
+    parent_->Layout();
+    parent_->SchedulePaint();
+  }
+}
+
+void ContentSettingImageView::AnimationCanceled(
+    const gfx::Animation* animation) {
+  AnimationEnded(animation);
+}
+
 void ContentSettingImageView::OnWidgetDestroying(views::Widget* widget) {
   DCHECK(bubble_view_);
   DCHECK_EQ(bubble_view_->GetWidget(), widget);
   widget->RemoveObserver(this);
   bubble_view_ = nullptr;
 
   if (pause_animation_) {
     slide_animator_.Reset(pause_animation_state_);
     pause_animation_ = false;
     slide_animator_.Show();
@@ skipping 11 matching lines @@
   if (slide_animator_.is_animating()) {
     // If the user clicks while we're animating, the bubble arrow will be
     // pointing to the image, and if we allow the animation to keep running, the
     // image will move away from the arrow (or we'll have to move the bubble,
     // which is even worse). So we want to stop the animation.  We have two
     // choices: jump to the final post-animation state (no label visible), or
     // pause the animation where we are and continue running after the bubble
     // closes. The former looks more jerky, so we avoid it unless the animation
     // hasn't even fully exposed the image yet, in which case pausing with half
     // an image visible will look broken.
-    const int final_width = image()->GetPreferredSize().width() +
-                            GetBubbleOuterPadding(true) +
-                            GetBubbleOuterPadding(false);
-    if (!pause_animation_ && ShouldShowBackground() && width() > final_width) {
+    if (!pause_animation_ && ShouldShowBackground() &&
+        (width() > MinimumWidthForImageWithBackgroundShown())) {
       pause_animation_ = true;
       pause_animation_state_ = slide_animator_.GetCurrentValue();
     }
     slide_animator_.Reset();
   }
 
   content::WebContents* web_contents = parent_->GetWebContents();
   if (web_contents && !bubble_view_) {
     bubble_view_ = new ContentSettingBubbleContents(
                 content_setting_image_model_->CreateBubbleModel(
@@ skipping 13 matching lines @@
       bubble_view_->SetArrowPaintType(views::BubbleBorder::PAINT_TRANSPARENT);
     }
     bubble_widget->Show();
   }
 }
 
 void ContentSettingImageView::UpdateImage() {
   SetImage(content_setting_image_model_->GetIcon(GetTextColor()).AsImageSkia());
   image()->SetTooltipText(content_setting_image_model_->get_tooltip());
 }
-
-bool ContentSettingImageView::IsBubbleShowing() const {
-  return bubble_view_ != nullptr;
-}