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

Unified Diff: sync/notifier/non_blocking_invalidator.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: sync/notifier/non_blocking_invalidator.cc
diff --git a/sync/notifier/non_blocking_invalidator.cc b/sync/notifier/non_blocking_invalidator.cc
index d4c602b4e45e223b6cc8f9c99eb5d0161205efa9..ca89132a92c4c1e5027ee38ed2b05b7a8a77560d 100644
--- a/sync/notifier/non_blocking_invalidator.cc
+++ b/sync/notifier/non_blocking_invalidator.cc
@@ -141,13 +141,12 @@ NonBlockingInvalidator::NonBlockingInvalidator(
const WeakHandle<InvalidationStateTracker>&
invalidation_state_tracker,
const std::string& client_info)
- : weak_ptr_factory_(this),
- core_(
- new Core(MakeWeakHandle(weak_ptr_factory_.GetWeakPtr()))),
- parent_task_runner_(
- base::ThreadTaskRunnerHandle::Get()),
- network_task_runner_(notifier_options.request_context_getter->
- GetNetworkTaskRunner()) {
+ : parent_task_runner_(base::ThreadTaskRunnerHandle::Get()),
+ network_task_runner_(
+ notifier_options.request_context_getter->GetNetworkTaskRunner()),
+ weak_ptr_factory_(this) {
+ core_ = new Core(MakeWeakHandle(weak_ptr_factory_.GetWeakPtr()));
+
if (!network_task_runner_->PostTask(
FROM_HERE,
base::Bind(

Powered by Google App Engine
This is Rietveld 408576698