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

Unified Diff: chrome/browser/policy/app_pack_updater.h

Issue 11572044: Automatically retry the following URLFetchers when the network changes: (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Document the retry behavior Created 8 years 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
« no previous file with comments | « chrome/browser/net/gaia/gaia_oauth_fetcher.cc ('k') | chrome/browser/policy/app_pack_updater.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/policy/app_pack_updater.h
diff --git a/chrome/browser/policy/app_pack_updater.h b/chrome/browser/policy/app_pack_updater.h
index 057b481277b250134e773c16860211b98203264f..142ec9aee05a3a4c0bce7636bf7e64a3a3312672 100644
--- a/chrome/browser/policy/app_pack_updater.h
+++ b/chrome/browser/policy/app_pack_updater.h
@@ -18,7 +18,6 @@
#include "chrome/browser/extensions/updater/extension_downloader_delegate.h"
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
-#include "net/base/network_change_notifier.h"
class GURL;
@@ -45,7 +44,6 @@ class EnterpriseInstallAttributes;
// device policy to be locally cached and installed into the Demo user account
// at login time.
class AppPackUpdater : public content::NotificationObserver,
- public net::NetworkChangeNotifier::IPAddressObserver,
public extensions::ExtensionDownloaderDelegate {
public:
// Callback to listen for updates to the screensaver extension's path.
@@ -77,18 +75,13 @@ class AppPackUpdater : public content::NotificationObserver,
void OnDamagedFileDetected(const FilePath& path);
private:
- struct AppPackEntry {
- std::string update_url;
- bool update_checked;
- };
-
struct CacheEntry {
std::string path;
std::string cached_version;
};
- // Maps an extension ID to its update URL and update information.
- typedef std::map<std::string, AppPackEntry> PolicyEntryMap;
+ // Maps an extension ID to its update URL.
+ typedef std::map<std::string, std::string> PolicyEntryMap;
// Maps an extension ID to a CacheEntry.
typedef std::map<std::string, CacheEntry> CacheEntryMap;
@@ -100,9 +93,6 @@ class AppPackUpdater : public content::NotificationObserver,
const content::NotificationSource& source,
const content::NotificationDetails& details) OVERRIDE;
- // net::NetworkChangeNotifier::IPAddressObserver:
- virtual void OnIPAddressChanged() OVERRIDE;
-
// Loads the current policy and schedules a cache update.
void LoadPolicy();
@@ -186,10 +176,6 @@ class AppPackUpdater : public content::NotificationObserver,
// appropriate.
void SetScreenSaverPath(const FilePath& path);
- // Marks extension |id| in |app_pack_extensions_| as having already been
- // checked for updates, if it exists.
- void SetUpdateChecked(const std::string& id);
-
base::WeakPtrFactory<AppPackUpdater> weak_ptr_factory_;
// Observes failures to install CRX files.
« no previous file with comments | « chrome/browser/net/gaia/gaia_oauth_fetcher.cc ('k') | chrome/browser/policy/app_pack_updater.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698