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

Unified Diff: chrome/browser/component_updater/component_updater_service.h

Issue 18006003: Consistently use notifications from component updater w/ on-demand PNaCl. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: cleanup test a bit Created 7 years, 6 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/component_updater/component_updater_service.h
diff --git a/chrome/browser/component_updater/component_updater_service.h b/chrome/browser/component_updater/component_updater_service.h
index bf011eb43e022d3940a0b874390ab09923eab858..34f70bb96fff58c149f09096dca885669e78651f 100644
--- a/chrome/browser/component_updater/component_updater_service.h
+++ b/chrome/browser/component_updater/component_updater_service.h
@@ -20,6 +20,10 @@ class DictionaryValue;
class FilePath;
}
+namespace content {
+class NotificationSource;
+}
+
class ComponentPatcher;
// Component specific installers must derive from this class and implement
@@ -168,6 +172,14 @@ class ComponentUpdateService {
// via the Install() hook.
virtual Status CheckForUpdateSoon(const CrxComponent& component) = 0;
+ typedef std::string UpdateSource;
cpu_(ooo_6.6-7.5) 2013/07/01 18:30:22 not big fan of this typedef
jvoung (off chromium) 2013/07/02 01:04:26 Ok, removed and changed back to string.
+
+ // Return true if a NOTIFICATION_COMPONENT_UPDATE_FOUND or
+ // NOTIFICATION_COMPONENT_UPDATE_READY is for the given component.
+ static bool IsUpdateNotificationForComponent(
+ const content::NotificationSource& src,
+ const CrxComponent& component);
+
virtual ~ComponentUpdateService() {}
};

Powered by Google App Engine
This is Rietveld 408576698