Invokes SchedulePaint when animation ended to repaint the background.

ui/app_list/views/app_list_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 "ui/app_list/views/app_list_view.h"
 
 #include "base/command_line.h"
 #include "base/strings/string_util.h"
 #include "base/win/windows_version.h"
 #include "ui/app_list/app_list_constants.h"
@@ skipping 69 matching lines @@
   return false;
 #endif
   return true;
 }
 
 }  // namespace
 
 // An animation observer to hide the view at the end of the animation.
 class HideViewAnimationObserver : public ui::ImplicitAnimationObserver {
  public:
-  HideViewAnimationObserver() : target_(NULL) {
+  HideViewAnimationObserver(views::View* parent)
+      : parent_(parent),
+        target_(NULL) {
   }
 
   virtual ~HideViewAnimationObserver() {
     if (target_)
       StopObservingImplicitAnimations();
   }
 
   void SetTarget(views::View* target) {
     if (!target_)
       StopObservingImplicitAnimations();
     target_ = target;
   }
 
  private:
   // Overridden from ui::ImplicitAnimationObserver:
   virtual void OnImplicitAnimationsCompleted() OVERRIDE {
     if (target_) {
       target_->SetVisible(false);
       target_ = NULL;
+
+      // Should update the background by invoking SchedulePaint().
+      parent_->SchedulePaint();

[xiyuan, 2014/01/15 01:22:05]

Think we should use app_list_view_->GetBubbleFrameView()->SchedulePaint()
because the AppListBubbleBackground is part of the frame view instead of
AppListView.

[Jun Mukai, 2014/01/15 02:03:44]

Done.

     }
   }
 
+  views::View* parent_;
   views::View* target_;
 
   DISALLOW_COPY_AND_ASSIGN(HideViewAnimationObserver);
 };
 
 ////////////////////////////////////////////////////////////////////////////////
 // AppListView:
 
 AppListView::AppListView(AppListViewDelegate* delegate)
     : delegate_(delegate),
       app_list_main_view_(NULL),
       signin_view_(NULL),
       speech_view_(NULL),
-      animation_observer_(new HideViewAnimationObserver()) {
+      animation_observer_(new HideViewAnimationObserver(this)) {
   CHECK(delegate);
 
   delegate_->AddObserver(this);
   delegate_->GetSpeechUI()->AddObserver(this);
 }
 
 AppListView::~AppListView() {
   delegate_->GetSpeechUI()->RemoveObserver(this);
   delegate_->RemoveObserver(this);
   animation_observer_.reset();
@@ skipping 333 matching lines @@
       speech_view_->layer()->SetTransform(speech_transform);
   }
 
   if (recognizing)
     speech_view_->SetVisible(true);
   else
     app_list_main_view_->SetVisible(true);
 #else
   speech_view_->SetVisible(recognizing);
   app_list_main_view_->SetVisible(!recognizing);
-#endif
 
   // Needs to schedule paint of AppListView itself, to repaint the background.
   SchedulePaint();

[xiyuan, 2014/01/15 01:22:05]

Let's use GetBubbleFrameView()->SchedulePaint() instead.

[Jun Mukai, 2014/01/15 02:03:44]

Done.

+#endif
 }
 
 }  // namespace app_list