Fix up views web intents location bar decoration.

chrome/browser/ui/views/location_bar/web_intents_button_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/web_intents_button_view.h"
 
 #include "chrome/browser/ui/intents/web_intent_picker_controller.h"
 #include "chrome/browser/ui/tab_contents/tab_contents.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 #include "grit/generated_resources.h"
@@ skipping 16 matching lines @@
       !tab_contents->web_intent_picker_controller()->
           ShowLocationBarPickerTool()) {
     SetVisible(false);
   } else {
     SetVisible(true);
   }
 
   int animated_string_id = IDS_INTENT_PICKER_USE_ANOTHER_SERVICE;
   string16 animated_text = l10n_util::GetStringUTF16(animated_string_id);
   SetTooltipText(animated_text);
+  PreferredSizeChanged();
 
+  AlwaysDrawText();

[Peter Kasting, 2012/09/15 00:23:46]

Nit: If the order of these statements matters, write a comment noting that and
explaining why.

[Greg Billock, 2012/09/15 04:58:20]

It shouldn't, but belongs this way. I'll add a comment.


On 2012/09/15 00:23:46, Peter Kasting wrote:

   StartLabelAnimation(animated_text, kMoveTimeMs);
-  AlwaysDrawText();
 }
 
 void WebIntentsButtonView::OnClick(LocationBarView* parent) {
   TabContents* tab_contents = parent->GetTabContents();
   if (!tab_contents)
     return;
 
   tab_contents->web_intent_picker_controller()->LocationBarPickerToolClicked();
 }
 
 int WebIntentsButtonView::GetTextAnimationSize(double state, int text_size) {
   if (state < 1.0) {
     return static_cast<int>(text_size * state);
   }
 
   return text_size;
 }