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

Unified Diff: chrome/browser/sync/notifier/sync_notifier_factory.h

Issue 9550009: [Sync] Move dependency on Chrome switches from SyncNotifierFactory (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: sync to head Created 8 years, 9 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/sync/notifier/sync_notifier_factory.h
diff --git a/chrome/browser/sync/notifier/sync_notifier_factory.h b/chrome/browser/sync/notifier/sync_notifier_factory.h
index 96c825d6aa8533476bbc918882165164432b0646..d763d1a76cf100908358b095d016f826496e93fd 100644
--- a/chrome/browser/sync/notifier/sync_notifier_factory.h
+++ b/chrome/browser/sync/notifier/sync_notifier_factory.h
@@ -7,16 +7,10 @@
#include <string>
-#include "base/memory/ref_counted.h"
#include "base/memory/weak_ptr.h"
#include "chrome/browser/sync/notifier/invalidation_version_tracker.h"
#include "chrome/browser/sync/util/weak_handle.h"
-
-class CommandLine;
-
-namespace net {
-class URLRequestContextGetter;
-}
+#include "jingle/notifier/base/notifier_options.h"
namespace sync_notifier {
@@ -30,12 +24,10 @@ class SyncNotifierFactory {
// agent string. |invalidation_version_tracker| may be NULL (for
// tests).
SyncNotifierFactory(
+ const notifier::NotifierOptions& notifier_options,
const std::string& client_info,
- const scoped_refptr<net::URLRequestContextGetter>&
- request_context_getter,
const base::WeakPtr<InvalidationVersionTracker>&
- invalidation_version_tracker,
- const CommandLine& command_line);
+ invalidation_version_tracker);
~SyncNotifierFactory();
// Creates a sync notifier. Caller takes ownership of the returned
@@ -44,12 +36,11 @@ class SyncNotifierFactory {
SyncNotifier* CreateSyncNotifier();
private:
+ const notifier::NotifierOptions notifier_options_;
const std::string client_info_;
- const scoped_refptr<net::URLRequestContextGetter> request_context_getter_;
const InvalidationVersionMap initial_max_invalidation_versions_;
const browser_sync::WeakHandle<InvalidationVersionTracker>
invalidation_version_tracker_;
- const CommandLine& command_line_;
};
} // namespace sync_notifier
« no previous file with comments | « chrome/browser/sync/glue/sync_backend_host.cc ('k') | chrome/browser/sync/notifier/sync_notifier_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698