Open the launcher when hotword is triggered in always-on mode.

chrome/browser/ui/app_list/start_page_service.h

Index: chrome/browser/ui/app_list/start_page_service.h
diff --git a/chrome/browser/ui/app_list/start_page_service.h b/chrome/browser/ui/app_list/start_page_service.h
index 33db7b2e32c43f8808b3f414c45eeec6c87e1e09..8c46f55e0e29d1126af19f3a95540f3be6b28ede 100644
--- a/chrome/browser/ui/app_list/start_page_service.h
+++ b/chrome/browser/ui/app_list/start_page_service.h
@@ -5,9 +5,11 @@
 #ifndef CHROME_BROWSER_UI_APP_LIST_START_PAGE_SERVICE_H_
 #define CHROME_BROWSER_UI_APP_LIST_START_PAGE_SERVICE_H_
 
+#include <list>
 #include <vector>
 
 #include "base/basictypes.h"
+#include "base/callback_forward.h"

[tapted, 2014/10/09 03:37:00]

nit: pretty sure if you have a data member you "need" the whole header, so
something else is probably including it for you

[Anand Mistry (off Chromium), 2014/10/13 21:52:00]

Ah right. I put this in before I noticed that Callbacks work differently than
google code.

P.S. I prefer google-style callbacks.

 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
@@ -43,6 +45,9 @@ class StartPageService : public KeyedService {
   void AppListHidden();
   void ToggleSpeechRecognition();
 
+  // Called when the WebUI has finished loading.
+  void WebUILoaded();
+
   // Returns true if the hotword is enabled in the app-launcher.
   bool HotwordEnabled();
 
@@ -89,6 +94,9 @@ class StartPageService : public KeyedService {
   bool speech_button_toggled_manually_;
   bool speech_result_obtained_;
 
+  bool webui_finished_loading_;
+  std::list<base::Closure> pending_webui_callbacks_;

[Matt Giuca, 2014/10/09 18:07:12]

Is there a reason to use list over vector? I've never seen list used in Chromium
code; it's almost always a vector.

[Anand Mistry (off Chromium), 2014/10/13 21:52:00]

In this particular case, it doesn't matter since this will contain either 0 or 1
elements in the majority of cases. But vector does suck for some things compared
to a list.

+
   DISALLOW_COPY_AND_ASSIGN(StartPageService);
 };