Refactor out duplicate code from GTK/Views extension installed bubble.

chrome/browser/ui/gtk/extensions/extension_installed_bubble_gtk.cc

Index: chrome/browser/ui/gtk/extensions/extension_installed_bubble_gtk.cc
diff --git a/chrome/browser/ui/gtk/extensions/extension_installed_bubble_gtk.cc b/chrome/browser/ui/gtk/extensions/extension_installed_bubble_gtk.cc
index b1531060bea3f2b8f66d3477aefa69cd1ca9e752..feeb17234c316b915d2aba573cf4a65a686a8736 100644
--- a/chrome/browser/ui/gtk/extensions/extension_installed_bubble_gtk.cc
+++ b/chrome/browser/ui/gtk/extensions/extension_installed_bubble_gtk.cc
@@ -6,10 +6,7 @@
 
 #include <string>
 
-#include "base/bind.h"
-#include "base/bind_helpers.h"
 #include "base/i18n/rtl.h"
-#include "base/message_loop/message_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/api/commands/command_service.h"
@@ -24,7 +21,6 @@
 #include "chrome/browser/ui/gtk/gtk_util.h"
 #include "chrome/browser/ui/gtk/location_bar_view_gtk.h"
 #include "chrome/browser/ui/singleton_tabs.h"
-#include "chrome/common/extensions/api/extension_action/action_info.h"
 #include "chrome/common/extensions/api/omnibox/omnibox_handler.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/url_constants.h"
@@ -49,12 +45,6 @@ const int kIconSize = 43;
 const int kTextColumnVerticalSpacing = 7;
 const int kTextColumnWidth = 350;
 
