Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(467)

Unified Diff: chrome/browser/ui/webui/ntp/app_launcher_handler.h

Issue 7776001: ntp4: improved app install, try 2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove #app-id altogether Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/ntp/app_launcher_handler.h
diff --git a/chrome/browser/ui/webui/ntp/app_launcher_handler.h b/chrome/browser/ui/webui/ntp/app_launcher_handler.h
index fedeb3cd816564ba33fe43b0486958edf9d56b28..ed21da9828be99cb01623a7d89a7f29fbf898241 100644
--- a/chrome/browser/ui/webui/ntp/app_launcher_handler.h
+++ b/chrome/browser/ui/webui/ntp/app_launcher_handler.h
@@ -163,6 +163,9 @@ class AppLauncherHandler : public WebUIMessageHandler,
void OnFaviconForApp(FaviconService::Handle handle,
history::FaviconData data);
+ // Sends |highlight_app_id_| to the js.
+ void SetAppToBeHighlighted();
+
// The apps are represented in the extensions model, which
// outlives us since it's owned by our containing profile.
ExtensionService* const extension_service_;
@@ -194,6 +197,14 @@ class AppLauncherHandler : public WebUIMessageHandler,
// waiting to hear about success or failure from the extensions system.
bool attempted_bookmark_app_install_;
+ // True if we have executed HandleGetApps() at least once.
+ bool has_loaded_apps_;
+
+ // The ID of the app to be highlighted on the NTP (i.e. shown on the page
+ // and pulsed). This is done for new installs. The actual higlighting occurs
+ // when the app is added to the page (via getAppsCallback or appAdded).
+ std::string highlight_app_id_;
+
// Hold state for favicon requests.
CancelableRequestConsumerTSimple<AppInstallInfo*> favicon_consumer_;

Powered by Google App Engine
This is Rietveld 408576698