If a user chooses to open a PDF with Reader, ask them if they want to do so a...

chrome/browser/pdf_unsupported_feature.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/pdf_unsupported_feature.h"
 
 #include "base/utf_string_conversions.h"
 #include "base/values.h"
 #include "base/version.h"
+#include "chrome/browser/metrics/user_metrics.h"
 #include "chrome/browser/plugin_service.h"
 #include "chrome/browser/renderer_host/render_process_host.h"
 #include "chrome/browser/renderer_host/render_view_host.h"
 #include "chrome/browser/tab_contents/infobar_delegate.h"
 #include "chrome/browser/tab_contents/interstitial_page.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/common/jstemplate_builder.h"
+#include "chrome/common/pepper_plugin_registry.h"
 #include "grit/browser_resources.h"
 #include "grit/generated_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "webkit/plugins/npapi/plugin_group.h"
 #include "webkit/plugins/npapi/plugin_list.h"
 #include "webkit/plugins/npapi/webplugininfo.h"
 
 using webkit::npapi::PluginGroup;
 using webkit::npapi::PluginList;
 using webkit::npapi::WebPluginInfo;
 
 // Only launch Adobe Reader X or later.
 static const uint16 kMinReaderVersionToUse = 10;
 
 namespace {
 
+// The info bar delegate used to ask the user if they want to use Adobe Reader
+// by default.
+class PDFEnableAdobeReaderConfirmInfoBarDelegate
+    : public ConfirmInfoBarDelegate {
+ public:
+  PDFEnableAdobeReaderConfirmInfoBarDelegate(
+      TabContents* tab_contents)
+      : ConfirmInfoBarDelegate(tab_contents) {
+    UserMetrics::RecordAction(UserMetricsAction(
+        "PDF_EnableReaderInfoBarShown"));

[Chris Evans, 2011/01/26 23:31:30]

Will this style of linebreak match the regex in extract_actions.py ? (I just
learned about this from Bernhard). Perhaps
RecordAction\nUserMetricsAction("blah")) is preferred?

[jam, 2011/01/26 23:45:57]

ah, good point. i  forgot to run the script to verify this. fixed.

+  }
+
+  // ConfirmInfoBarDelegate
+  virtual void InfoBarClosed() {

[Chris Evans, 2011/01/26 23:31:30]

Do you want to UMA the act of closing this bar?

[jam, 2011/01/26 23:45:57]

that's done in virtual bool Cancel() {

+    delete this;
+  }

[Chris Evans, 2011/01/26 23:31:30]

Style: newline here?

[jam, 2011/01/26 23:45:57]

Done.

+  virtual Type GetInfoBarType() {

[Chris Evans, 2011/01/26 23:31:30]

I think you're missing a const -- wrong overload?

[jam, 2011/01/26 23:45:57]

good catch, done

+    return PAGE_ACTION_TYPE;
+  }
+  virtual bool Accept() {
+    UserMetrics::RecordAction(UserMetricsAction(
+        "PDF_EnableReaderInfoBarOK"));
+    webkit::npapi::PluginList::Singleton()->EnableGroup(
+        false, ASCIIToUTF16(PepperPluginRegistry::kPDFPluginName));
+    webkit::npapi::PluginList::Singleton()->EnableGroup(
+        true, ASCIIToUTF16(webkit::npapi::PluginGroup::kAdobeReaderGroupName));
+    return true;
+  }
+  virtual bool Cancel() {
+    UserMetrics::RecordAction(UserMetricsAction(
+        "PDF_EnableReaderInfoBarCancel"));
+    return true;
+  }
+  virtual int GetButtons() const {
+    return BUTTON_OK | BUTTON_CANCEL;
+  }
+  virtual string16 GetButtonLabel(InfoBarButton button) const {
+    switch (button) {
+      case BUTTON_OK:
+        return l10n_util::GetStringUTF16(
+            IDS_CONFIRM_MESSAGEBOX_YES_BUTTON_LABEL);
+      case BUTTON_CANCEL:
+        return l10n_util::GetStringUTF16(
+            IDS_CONFIRM_MESSAGEBOX_NO_BUTTON_LABEL);
+      default:
+        // All buttons are labeled above.
+        NOTREACHED() << "Bad button id " << button;
+        return string16();
+    }
+  }
+  virtual string16 GetMessageText() const {
+    return l10n_util::GetStringUTF16(
+        IDS_PDF_INFOBAR_QUESTION_ALWAYS_USE_READER);

[Chris Evans, 2011/01/26 23:31:30]

This does not seem be to be defined in this CL?
We should also check the message; I don't want it to be too easy for a user
click to semi-permanently disable our faster and more secure built-in reader.

[jam, 2011/01/26 23:45:57]

it was checked in earlier.
we normally don't check this, because if the strings are out of sync with the
code, chrome is pretty hosed at this point.

+  }
+
+ private:
+  DISALLOW_IMPLICIT_CONSTRUCTORS(PDFEnableAdobeReaderConfirmInfoBarDelegate);
+};
+
 // Launch the url to get the latest Adbobe Reader installer.
 void OpenReaderUpdateURL(TabContents* tab) {
   tab->OpenURL(GURL(PluginGroup::kAdobeReaderUpdateURL), GURL(), CURRENT_TAB,
                PageTransition::LINK);
 }
 
 // Opens the PDF using Adobe Reader.
