Close ExtensionPopup on tab switches; add test.

chrome/browser/ui/views/extensions/extension_popup.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_POPUP_H_
 #define CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_POPUP_H_
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
+#include "chrome/browser/ui/tabs/tab_strip_model_observer.h"
 #include "chrome/browser/ui/views/extensions/extension_view_views.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "ui/views/bubble/bubble_delegate.h"
 #include "ui/views/focus/widget_focus_manager.h"
 #include "url/gurl.h"
 
 #if defined(USE_AURA)
 #include "ui/aura/client/activation_change_observer.h"
 #endif
 
 class Browser;
 namespace views {
 class Widget;
 }
 
 namespace content {
 class DevToolsAgentHost;
 }
 
 namespace extensions {
 class ExtensionViewHost;
 }
 
 class ExtensionPopup : public views::BubbleDelegateView,
 #if defined(USE_AURA)
                        public aura::client::ActivationChangeObserver,
 #endif
                        public ExtensionViewViews::Container,
-                       public content::NotificationObserver {
+                       public content::NotificationObserver,
+                       public TabStripModelObserver {
  public:
   enum ShowAction {
     SHOW,
     SHOW_AND_INSPECT
   };
 
   virtual ~ExtensionPopup();
 
   // Create and show a popup with |url| positioned adjacent to |anchor_view|.
   // |browser| is the browser to which the pop-up will be attached.  NULL is a
@@ skipping 27 matching lines @@
   virtual void OnWidgetDestroying(views::Widget* widget) OVERRIDE;
   virtual void OnWidgetActivationChanged(views::Widget* widget,
                                          bool active) OVERRIDE;
 
 #if defined(USE_AURA)
   // aura::client::ActivationChangeObserver overrides.
   virtual void OnWindowActivated(aura::Window* gained_active,
                                  aura::Window* lost_active) OVERRIDE;
 #endif
 
+  // TabStripModelObserver overrides.
+  virtual void ActiveTabChanged(content::WebContents* old_contents,
+                                content::WebContents* new_contents,
+                                int index,
+                                int reason) OVERRIDE;
+
   // The min/max height of popups.
   static const int kMinWidth;
   static const int kMinHeight;
   static const int kMaxWidth;
   static const int kMaxHeight;
 
  private:
   ExtensionPopup(extensions::ExtensionViewHost* host,
                  views::View* anchor_view,
                  views::BubbleBorder::Arrow arrow,
@@ skipping 12 matching lines @@
   bool inspect_with_devtools_;
 
   content::NotificationRegistrar registrar_;
 
   base::Callback<void(content::DevToolsAgentHost*, bool)> devtools_callback_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionPopup);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_EXTENSIONS_EXTENSION_POPUP_H_