Propagates the search result of start page to the app-list search box.

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 f729ec022956db1ad42325cf0d15e14d6195ad6b..c967c4c72d81b2b9ab4eb99d1421b27ea91da5ba 100644
--- a/chrome/browser/ui/app_list/start_page_service.h
+++ b/chrome/browser/ui/app_list/start_page_service.h
@@ -10,6 +10,7 @@
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/strings/string16.h"
 #include "components/browser_context_keyed_service/browser_context_keyed_service.h"
 #include "content/public/browser/web_contents.h"
 
@@ -21,6 +22,7 @@ class Profile;
 
 namespace app_list {
 
+class AppListModel;
 class RecommendedApps;
 
 // StartPageService collects data to be displayed in app list's start page
@@ -32,8 +34,11 @@ class StartPageService : public BrowserContextKeyedService {
   // Gets the instance for the given profile.
   static StartPageService* Get(Profile* profile);
 
+  void SetAppListModel(AppListModel* model);
+
   content::WebContents* contents() { return contents_.get(); }
   RecommendedApps* recommended_apps() { return recommended_apps_.get(); }
+  void OnSearch(const base::string16& query);
 
  private:
   // A BrowserContextKeyedServiceFactory for this service.
@@ -55,6 +60,7 @@ class StartPageService : public BrowserContextKeyedService {
   virtual void Shutdown() OVERRIDE;
 
   Profile* profile_;
+  AppListModel* model_;

[xiyuan, 2013/11/01 04:24:12]

I don't like to put the weak reference to model here.

Would the following way work?
- Add a StartPageServiceObserver interface with a OnSearch (or
OnSpeechRecognized) method;
- Make AppListViewDelegate such an observer and observes StartPageService
- When StartPageHandler needs to handle "search" (or "speechRecognized")
message, it finds StartPageService and notifies the observers;
- AppListViewDelegate have access to the model and can update searchbox model to
trigger the search

What do you think?

[Jun Mukai, 2013/11/01 17:25:24]

Agreed, done.  PTAL.

   scoped_ptr<content::WebContents> contents_;
   scoped_ptr<StartPageWebContentsDelegate> contents_delegate_;
   scoped_ptr<ExitObserver> exit_observer_;