Add a new field "source" in launchData of chrome.app.runtime.onLaunched() to trace launch source.

extensions/browser/api/app_runtime/app_runtime_api.cc

 // Copyright 2014 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 "extensions/browser/api/app_runtime/app_runtime_api.h"
 
+#include "base/metrics/histogram.h"
 #include "base/time/time.h"
 #include "base/values.h"
 #include "extensions/browser/event_router.h"
 #include "extensions/browser/extension_prefs.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/browser/extensions_browser_client.h"
 #include "extensions/browser/granted_file_entry.h"
 #include "extensions/common/api/app_runtime.h"
+#include "extensions/common/constants.h"
+#include "extensions/common/feature_switch.h"
 #include "url/gurl.h"
 
 using content::BrowserContext;
 
 namespace extensions {
 
 namespace app_runtime = core_api::app_runtime;
 
 namespace {
 
 void DispatchOnEmbedRequestedEventImpl(
     const std::string& extension_id,
     scoped_ptr<base::DictionaryValue> app_embedding_request_data,
     content::BrowserContext* context) {
   scoped_ptr<base::ListValue> args(new base::ListValue());
   args->Append(app_embedding_request_data.release());
   ExtensionSystem* system = ExtensionSystem::Get(context);
   scoped_ptr<Event> event(
       new Event(app_runtime::OnEmbedRequested::kEventName, args.Pass()));
   event->restrict_to_browser_context = context;
   system->event_router()->DispatchEventWithLazyListener(extension_id,
                                                         event.Pass());
 
   ExtensionPrefs::Get(context)
       ->SetLastLaunchTime(extension_id, base::Time::Now());
 }
 
 void DispatchOnLaunchedEventImpl(const std::string& extension_id,
                                  scoped_ptr<base::DictionaryValue> launch_data,
                                  BrowserContext* context) {

[benwells, 2014/10/30 06:04:28]

Please add source in here and do the metrics reporting here. BTW I don't think
the UMA reporting should be gated on the feature.

[cylee1, 2014/10/30 13:03:43]

Tried, but since source is already translated into a string when passed in this
function, I'll need to parse it back. Otherwise it needs some refactoring, which
you might want to avoid.

[benwells, 2014/10/30 22:30:55]

I meant to pass in the source as an app_runtime::LaunchSource parameter. As it
is only called in three places in this file I don't think it is a problem.

[cylee1, 2014/10/31 06:37:53]

Done.

   // "Forced app mode" is true for Chrome OS kiosk mode.
   launch_data->SetBoolean(
       "isKioskSession",
       ExtensionsBrowserClient::Get()->IsRunningInForcedAppMode());
   scoped_ptr<base::ListValue> args(new base::ListValue());
   args->Append(launch_data.release());
   scoped_ptr<Event> event(
       new Event(app_runtime::OnLaunched::kEventName, args.Pass()));
   event->restrict_to_browser_context = context;
   EventRouter::Get(context)
       ->DispatchEventWithLazyListener(extension_id, event.Pass());
   ExtensionPrefs::Get(context)
       ->SetLastLaunchTime(extension_id, base::Time::Now());
 }
 
+app_runtime::LaunchSource getLaunchSourceEnum(
+    extensions::AppLaunchSource source) {
+  switch (source) {
+    case extensions::SOURCE_APP_LAUNCHER:
+      return app_runtime::LAUNCH_SOURCE_APP_LAUNCHER;
+    case extensions::SOURCE_NEW_TAB_PAGE:
+      return app_runtime::LAUNCH_SOURCE_NEW_TAB_PAGE;
+    case extensions::SOURCE_RELOAD:
+      return app_runtime::LAUNCH_SOURCE_RELOAD;
+    case extensions::SOURCE_RESTART:
+      return app_runtime::LAUNCH_SOURCE_RESTART;
+    case extensions::SOURCE_LOAD_AND_LAUNCH:
+      return app_runtime::LAUNCH_SOURCE_LOAD_AND_LAUNCH;
+    case extensions::SOURCE_COMMAND_LINE:
+      return app_runtime::LAUNCH_SOURCE_COMMAND_LINE;
+    case extensions::SOURCE_FILE_HANDLER:
+      return app_runtime::LAUNCH_SOURCE_FILE_HANDLER;
+    case extensions::SOURCE_URL_HANDLER:
+      return app_runtime::LAUNCH_SOURCE_URL_HANDLER;
+
+    case extensions::SOURCE_SYSTEM_TRAY:
+      return app_runtime::LAUNCH_SOURCE_SYSTEM_TRAY;
+    case extensions::SOURCE_ABOUT_PAGE:
+      return app_runtime::LAUNCH_SOURCE_ABOUT_PAGE;
+    case extensions::SOURCE_KEYBOARD:
+      return app_runtime::LAUNCH_SOURCE_KEYBOARD;
+
+    default:
+      return app_runtime::LAUNCH_SOURCE_NONE;
+  }
+}
+
+void ReportMetrics(app_runtime::LaunchSource source) {
+  UMA_HISTOGRAM_ENUMERATION(
+      "Extensions.AppLaunchSource", source, NUM_APP_LAUNCH_SOURCES);
+}
+
 }  // namespace
 
 // static
 void AppRuntimeEventRouter::DispatchOnEmbedRequestedEvent(
     content::BrowserContext* context,
     scoped_ptr<base::DictionaryValue> embed_app_data,
     const Extension* extension) {
   DispatchOnEmbedRequestedEventImpl(
       extension->id(), embed_app_data.Pass(), context);
 }
 
 // static
 void AppRuntimeEventRouter::DispatchOnLaunchedEvent(
     BrowserContext* context,
-    const Extension* extension) {
-  scoped_ptr<base::DictionaryValue> launch_data(new base::DictionaryValue());
-  DispatchOnLaunchedEventImpl(extension->id(), launch_data.Pass(), context);
+    const Extension* extension,
+    extensions::AppLaunchSource source) {
+  app_runtime::LaunchData launch_data;
+  if (extensions::FeatureSwitch::trace_app_source()->IsEnabled()) {
+    launch_data.source = getLaunchSourceEnum(source);
+    ReportMetrics(launch_data.source);
+  }
+  DispatchOnLaunchedEventImpl(
+      extension->id(), launch_data.ToValue().Pass(), context);
 }
 
 // static
 void AppRuntimeEventRouter::DispatchOnRestartedEvent(
     BrowserContext* context,
     const Extension* extension) {
   scoped_ptr<base::ListValue> arguments(new base::ListValue());
   scoped_ptr<Event> event(
       new Event(app_runtime::OnRestarted::kEventName, arguments.Pass()));
   event->restrict_to_browser_context = context;
   EventRouter::Get(context)
       ->DispatchEventToExtension(extension->id(), event.Pass());
 }
 
 // static
 void AppRuntimeEventRouter::DispatchOnLaunchedEventWithFileEntries(
     BrowserContext* context,
     const Extension* extension,
     const std::string& handler_id,
     const std::vector<std::string>& mime_types,
     const std::vector<GrantedFileEntry>& file_entries) {
   // TODO(sergeygs): Use the same way of creating an event (using the generated
   // boilerplate) as below in DispatchOnLaunchedEventWithUrl.
-  scoped_ptr<base::DictionaryValue> launch_data(new base::DictionaryValue);
+  base::DictionaryValue* launch_data = new base::DictionaryValue();
   launch_data->SetString("id", handler_id);
-  scoped_ptr<base::ListValue> items(new base::ListValue);
+
+  if (extensions::FeatureSwitch::trace_app_source()->IsEnabled()) {
+    launch_data->SetString(
+        "source",
+        app_runtime::ToString(app_runtime::LAUNCH_SOURCE_FILE_HANDLER));
+    ReportMetrics(app_runtime::LAUNCH_SOURCE_FILE_HANDLER);
+  }
+
+  base::ListValue* items = new base::ListValue();
   DCHECK(file_entries.size() == mime_types.size());
   for (size_t i = 0; i < file_entries.size(); ++i) {
-    scoped_ptr<base::DictionaryValue> launch_item(new base::DictionaryValue);
-    launch_item->SetString("fileSystemId", file_entries[i].filesystem_id);
-    launch_item->SetString("baseName", file_entries[i].registered_name);
-    launch_item->SetString("mimeType", mime_types[i]);
-    launch_item->SetString("entryId", file_entries[i].id);
-    items->Append(launch_item.release());
+    base::DictionaryValue* launch_item = new base::DictionaryValue();
+
+    base::DictionaryValue* raw_entry = new base::DictionaryValue();
+    raw_entry->SetString("fileSystemId", file_entries[i].filesystem_id);
+    raw_entry->SetString("baseName", file_entries[i].registered_name);
+    raw_entry->SetString("entryId", file_entries[i].id);
+
+    launch_item->Set("rawEntry", raw_entry);
+    launch_item->SetString("type", mime_types[i]);
+
+    items->Append(launch_item);
   }
-  launch_data->Set("items", items.release());
-  DispatchOnLaunchedEventImpl(extension->id(), launch_data.Pass(), context);
+  launch_data->Set("items", items);
+  DispatchOnLaunchedEventImpl(
+      extension->id(), scoped_ptr<base::DictionaryValue>(launch_data), context);
 }
 
 // static
 void AppRuntimeEventRouter::DispatchOnLaunchedEventWithUrl(
     BrowserContext* context,
     const Extension* extension,
     const std::string& handler_id,
     const GURL& url,
     const GURL& referrer_url) {
   app_runtime::LaunchData launch_data;
   launch_data.id.reset(new std::string(handler_id));
   launch_data.url.reset(new std::string(url.spec()));
   launch_data.referrer_url.reset(new std::string(referrer_url.spec()));
+  if (extensions::FeatureSwitch::trace_app_source()->IsEnabled()) {
+    launch_data.source = app_runtime::LAUNCH_SOURCE_URL_HANDLER;
+    ReportMetrics(app_runtime::LAUNCH_SOURCE_URL_HANDLER);
+  }
   DispatchOnLaunchedEventImpl(
       extension->id(), launch_data.ToValue().Pass(), context);
 }
 
 }  // namespace extensions