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

Unified Diff: chrome/browser/net/sdch_dictionary_fetcher.h

Issue 298063006: Make SdchManager per-profile. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove whitespace before asterix. Created 6 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/net/sdch_dictionary_fetcher.h
diff --git a/chrome/browser/net/sdch_dictionary_fetcher.h b/chrome/browser/net/sdch_dictionary_fetcher.h
index 96617438fb90477afbfca891aebd4e64e1d7f51b..7eea42aba4fe26dfd0c000f75c9fd367dd564720 100644
--- a/chrome/browser/net/sdch_dictionary_fetcher.h
+++ b/chrome/browser/net/sdch_dictionary_fetcher.h
@@ -29,13 +29,13 @@ class SdchDictionaryFetcher
public net::SdchFetcher,
public base::NonThreadSafe {
public:
- explicit SdchDictionaryFetcher(net::URLRequestContextGetter* context);
+ // Consumer must guarantee that the SdchManager pointer outlives
+ // this object. The current implementation guarantees this by
+ // the SdchManager owning this object.
+ SdchDictionaryFetcher(net::SdchManager* manager,
+ net::URLRequestContextGetter* context);
virtual ~SdchDictionaryFetcher();
- // Stop fetching dictionaries, and abandon any current URLFetcheer operations
- // so that the IO thread can be stopped.
- static void Shutdown();
-
// Implementation of SdchFetcher class.
// This method gets the requested dictionary, and then calls back into the
// SdchManager class with the dictionary's text.
@@ -59,6 +59,8 @@ class SdchDictionaryFetcher
// completes (either successfully or with failure).
virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
+ net::SdchManager* const manager_;
+
// A queue of URLs that are being used to download dictionaries.
std::queue<GURL> fetch_queue_;
// The currently outstanding URL fetch of a dicitonary.

Powered by Google App Engine
This is Rietveld 408576698