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

Unified Diff: chrome/browser/sync/glue/session_change_processor.cc

Issue 26594002: Chrome sync: Put WeakPtrFactory members at the end (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge (see prior patch for try jobs) Created 7 years, 2 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/glue/session_change_processor.cc
diff --git a/chrome/browser/sync/glue/session_change_processor.cc b/chrome/browser/sync/glue/session_change_processor.cc
index efc51ee24360a6068a02358458d2ad62d9f7b703..00cba9ee8ed5525ccf402a104c2ecac2064510de 100644
--- a/chrome/browser/sync/glue/session_change_processor.cc
+++ b/chrome/browser/sync/glue/session_change_processor.cc
@@ -58,10 +58,10 @@ SessionChangeProcessor::SessionChangeProcessor(
DataTypeErrorHandler* error_handler,
SessionModelAssociator* session_model_associator)
: ChangeProcessor(error_handler),
- weak_ptr_factory_(this),
session_model_associator_(session_model_associator),
profile_(NULL),
- setup_for_test_(false) {
+ setup_for_test_(false),
+ weak_ptr_factory_(this) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
DCHECK(error_handler);
DCHECK(session_model_associator_);
@@ -72,10 +72,10 @@ SessionChangeProcessor::SessionChangeProcessor(
SessionModelAssociator* session_model_associator,
bool setup_for_test)
: ChangeProcessor(error_handler),
- weak_ptr_factory_(this),
session_model_associator_(session_model_associator),
profile_(NULL),
- setup_for_test_(setup_for_test) {
+ setup_for_test_(setup_for_test),
+ weak_ptr_factory_(this) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
DCHECK(error_handler);
DCHECK(session_model_associator_);

Powered by Google App Engine
This is Rietveld 408576698