-void OpenUsingReader(TabContents* tab, const WebPluginInfo& reader_plugin) {
+void OpenUsingReader(TabContents* tab,
+                     const WebPluginInfo& reader_plugin,
+                     InfoBarDelegate* old_delegate) {
   PluginService::OverriddenPlugin plugin;
   plugin.render_process_id = tab->GetRenderProcessHost()->id();
   plugin.render_view_id = tab->render_view_host()->routing_id();
   plugin.url = tab->GetURL();
   plugin.plugin = reader_plugin;
+  // The plugin is disabled, so enable it to get around the renderer check.
+  // Also give it a new version so that the renderer doesn't show the blocked
+  // plugin UI if it's vulnerable, since we already went through the
+  // interstitial.
+  plugin.plugin.enabled = WebPluginInfo::USER_ENABLED;
+  plugin.plugin.version = ASCIIToUTF16("11.0.0.0");
 
   PluginService::GetInstance()->OverridePluginForTab(plugin);
   tab->render_view_host()->ReloadFrame();
+
+  if (old_delegate) {
+    tab->ReplaceInfoBar(
+        old_delegate, new PDFEnableAdobeReaderConfirmInfoBarDelegate(tab));

[Chris Evans, 2011/01/26 23:31:30]

Do the common construction of that object once, outside the if?

[jam, 2011/01/26 23:45:57]

Done.

+  } else {
+    tab->AddInfoBar(new PDFEnableAdobeReaderConfirmInfoBarDelegate(tab));
+  }
 }
 
 // An interstitial to be used when the user chooses to open a PDF using Adobe
 // Reader, but it is out of date.
 class PDFUnsupportedFeatureInterstitial : public InterstitialPage {
  public:
   PDFUnsupportedFeatureInterstitial(
       TabContents* tab,
       const WebPluginInfo& reader_webplugininfo)
       : InterstitialPage(tab, false, tab->GetURL()),
         reader_webplugininfo_(reader_webplugininfo) {
+    UserMetrics::RecordAction(UserMetricsAction("PDF_ReaderInterstitialShown"));
   }
 
  protected:
   // InterstitialPage implementation.
   virtual std::string GetHTMLContents() {
     DictionaryValue strings;
     strings.SetString(
         "title",
         l10n_util::GetStringUTF16(IDS_READER_OUT_OF_DATE_BLOCKING_PAGE_TITLE));
     strings.SetString(
@@ skipping 14 matching lines @@
         l10n_util::GetStringUTF16(IDS_READER_OUT_OF_DATE_BLOCKING_PAGE_CANCEL));
 
     base::StringPiece html(ResourceBundle::GetSharedInstance().
         GetRawDataResource(IDR_READER_OUT_OF_DATE_HTML));
 
     return jstemplate_builder::GetI18nTemplateHtml(html, &strings);
   }
 
   virtual void CommandReceived(const std::string& command) {
     if (command == "0") {
+      UserMetrics::RecordAction(UserMetricsAction(
+          "PDF_ReaderInterstitialCancel"));
       DontProceed();
       return;
     }
 
     if (command == "1") {
+      UserMetrics::RecordAction(UserMetricsAction(
+          "PDF_ReaderInterstitialUpdate"));
       OpenReaderUpdateURL(tab());
     } else if (command == "2") {
-      OpenUsingReader(tab(), reader_webplugininfo_);
+      UserMetrics::RecordAction(UserMetricsAction(
+          "PDF_ReaderInterstitialIgnore"));
+      OpenUsingReader(tab(), reader_webplugininfo_, NULL);
     } else {
       NOTREACHED();
     }
     Proceed();
   }
 
  private:
   WebPluginInfo reader_webplugininfo_;
 
   DISALLOW_COPY_AND_ASSIGN(PDFUnsupportedFeatureInterstitial);
 };
 
 // The info bar delegate used to inform the user that we don't support a feature
 // in the PDF.
 class PDFUnsupportedFeatureConfirmInfoBarDelegate
     : public ConfirmInfoBarDelegate {
  public:
   PDFUnsupportedFeatureConfirmInfoBarDelegate(
       TabContents* tab_contents,
       PluginGroup* reader_group)  // NULL if Adobe Reader isn't installed.
       : ConfirmInfoBarDelegate(tab_contents),
         tab_contents_(tab_contents),
         reader_installed_(!!reader_group),
         reader_vulnerable_(false) {
     if (reader_installed_) {
+      UserMetrics::RecordAction(UserMetricsAction("PDF_UseReaderInfoBarShown"));
       std::vector<WebPluginInfo> plugins = reader_group->web_plugin_infos();
       DCHECK_EQ(plugins.size(), 1u);
       reader_webplugininfo_ = plugins[0];
 
       reader_vulnerable_ = reader_group->IsVulnerable();
       if (!reader_vulnerable_) {
         scoped_ptr<Version> version(PluginGroup::CreateVersionFromString(
             reader_webplugininfo_.version));
         if (version.get()) {
           if (version->components()[0] < kMinReaderVersionToUse)
             reader_vulnerable_ = true;
         }
       }
+    } else {
+      UserMetrics::RecordAction(UserMetricsAction(
+          "PDF_InstallReaderInfoBarShown"));
     }
   }
 
   // ConfirmInfoBarDelegate
   virtual void InfoBarClosed() {
     delete this;
   }
   virtual Type GetInfoBarType() {
     return PAGE_ACTION_TYPE;
   }
   virtual bool Accept() {
-    LaunchReader();
+    if (!reader_installed_) {
+      UserMetrics::RecordAction(UserMetricsAction(
+          "PDF_InstallReaderInfoBarOK"));
+      OpenReaderUpdateURL(tab_contents_);
+      return true;
+    }
+
+    UserMetrics::RecordAction(UserMetricsAction(
+          "PDF_UseReaderInfoBarOK"));
+
+    if (reader_vulnerable_) {
+      PDFUnsupportedFeatureInterstitial* interstitial = new
+          PDFUnsupportedFeatureInterstitial(
+              tab_contents_, reader_webplugininfo_);
+      interstitial->Show();
+      return true;
+    }
+
+    OpenUsingReader(tab_contents_, reader_webplugininfo_, this);
+    return false;
+  }
+  virtual bool Cancel() {
+    if (reader_installed_) {
+      UserMetrics::RecordAction(UserMetricsAction(
+          "PDF_UseReaderInfoBarCancel"));
+    } else {
+      UserMetrics::RecordAction(UserMetricsAction(
+          "PDF_InstallReaderInfoBarCancel"));
+    }
     return true;
   }
   virtual int GetButtons() const {
     return BUTTON_OK | BUTTON_CANCEL;
   }
   virtual string16 GetButtonLabel(InfoBarButton button) const {
     switch (button) {
       case BUTTON_OK:
         return l10n_util::GetStringUTF16(
             IDS_CONFIRM_MESSAGEBOX_YES_BUTTON_LABEL);
       case BUTTON_CANCEL:
         return l10n_util::GetStringUTF16(
             IDS_CONFIRM_MESSAGEBOX_NO_BUTTON_LABEL);
       default:
         // All buttons are labeled above.
         NOTREACHED() << "Bad button id " << button;
         return string16();
     }
   }
   virtual string16 GetMessageText() const {
     return l10n_util::GetStringUTF16(reader_installed_ ?
         IDS_PDF_INFOBAR_QUESTION_READER_INSTALLED :
         IDS_PDF_INFOBAR_QUESTION_READER_NOT_INSTALLED);
   }
 
  private:
-  void LaunchReader() {
-    if (!reader_installed_) {
-      OpenReaderUpdateURL(tab_contents_);
-      return;
-    }
-
-    if (reader_vulnerable_) {
-      PDFUnsupportedFeatureInterstitial* interstitial = new
-          PDFUnsupportedFeatureInterstitial(
-              tab_contents_, reader_webplugininfo_);
-      interstitial->Show();
-      return;
-    }
-
-    OpenUsingReader(tab_contents_, reader_webplugininfo_);
-  }
-
   TabContents* tab_contents_;
   bool reader_installed_;
   bool reader_vulnerable_;
   WebPluginInfo reader_webplugininfo_;
 
   DISALLOW_IMPLICIT_CONSTRUCTORS(PDFUnsupportedFeatureConfirmInfoBarDelegate);
 };
 
 }  // namespace
 
 void PDFHasUnsupportedFeature(TabContents* tab) {
 #if !defined(OS_WIN)
   // Only works for Windows for now.  For Mac, we'll have to launch the file
   // externally since Adobe Reader doesn't work inside Chrome.
   return;
 #endif
+  string16 reader_group_name(ASCIIToUTF16(PluginGroup::kAdobeReaderGroupName));
+
+  // If the Reader plugin is disabled by policy, don't prompt them.
+  if (PluginGroup::IsPluginNameDisabledByPolicy(reader_group_name))
+    return;

[Chris Evans, 2011/01/26 23:31:30]

Is there a command line flag for disabling this? I expect there will be some
security-conscious enterprises who wish to avoid offering to install (or bounce
out to) Reader all the time.

[jam, 2011/01/26 23:45:57]

this code is doing exactly what you're asking for :)  if group policy disables
reader, we don't offer this.

 
   PluginGroup* reader_group = NULL;
   std::vector<PluginGroup> plugin_groups;
   PluginList::Singleton()->GetPluginGroups(
       false, &plugin_groups);
-  string16 reader_group_name(UTF8ToUTF16(PluginGroup::kAdobeReaderGroupName));
   for (size_t i = 0; i < plugin_groups.size(); ++i) {
     if (plugin_groups[i].GetGroupName() == reader_group_name) {
       reader_group = &plugin_groups[i];
       break;
     }
   }
 
-  // If the plugin is disabled by policy or by the user, don't prompt them.
-  if (reader_group && !reader_group->Enabled())
-    return;
-
   tab->AddInfoBar(new PDFUnsupportedFeatureConfirmInfoBarDelegate(
       tab, reader_group));
 }