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

chrome/browser/ui/app_list/start_page_service.h

 // Copyright 2013 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.
 
 #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"
 #include "base/strings/string16.h"
 #include "components/keyed_service/core/keyed_service.h"
 #include "content/public/browser/web_contents.h"
 #include "ui/app_list/speech_ui_model_observer.h"
 
 namespace extensions {
 class Extension;
@@ skipping 15 matching lines @@
   // Gets the instance for the given profile.
   static StartPageService* Get(Profile* profile);
 
   void AddObserver(StartPageObserver* observer);
   void RemoveObserver(StartPageObserver* observer);
 
   void AppListShown();
   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();
 
   // They return essentially the same web contents but might return NULL when
   // some flag disables the feature.
   content::WebContents* GetStartPageContents();
   content::WebContents* GetSpeechRecognitionContents();
 
   RecommendedApps* recommended_apps() { return recommended_apps_.get(); }
   Profile* profile() { return profile_; }
@@ skipping 26 matching lines @@
   Profile* profile_;
   scoped_ptr<content::WebContents> contents_;
   scoped_ptr<StartPageWebContentsDelegate> contents_delegate_;
   scoped_ptr<ProfileDestroyObserver> profile_destroy_observer_;
   scoped_ptr<RecommendedApps> recommended_apps_;
   SpeechRecognitionState state_;
   ObserverList<StartPageObserver> observers_;
   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);
 };
 
 }  // namespace app_list
 
 #endif  // CHROME_BROWSER_UI_APP_LIST_START_PAGE_SERVICE_H_