Update UMA user actions parsing, primarily to include WebUI metrics.

chrome/browser/pdf_unsupported_feature.cc

Index: chrome/browser/pdf_unsupported_feature.cc
diff --git a/chrome/browser/pdf_unsupported_feature.cc b/chrome/browser/pdf_unsupported_feature.cc
index 0e68c0a978d379bcfc8eb9d0c8aff175239a6fa9..90c87b6febf1586e7b3b1a598993993fb2c31bbb 100644
--- a/chrome/browser/pdf_unsupported_feature.cc
+++ b/chrome/browser/pdf_unsupported_feature.cc
@@ -349,8 +349,12 @@ bool PDFUnsupportedFeatureInfoBarDelegate::OnYes() {
 }
 
 void PDFUnsupportedFeatureInfoBarDelegate::OnNo() {
-  UserMetrics::RecordAction(UserMetricsAction(reader_installed_ ?

[James Hawkins, 2011/07/07 16:30:50]

What's wrong with this version?

[Ilya Sherman, 2011/07/07 18:59:44]

This script scans for the literal string """ UserMetricsAction(" """ to detect
metrics used with UserMetrics::RecordAction.  I could rewrite this as

UserMetrics::RecordAction(reader_installed_ ?
    UserMetricsAction("PDF_UseReaderInfoBarCancel") :
    UserMetricsAction("PDF_InstallReaderInfoBarCancel"));

if that style is preferable.

[James Hawkins, 2011/07/07 20:44:19]

Yea, I think I prefer that. It's at least in the spirit of the old code.

[Ilya Sherman, 2011/07/07 21:01:01]

Ok, switched to that style.

-      "PDF_UseReaderInfoBarCancel" : "PDF_InstallReaderInfoBarCancel"));
+  if (reader_installed_) {
+    UserMetrics::RecordAction(UserMetricsAction("PDF_UseReaderInfoBarCancel"));
+  } else {
+    UserMetrics::RecordAction(
+        UserMetricsAction("PDF_InstallReaderInfoBarCancel"));
+  }
 }
 
 }  // namespace