Add metrics for window.external.AddSearchProvider and

chrome/renderer/external_extension.cc

Index: chrome/renderer/external_extension.cc
diff --git a/chrome/renderer/external_extension.cc b/chrome/renderer/external_extension.cc
index 014d388f581c7150bd237603c2fcc42dd813aff3..070c76d6410686b2838a97a9a0243ef40ff55aaf 100644
--- a/chrome/renderer/external_extension.cc
+++ b/chrome/renderer/external_extension.cc
@@ -6,6 +6,7 @@
 
 #include "chrome/common/render_messages.h"
 #include "chrome/common/search_provider.h"
+#include "content/public/renderer/render_thread.h"
 #include "content/public/renderer/render_view.h"
 #include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebLocalFrame.h"
@@ -116,6 +117,8 @@ void ExternalExtensionWrapper::AddSearchProvider(
 
   GURL osdd_url = GURL(webframe->document().url()).Resolve(osdd_string);
   if (!osdd_url.is_empty() && osdd_url.is_valid()) {
+    content::RenderThread::Get()->RecordAction(
+        base::UserMetricsAction("window.external.AddSearchProvider"));
     render_view->Send(new ChromeViewHostMsg_PageHasOSDD(
         render_view->GetRoutingID(), webframe->document().url(), osdd_url,
         search_provider::EXPLICIT_PROVIDER));
@@ -144,11 +147,13 @@ void ExternalExtensionWrapper::IsSearchProviderInstalled(
   search_provider::InstallState install = search_provider::DENIED;
   GURL inquiry_url = GURL(webframe->document().url()).Resolve(name);
   if (!inquiry_url.is_empty()) {
-      render_view->Send(new ChromeViewHostMsg_GetSearchProviderInstallState(
-          render_view->GetRoutingID(),
-          webframe->document().url(),
-          inquiry_url,
-          &install));
+    content::RenderThread::Get()->RecordAction(
+        base::UserMetricsAction("window.external.IsSearchProviderInstalled"));
+    render_view->Send(new ChromeViewHostMsg_GetSearchProviderInstallState(
+        render_view->GetRoutingID(),
+        webframe->document().url(),
+        inquiry_url,
+        &install));
   }
 
   if (install == search_provider::DENIED) {
@@ -157,6 +162,9 @@ void ExternalExtensionWrapper::IsSearchProviderInstalled(
     isolate->ThrowException(v8::Exception::Error(v8::String::Empty(isolate)));
     return;
   }
+
+  content::RenderThread::Get()->RecordAction(
+      base::UserMetricsAction("window.external.IsSearchProviderInstalled_success"));

[Peter Kasting, 2015/10/13 18:36:20]

80 columns

[Evan Stade, 2015/10/13 18:40:57]

I don't think I can because of this comment[1], unless I reduce the indentation.

[1]
  // WARNING: When using base::UserMetricsAction, base::UserMetricsAction
  // and a string literal parameter must be on the same line, e.g.
  //   RenderThread::Get()->RecordAction(
  //       base::UserMetricsAction("my extremely long action name"));
  // because otherwise our processing scripts won't pick up on new actions.

[Peter Kasting, 2015/10/13 19:01:41]

Based on extract_actions.py line 53, I think this comment is incorrect.  You
should check to be sure, but I think the right thing to do is to remove or
modify that comment and rewrap your code.

   args.GetReturnValue().Set(static_cast<int32_t>(install));
 }