New location security strings and animation.

chrome/browser/ui/views/location_bar/location_icon_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/location_icon_view.h"
 
+#include <iostream>
+
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ssl/chrome_security_state_model_client.h"
 #include "chrome/browser/ui/view_ids.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 #include "chrome/browser/ui/views/website_settings/website_settings_popup_view.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/omnibox/browser/omnibox_edit_model.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/web_contents.h"
@@ skipping 10 matching lines @@
 using content::WebContents;
 
 LocationIconView::LocationIconView(const gfx::FontList& font_list,
                                    SkColor parent_background_color,
                                    LocationBarView* location_bar)
     : IconLabelBubbleView(IDR_OMNIBOX_HTTPS_INVALID,
                           font_list,
                           parent_background_color,
                           true),
       suppress_mouse_released_action_(false),
-      location_bar_(location_bar) {
+      location_bar_(location_bar),
+      animation_(this),
+      animation_already_started_(false) {
   set_id(VIEW_ID_LOCATION_ICON);
 
 #if defined(OS_MACOSX)
   SetFocusBehavior(FocusBehavior::ACCESSIBLE_ONLY);
 #else
   SetFocusBehavior(FocusBehavior::ALWAYS);
 #endif
 
   SetBackground(false);
+  std::cout << "constructed " << (void*)this << "\n";
 }
 
 LocationIconView::~LocationIconView() {
+  std::cout << "destructed " << (void*)this << "\n";
 }
 
 bool LocationIconView::OnMousePressed(const ui::MouseEvent& event) {
+  std::cout << "pressed " << (void*)this << "\n";
   if (event.IsOnlyMiddleMouseButton() &&
       ui::Clipboard::IsSupportedClipboardType(ui::CLIPBOARD_TYPE_SELECTION)) {
     base::string16 text;
     ui::Clipboard::GetForCurrentThread()->ReadText(
         ui::CLIPBOARD_TYPE_SELECTION, &text);
     text = OmniboxView::SanitizeTextForPaste(text);
     OmniboxEditModel* model = location_bar_->GetOmniboxView()->model();
     if (model->CanPasteAndGo(text))
       model->PasteAndGo(text);
   }
@@ skipping 96 matching lines @@
   return size;
 }
 
 void LocationIconView::SetBackground(bool should_show_ev) {
   static const int kEvBackgroundImages[] = IMAGE_GRID(IDR_OMNIBOX_EV_BUBBLE);
   if (should_show_ev)
     SetBackgroundImageGrid(kEvBackgroundImages);
   else
     UnsetBackgroundImageGrid();
 }
+
+void LocationIconView::StartAnimation() {
+  if (!animation_already_started_) {
+    std::cout << "width is " << label()->GetPreferredSize().width() << "\n";
+    animation_.SetSlideDuration(label()->GetPreferredSize().width() * 10);
+    // They don't tell you but, to get it slide "down", you must do both:

[Peter Kasting, 2016/07/19 21:17:21]

Who doesn't tell you?  If you're trying to animate from 1 to 0, then it seems
pretty clear you have to ask to start at 1 and then animate to 0, since when
you're constructed you default to state 0 and when an animation stops it leaves
the state wherever it is.

If you want to simply run the animation and have that mean "make the width
shrink", you can always modify the code that gets the current value to return
something that will shrink as the animation goes from 0 to 1.

[Kevin Bailey, 2016/08/12 18:49:15]

slide_animation.h says:

  // Begin a hiding animation or reverse a showing animation in progress.
  virtual void Hide();

I think the "Begin" part is a little misleading. If you don't call 'Reset(1)',
then 'Hide()' simply reverses the direction, so maybe that's what it should have
been called.

Backing up a bit, I would have expected the methods to be named more like,
'Grow()' and 'Shrink()'. 'Show()' and 'Hide()' are too similar (for my taste) to
the well-known methods to display a window after creation.

I know I'm being fairly nitpicky here, but when method names tend to evoke the
wrong understanding, it doesn't hurt when there's a comment to clear things up.

+    animation_.Reset(1);
+    animation_.Hide();
+    animation_already_started_ = true;
+  }
+}
+
+void LocationIconView::AnimationProgressed(const gfx::Animation*) {
+  std::cout << "progressed " << animation_.GetCurrentValue() << "\n";
+  // Since things are moving (sideways), we need a re-layout and paint.
+  location_bar_->Layout();
+  location_bar_->SchedulePaint();
+}
+
+/*
+int LocationIconView::GetLabelPreferredWidth() const {
+  gfx::Rect size(label()->GetPreferredSize());
+  return size.width() * animation_.GetCurrentValue();
+}
+*/
+double LocationIconView::WidthMultiplier() const {
+  return animation_.GetCurrentValue();
+}
+
+bool LocationIconView::IsShrinking() const {
+  return true;
+}