Prefer opening PDF downloads in the browser.

tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index 85787bf4a2f36ad8a1f6fae45961c9ed54f53c96..b2aca69435ef7f115d7420c16c70047a0d391b88 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -2966,6 +2966,14 @@ other types of suffix sets.
   </summary>
 </histogram>
 
+<histogram name="Download.OpenMethod" enum="DownloadOpenMethod">
+  <summary>
+    Invocation count for methods of opening a download. For some file types,
+    Chrome defaults to opening the file in the browser instead of invoking the
+    system handler. The user has the option of overriding this behavior.
+  </summary>
+</histogram>
+
 <histogram name="Download.OpensOutstanding">
   <summary>The number of unopened downloads, when one is opened.</summary>
 </histogram>
@@ -21144,6 +21152,12 @@ other types of suffix sets.
   <int value="8" label="POTENTIALLY_UNWANTED"/>
 </enum>
 
+<enum name="DownloadOpenMethod" type="int">
+  <int value="0" label="Opened with plaform handler by default"/>
+  <int value="1" label="Opened in browser by default"/>
+  <int value="2" label="Opened with plaform handler by user choice"/>

[Ilya Sherman, 2013/11/05 23:14:03]

Should there also be an entry for "Opened in the browser by user choice"?

[asanka, 2013/11/05 23:24:39]

That would have been the "Open in Chrome" option that was removed in favor of
making that the default for PDF and allowing the user to manually choose the
platform handler.

I didn't add that value to the enum because now there's no direct way to
manually choose to open a download in Chrome. They could conceivably select
"Open" and select a file that they downloaded, but we can't easily tell at this
point whether it was the same file that the browser downloaded some time earlier
or some other file.

+</enum>
+
 <enum name="DownloadSavePackageEvent" type="int">
   <int value="0" label="Started"/>
   <int value="1" label="Cancelled"/>