app_list: Show apps grid after installing from cws result.

chrome/browser/ui/app_list/search/webstore_result.cc

 // 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.
 
 #include "chrome/browser/ui/app_list/search/webstore_result.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/memory/ref_counted.h"
@@ skipping 165 matching lines @@
 
 void WebstoreResult::OnInstallFailure(const std::string& extension_id) {}
 
 void WebstoreResult::OnExtensionInstalled(
     const extensions::Extension* extension) {
   if (extension->id() != app_id_)
     return;
 
   SetIsInstalling(false);
   UpdateActions();
+  controller_->ShowAppsGrid();

[benwells, 2013/08/01 22:53:12]

Can the websore result take a reference to the view instead of plumbing through
the controller? Are there lifetime issues with that if the view is hidden? It
would be much simpler....

[xiyuan, 2013/08/01 23:18:28]

Right now, the AppListModel is independent of AppListView. WebstoreResult is
part of the model. It does not sound right to put a view reference in the model.

Besides, the model is constructed without the view reference. To pass it down
would also take some work.

 }
 
 void WebstoreResult::OnExtensionLoaded(
     const extensions::Extension* extension) {}
 
 void WebstoreResult::OnExtensionUnloaded(
     const extensions::Extension* extension) {}
 
 void WebstoreResult::OnExtensionUninstalled(
     const extensions::Extension* extension) {}
 
 void WebstoreResult::OnAppsReordered() {}
 
 void WebstoreResult::OnAppInstalledToAppList(const std::string& extension_id) {}
 
 void WebstoreResult::OnShutdown() {
   StopObservingInstall();
 }
 
 ChromeSearchResultType WebstoreResult::GetType() {
   return SEARCH_WEBSTORE_SEARCH_RESULT;
 }
 
 }  // namespace app_list