-// When showing the bubble for a new browser action, we may have to wait for
-// the toolbar to finish animating to know where the item's final position
-// will be.
-const int kAnimationWaitRetries = 10;
-const int kAnimationWaitMS = 50;
-
 }  // namespace
 
 namespace chrome {
@@ -75,68 +65,11 @@ void ExtensionInstalledBubbleGtk::Show(const Extension* extension,
 
 ExtensionInstalledBubbleGtk::ExtensionInstalledBubbleGtk(
     const Extension* extension, Browser *browser, const SkBitmap& icon)
-    : extension_(extension),
-      browser_(browser),
-      icon_(icon),
-      animation_wait_retries_(kAnimationWaitRetries),
-      bubble_(NULL),
-      weak_factory_(this) {
-  if (!extensions::OmniboxInfo::GetKeyword(extension_).empty())
-    type_ = OMNIBOX_KEYWORD;
-  else if (extensions::ActionInfo::GetBrowserActionInfo(extension_))
-    type_ = BROWSER_ACTION;
-  else if (extensions::ActionInfo::GetPageActionInfo(extension) &&
-           extensions::ActionInfo::IsVerboseInstallMessage(extension))
-    type_ = PAGE_ACTION;
-  else
-    type_ = GENERIC;
-
-  // |extension| has been initialized but not loaded at this point. We need
-  // to wait on showing the Bubble until not only the EXTENSION_LOADED gets
-  // fired, but all of the EXTENSION_LOADED Observers have run. Only then can we
-  // be sure that a browser action or page action has had views created which we
-  // can inspect for the purpose of pointing to them.
-  registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_LOADED,
-      content::Source<Profile>(browser->profile()));
-  registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_UNLOADED,
-      content::Source<Profile>(browser->profile()));
-  registrar_.Add(this, chrome::NOTIFICATION_BROWSER_CLOSING,
-      content::Source<Browser>(browser));
+    : ExtensionInstalledBubble(extension, browser, icon) {
 }
 
 ExtensionInstalledBubbleGtk::~ExtensionInstalledBubbleGtk() {}
 
-void ExtensionInstalledBubbleGtk::Observe(
-    int type,
-    const content::NotificationSource& source,
-    const content::NotificationDetails& details) {
-  if (type == chrome::NOTIFICATION_EXTENSION_LOADED) {
-    const Extension* extension =
-        content::Details<const Extension>(details).ptr();
-    if (extension == extension_) {
-      // PostTask to ourself to allow all EXTENSION_LOADED Observers to run.
-      base::MessageLoopForUI::current()->PostTask(
-          FROM_HERE,
-          base::Bind(&ExtensionInstalledBubbleGtk::ShowInternal,
-                     weak_factory_.GetWeakPtr()));
-    }
-  } else if (type == chrome::NOTIFICATION_EXTENSION_UNLOADED) {
-    const Extension* extension =
-        content::Details<extensions::UnloadedExtensionInfo>(details)->extension;
-    if (extension == extension_) {
-      // Extension is going away, make sure ShowInternal won't be called.
-      weak_factory_.InvalidateWeakPtrs();
-      extension_ = NULL;
-    }
-  } else if (type == chrome::NOTIFICATION_BROWSER_CLOSING) {
-    // The browser closed before the bubble could be created.
-    if (!bubble_)
-      delete this;
-  } else {
-    NOTREACHED() << L"Received unexpected notification";
-  }
-}
-
 void ExtensionInstalledBubbleGtk::OnDestroy(GtkWidget* widget) {
   bubble_ = NULL;
   delete this;
@@ -145,24 +78,19 @@ void ExtensionInstalledBubbleGtk::OnDestroy(GtkWidget* widget) {
 void ExtensionInstalledBubbleGtk::ShowInternal() {
   BrowserWindowGtk* browser_window =
       BrowserWindowGtk::GetBrowserWindowForNativeWindow(
-          browser_->window()->GetNativeWindow());
+          browser()->window()->GetNativeWindow());
 
   GtkWidget* reference_widget = NULL;
 
-  if (type_ == BROWSER_ACTION) {
+  if (type() == BROWSER_ACTION) {
     BrowserActionsToolbarGtk* toolbar =
         browser_window->GetToolbar()->GetBrowserActionsToolbar();
-
-    if (toolbar->animating() && animation_wait_retries_-- > 0) {
-      base::MessageLoopForUI::current()->PostDelayedTask(
-          FROM_HERE,
-          base::Bind(&ExtensionInstalledBubbleGtk::ShowInternal,
-                     weak_factory_.GetWeakPtr()),
-          base::TimeDelta::FromMilliseconds(kAnimationWaitMS));
+    if (toolbar->animating()) {
+      MaybeShowLater();
       return;
     }
 
-    reference_widget = toolbar->GetBrowserActionWidget(extension_);
+    reference_widget = toolbar->GetBrowserActionWidget(extension());
     // glib delays recalculating layout, but we need reference_widget to know
     // its coordinates, so we force a check_resize here.
     gtk_container_check_resize(GTK_CONTAINER(
@@ -174,12 +102,12 @@ void ExtensionInstalledBubbleGtk::ShowInternal() {
       reference_widget = gtk_widget_get_visible(toolbar->chevron()) ?
           toolbar->chevron() : NULL;
     }
-  } else if (type_ == PAGE_ACTION) {
+  } else if (type() == PAGE_ACTION) {
     LocationBarViewGtk* location_bar_view =
         browser_window->GetToolbar()->GetLocationBarView();
     ExtensionAction* page_action =
-        ExtensionActionManager::Get(browser_->profile())->
-        GetPageAction(*extension_);
+        ExtensionActionManager::Get(browser()->profile())->
+        GetPageAction(*extension());
     location_bar_view->SetPreviewEnabledPageAction(page_action,
                                                    true);  // preview_enabled
     reference_widget = location_bar_view->GetPageActionWidget(page_action);
@@ -188,7 +116,7 @@ void ExtensionInstalledBubbleGtk::ShowInternal() {
     gtk_container_check_resize(GTK_CONTAINER(
         browser_window->GetToolbar()->widget()));
     DCHECK(reference_widget);
-  } else if (type_ == OMNIBOX_KEYWORD) {
+  } else if (type() == OMNIBOX_KEYWORD) {
     LocationBarViewGtk* location_bar_view =
         browser_window->GetToolbar()->GetLocationBarView();
     reference_widget = location_bar_view->location_entry_widget();
@@ -200,17 +128,17 @@ void ExtensionInstalledBubbleGtk::ShowInternal() {
     reference_widget = browser_window->GetToolbar()->GetAppMenuButton();
 
   GtkThemeService* theme_provider = GtkThemeService::GetFrom(
-      browser_->profile());
+      browser()->profile());
 
   // Setup the BubbleGtk content.
   GtkWidget* bubble_content = gtk_hbox_new(FALSE, kHorizontalColumnSpacing);
   gtk_container_set_border_width(GTK_CONTAINER(bubble_content),
                                  ui::kContentAreaBorder);
 
-  if (!icon_.isNull()) {
+  if (!icon().isNull()) {
     // Scale icon down to 43x43, but allow smaller icons (don't scale up).
-    GdkPixbuf* pixbuf = gfx::GdkPixbufFromSkBitmap(icon_);
-    gfx::Size size(icon_.width(), icon_.height());
+    GdkPixbuf* pixbuf = gfx::GdkPixbufFromSkBitmap(icon());
+    gfx::Size size(icon().width(), icon().height());
     if (size.width() > kIconSize || size.height() > kIconSize) {
       if (size.width() > size.height()) {
         size.set_height(size.height() * kIconSize / size.width());
@@ -243,7 +171,7 @@ void ExtensionInstalledBubbleGtk::ShowInternal() {
 
   // Heading label.
   GtkWidget* heading_label = gtk_label_new(NULL);
-  string16 extension_name = UTF8ToUTF16(extension_->name());
+  string16 extension_name = UTF8ToUTF16(extension()->name());
   base::i18n::AdjustStringForLocaleDirection(&extension_name);
   std::string heading_text = l10n_util::GetStringFUTF8(
       IDS_EXTENSION_INSTALLED_HEADING, extension_name);
@@ -258,13 +186,13 @@ void ExtensionInstalledBubbleGtk::ShowInternal() {
   bool has_keybinding = false;
 
   // Browser action label.
-  if (type_ == BROWSER_ACTION) {
+  if (type() == BROWSER_ACTION) {
     extensions::CommandService* command_service =
-        extensions::CommandService::Get(browser_->profile());
+        extensions::CommandService::Get(browser()->profile());
     extensions::Command browser_action_command;
     GtkWidget* info_label;
     if (!command_service->GetBrowserActionCommand(
-            extension_->id(),
+            extension()->id(),
             extensions::CommandService::ACTIVE_ONLY,
             &browser_action_command,
             NULL)) {
@@ -281,13 +209,13 @@ void ExtensionInstalledBubbleGtk::ShowInternal() {
   }
 
   // Page action label.
-  if (type_ == PAGE_ACTION) {
+  if (type() == PAGE_ACTION) {
     extensions::CommandService* command_service =
-        extensions::CommandService::Get(browser_->profile());
+        extensions::CommandService::Get(browser()->profile());
     extensions::Command page_action_command;
     GtkWidget* info_label;
     if (!command_service->GetPageActionCommand(
-            extension_->id(),
+            extension()->id(),
             extensions::CommandService::ACTIVE_ONLY,
             &page_action_command,
             NULL)) {
@@ -304,10 +232,10 @@ void ExtensionInstalledBubbleGtk::ShowInternal() {
   }
 
   // Omnibox keyword label.
-  if (type_ == OMNIBOX_KEYWORD) {
+  if (type() == OMNIBOX_KEYWORD) {
     GtkWidget* info_label = gtk_label_new(l10n_util::GetStringFUTF8(
         IDS_EXTENSION_INSTALLED_OMNIBOX_KEYWORD_INFO,
-        UTF8ToUTF16(extensions::OmniboxInfo::GetKeyword(extension_))).c_str());
+        UTF8ToUTF16(extensions::OmniboxInfo::GetKeyword(extension()))).c_str());
     gtk_util::SetLabelWidth(info_label, kTextColumnWidth);
     gtk_box_pack_start(GTK_BOX(text_column), info_label, FALSE, FALSE, 0);
   }
@@ -341,7 +269,7 @@ void ExtensionInstalledBubbleGtk::ShowInternal() {
   BubbleGtk::FrameStyle frame_style = BubbleGtk::ANCHOR_TOP_RIGHT;
 
   gfx::Rect bounds = gtk_util::WidgetBounds(reference_widget);
-  if (type_ == OMNIBOX_KEYWORD) {
+  if (type() == OMNIBOX_KEYWORD) {
     // Reverse the arrow for omnibox keywords, since the bubble will be on the
     // other side of the window. We also clear the width to avoid centering
     // the popup on the URL bar.
@@ -381,22 +309,22 @@ void ExtensionInstalledBubbleGtk::OnLinkClicked(GtkWidget* widget) {
   configure_url += chrome::kExtensionConfigureCommandsSubPage;
   chrome::NavigateParams params(
       chrome::GetSingletonTabNavigateParams(
-      browser_, GURL(configure_url.c_str())));
+          browser(), GURL(configure_url.c_str())));
   chrome::Navigate(&params);
 }
 
 void ExtensionInstalledBubbleGtk::BubbleClosing(BubbleGtk* bubble,
                                                 bool closed_by_escape) {
-  if (extension_ && type_ == PAGE_ACTION) {
+  if (extension() && type() == PAGE_ACTION) {
     // Turn the page action preview off.
     BrowserWindowGtk* browser_window =
           BrowserWindowGtk::GetBrowserWindowForNativeWindow(
-              browser_->window()->GetNativeWindow());
+              browser()->window()->GetNativeWindow());
     LocationBarViewGtk* location_bar_view =
         browser_window->GetToolbar()->GetLocationBarView();
     location_bar_view->SetPreviewEnabledPageAction(
-        ExtensionActionManager::Get(browser_->profile())->
-        GetPageAction(*extension_),
+        ExtensionActionManager::Get(browser()->profile())->
+        GetPageAction(*extension()),
         false);  // preview_enabled
   }
 }