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

Unified Diff: chrome/browser/sync/sessions/sync_session_context.h

Issue 553015: Support for multiple sync ModelSafeWorkers.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 11 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
« no previous file with comments | « chrome/browser/sync/sessions/sync_session.cc ('k') | chrome/browser/sync/sessions/sync_session_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/sessions/sync_session_context.h
===================================================================
--- chrome/browser/sync/sessions/sync_session_context.h (revision 36603)
+++ chrome/browser/sync/sessions/sync_session_context.h (working copy)
@@ -31,6 +31,7 @@
namespace browser_sync {
class ConflictResolver;
+class ModelSafeWorkerRegistrar;
class ServerConnectionManager;
namespace sessions {
@@ -41,12 +42,12 @@
public:
SyncSessionContext(ServerConnectionManager* connection_manager,
syncable::DirectoryManager* directory_manager,
- ModelSafeWorker* model_safe_worker)
+ ModelSafeWorkerRegistrar* model_safe_worker_registrar)
: resolver_(NULL),
syncer_event_channel_(NULL),
connection_manager_(connection_manager),
directory_manager_(directory_manager),
- model_safe_worker_(model_safe_worker),
+ registrar_(model_safe_worker_registrar),
extensions_activity_monitor_(new ExtensionsActivityMonitor()),
notifications_enabled_(false) {
}
@@ -69,8 +70,8 @@
SyncerEventChannel* syncer_event_channel() {
return syncer_event_channel_;
}
- ModelSafeWorker* model_safe_worker() {
- return model_safe_worker_.get();
+ ModelSafeWorkerRegistrar* registrar() {
+ return registrar_;
}
ExtensionsActivityMonitor* extensions_monitor() {
return extensions_activity_monitor_;
@@ -103,9 +104,9 @@
ServerConnectionManager* const connection_manager_;
syncable::DirectoryManager* const directory_manager_;
- // A worker capable of processing work closures on a thread that is
- // guaranteed to be safe for model modifications.
- scoped_ptr<ModelSafeWorker> model_safe_worker_;
+ // A registrar of workers capable of processing work closures on a thread
+ // that is guaranteed to be safe for model modifications.
+ ModelSafeWorkerRegistrar* registrar_;
// We use this to stuff extensions activity into CommitMessages so the server
// can correlate commit traffic with extension-related bookmark mutations.
« no previous file with comments | « chrome/browser/sync/sessions/sync_session.cc ('k') | chrome/browser/sync/sessions/sync_session_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698