Put some plug-ins behind an infobar, where they have:...

chrome/browser/tab_contents/tab_contents.cc

 // Copyright (c) 2011 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/tab_contents/tab_contents.h"
 
 #include <cmath>
 
 #include "base/auto_reset.h"
 #include "base/metrics/histogram.h"
@@ skipping 230 matching lines @@
   params->current_history_list_length = controller.entry_count();
   params->url = entry.url();
   params->referrer = entry.referrer();
   params->transition = entry.transition_type();
   params->state = entry.content_state();
   params->navigation_type =
       GetNavigationType(controller.profile(), entry, reload_type);
   params->request_time = base::Time::Now();
 }
 
+// PluginInfoBar --------------------------------------------------------------
 
-// OutdatedPluginInfoBar ------------------------------------------------------
-
-class OutdatedPluginInfoBar : public ConfirmInfoBarDelegate {
+class PluginInfoBar : public ConfirmInfoBarDelegate {
  public:
-  OutdatedPluginInfoBar(TabContents* tab_contents,
-                        const string16& name,
-                        const GURL& update_url);
-
- private:
-  virtual ~OutdatedPluginInfoBar();
+  PluginInfoBar(TabContents* tab_contents,
+                const string16& name,
+                const GURL& update_url);
 
   // ConfirmInfoBarDelegate:
   virtual void InfoBarClosed();
   virtual SkBitmap* GetIcon() const;
-  virtual string16 GetMessageText() const;
-  virtual int GetButtons() const;
+  virtual string16 GetMessageText() const = 0;
+  virtual int GetButtons() const = 0;
   virtual string16 GetButtonLabel(InfoBarButton button) const;
   virtual bool Accept();
   virtual bool Cancel();
   virtual string16 GetLinkText();
   virtual bool LinkClicked(WindowOpenDisposition disposition);
 
+ protected:
+  virtual ~PluginInfoBar();
+
+  string16 name_;
+
+ private:
+  virtual void RecordMetric(const std::string& action) const = 0;
+
   TabContents* tab_contents_;
-  string16 name_;
   GURL update_url_;
+
+  DISALLOW_COPY_AND_ASSIGN(PluginInfoBar);
+};
+
+PluginInfoBar::PluginInfoBar(TabContents* tab_contents,
+                             const string16& name,
+                             const GURL& update_url)
+    : ConfirmInfoBarDelegate(tab_contents),
+      name_(name),
+      tab_contents_(tab_contents),
+      update_url_(update_url) {
+}
+
+PluginInfoBar::~PluginInfoBar() {
+}
+
+void PluginInfoBar::InfoBarClosed() {
+  RecordMetric("Closed");
+  delete this;
+}
+
+SkBitmap* PluginInfoBar::GetIcon() const {
+  return ResourceBundle::GetSharedInstance().GetBitmapNamed(
+      IDR_INFOBAR_PLUGIN_INSTALL);
+}
+
+string16 PluginInfoBar::GetButtonLabel(InfoBarButton button) const {
+  return l10n_util::GetStringUTF16((button == BUTTON_OK) ?
+      IDS_PLUGIN_UPDATE : IDS_PLUGIN_ENABLE_TEMPORARILY);
+}
+
+bool PluginInfoBar::Accept() {
+  RecordMetric("Update");
+  tab_contents_->OpenURL(update_url_, GURL(), NEW_FOREGROUND_TAB,
+                         PageTransition::LINK);
+  return false;
+}
+
+bool PluginInfoBar::Cancel() {
+  RecordMetric("AllowThisTime");
+  tab_contents_->render_view_host()->LoadBlockedPlugins();
+  return false;
+}
+
+string16 PluginInfoBar::GetLinkText() {
+  return l10n_util::GetStringUTF16(IDS_LEARN_MORE);
+}
+
+bool PluginInfoBar::LinkClicked(WindowOpenDisposition disposition) {
+  RecordMetric("LearnMore");
+  // TODO(bauerb): Navigate to a help page explaining why we disabled
+  // or blocked the plugin, once we have one.
+  return false;
+}
+
+
+// BlockedPluginInfoBar -------------------------------------------------------
+
+class BlockedPluginInfoBar : public PluginInfoBar {
+ public:
+  BlockedPluginInfoBar(TabContents* tab_contents,
+                       const string16& name);
+
+  // ConfirmInfoBarDelegate:
+  virtual string16 GetMessageText() const;
+  virtual int GetButtons() const;
+
+ protected:
+  virtual ~BlockedPluginInfoBar();
+
+ private:
+  // PluginInfoBar:
+  virtual void RecordMetric(const std::string& action) const;
+
+  DISALLOW_COPY_AND_ASSIGN(BlockedPluginInfoBar);
+};
+
+BlockedPluginInfoBar::BlockedPluginInfoBar(TabContents* tab_contents,
+                                           const string16& name)
+    : PluginInfoBar(tab_contents, name, GURL()) {
+  tab_contents->AddInfoBar(this);
+  RecordMetric("Shown");
+}
+
+BlockedPluginInfoBar::~BlockedPluginInfoBar() {
+}
+
+string16 BlockedPluginInfoBar::GetMessageText() const {
+  return l10n_util::GetStringFUTF16(IDS_PLUGIN_NOT_AUTHORIZED, name_);
+}
+
+int BlockedPluginInfoBar::GetButtons() const {
+  return BUTTON_CANCEL;
+}
+
+void BlockedPluginInfoBar::RecordMetric(const std::string& action) const {
+  std::string uma("BlockedPluginInfobar");
+  uma += UTF16ToASCII(name_);
+  uma += ".";
+  uma += action;
+  UserMetrics::RecordAction(UserMetricsAction(uma.c_str()));

[Bernhard Bauer, 2011/01/24 12:56:33]

The problem with this is if you don't use a string literal for the
UserMetricsAction, the script won't pick it up.

Also, it's probably going to be a bit harder to find out aggregated numbers for
the infobar (how often is it shown in total, etc).

+}
+
+// OutdatedPluginInfoBar ------------------------------------------------------
+
+class OutdatedPluginInfoBar : public PluginInfoBar {
+ public:
+  OutdatedPluginInfoBar(TabContents* tab_contents,
+                        const string16& name,
+                        const GURL& update_url);
+
+  // ConfirmInfoBarDelegate:
+  virtual string16 GetMessageText() const;
+  virtual int GetButtons() const;
+
+ protected:
+  virtual ~OutdatedPluginInfoBar();
+
+ private:
+  // PluginInfoBar:
+  virtual void RecordMetric(const std::string& action) const;
+
+  DISALLOW_COPY_AND_ASSIGN(OutdatedPluginInfoBar);
 };
 
 OutdatedPluginInfoBar::OutdatedPluginInfoBar(TabContents* tab_contents,
                                              const string16& name,
                                              const GURL& update_url)
-    : ConfirmInfoBarDelegate(tab_contents),
-      tab_contents_(tab_contents),
-      name_(name),
-      update_url_(update_url) {
-  UserMetrics::RecordAction(UserMetricsAction("OutdatedPluginInfobar.Shown"));
+    : PluginInfoBar(tab_contents, name, update_url) {
   tab_contents->AddInfoBar(this);
+  RecordMetric("Shown");
 }
 
 OutdatedPluginInfoBar::~OutdatedPluginInfoBar() {
 }
 
-void OutdatedPluginInfoBar::InfoBarClosed() {
-  UserMetrics::RecordAction(UserMetricsAction("OutdatedPluginInfobar.Closed"));
-  delete this;
-}
-
-SkBitmap* OutdatedPluginInfoBar::GetIcon() const {
-  return ResourceBundle::GetSharedInstance().GetBitmapNamed(
-      IDR_INFOBAR_PLUGIN_INSTALL);
-}
-
 string16 OutdatedPluginInfoBar::GetMessageText() const {
   return l10n_util::GetStringFUTF16(IDS_PLUGIN_OUTDATED_PROMPT, name_);
 }
 
 int OutdatedPluginInfoBar::GetButtons() const {
   return BUTTON_OK | BUTTON_CANCEL;
 }
 
-string16 OutdatedPluginInfoBar::GetButtonLabel(InfoBarButton button) const {
-  return l10n_util::GetStringUTF16((button == BUTTON_OK) ?
-      IDS_PLUGIN_UPDATE : IDS_PLUGIN_ENABLE_TEMPORARILY);
-}
-
-bool OutdatedPluginInfoBar::Accept() {
-  UserMetrics::RecordAction(UserMetricsAction("OutdatedPluginInfobar.Update"));
-  tab_contents_->OpenURL(update_url_, GURL(), NEW_FOREGROUND_TAB,
-                         PageTransition::LINK);
-  return false;
-}
-
-bool OutdatedPluginInfoBar::Cancel() {
-  UserMetrics::RecordAction(
-      UserMetricsAction("OutdatedPluginInfobar.AllowThisTime"));
-  tab_contents_->render_view_host()->LoadBlockedPlugins();
-  return false;
-}
-
-string16 OutdatedPluginInfoBar::GetLinkText() {
-  return l10n_util::GetStringUTF16(IDS_LEARN_MORE);
-}
-
-bool OutdatedPluginInfoBar::LinkClicked(WindowOpenDisposition disposition) {
-  UserMetrics::RecordAction(
-      UserMetricsAction("OutdatedPluginInfobar.LearnMore"));
-  // TODO(bauerb): Navigate to a help page explaining why we disabled
-  // the plugin, once we have one.
-  return false;
+void OutdatedPluginInfoBar::RecordMetric(const std::string& action) const {
+  std::string uma("OutdatedPluginInfobar.");
+  uma += action;
+  UserMetrics::RecordAction(UserMetricsAction(uma.c_str()));
 }
 
 }  // namespace
 
 
 // TabContents ----------------------------------------------------------------
 
 // static
 int TabContents::find_request_id_counter_ = -1;
 
@@ skipping 1993 matching lines @@
     delegate()->OnDidGetApplicationInfo(this, page_id);
 }
 
 void TabContents::OnInstallApplication(const WebApplicationInfo& info) {
   if (delegate())
     delegate()->OnInstallApplication(this, info);
 }
 
 void TabContents::OnBlockedOutdatedPlugin(const string16& name,
                                           const GURL& update_url) {
-  new OutdatedPluginInfoBar(this, name, update_url);
+  if (!update_url.is_empty())
+    new OutdatedPluginInfoBar(this, name, update_url);
+  else
+    new BlockedPluginInfoBar(this, name);
 }
 
 void TabContents::OnPageContents(const GURL& url,
                                  int renderer_process_id,
                                  int32 page_id,
                                  const string16& contents,
                                  const std::string& language,
                                  bool page_translatable) {
   // Don't index any https pages. People generally don't want their bank
   // accounts, etc. indexed on their computer, especially since some of these
@@ skipping 1017 matching lines @@
   if (pm != NULL) {
     if (pm->MaybeUsePreloadedPage(this, url)) {
       // TODO(tburkard): If the preloaded page has not finished preloading
       // yet, we should not do this.
       DidStopLoading();
       return true;
     }
   }
   return false;
 }