Implement onOpenResult.

chrome/browser/chromeos/launcher_search_provider/service.cc

Index: chrome/browser/chromeos/launcher_search_provider/service.cc
diff --git a/chrome/browser/chromeos/launcher_search_provider/service.cc b/chrome/browser/chromeos/launcher_search_provider/service.cc
index bf4d130ed2e17679457568cb94332f6ab4d9116a..e096cad0ed617087f371e7c02670fcb8360d6b19 100644
--- a/chrome/browser/chromeos/launcher_search_provider/service.cc
+++ b/chrome/browser/chromeos/launcher_search_provider/service.cc
@@ -84,6 +84,22 @@ void Service::OnQueryEnded() {
   is_query_running_ = false;
 }
 
+void Service::OnOpenResult(const extensions::Extension* extension,

[satorux1, 2015/04/17 06:21:52]

if it must not be null, how about making it const extensions::Extension&

[yawano, 2015/04/17 06:39:07]

In this method, we only need ExtensionId instead of Extension*. I changed the
interface from Extension* to ExtensionId.

[satorux1, 2015/04/17 06:50:21]

that's nicer! Law of Demeter

+                           const std::string& item_id) {
+  // If |extension| is not contained in listener extension, ignore it.
+  std::set<ExtensionId> extension_ids = GetListenerExtensionIds();
+  if (!ContainsValue(extension_ids, extension->id()))
+    return;
+
+  extensions::EventRouter* event_router =
+      extensions::EventRouter::Get(profile_);
+  event_router->DispatchEventToExtension(
+      extension->id(),
+      make_scoped_ptr(new extensions::Event(
+          api_launcher_search_provider::OnOpenResult::kEventName,
+          api_launcher_search_provider::OnOpenResult::Create(item_id))));
+}
+
 void Service::SetSearchResults(
     const extensions::Extension* extension,
     const std::string& query_id,