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

Unified Diff: sync/notifier/invalidator_test_template.h

Issue 221963003: Reduce dependency of TiclInvalidationService on Profile (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Made InvalidationStateTracker not vend WeakPtrs. Created 6 years, 8 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/invalidator_test_template.h
diff --git a/sync/notifier/invalidator_test_template.h b/sync/notifier/invalidator_test_template.h
index 67cd0536eb7b956ed1c8eaad6b20769b6f670e65..d7851fd95f3e47d20c5c5fb33e41460dfd8f6cc4 100644
--- a/sync/notifier/invalidator_test_template.h
+++ b/sync/notifier/invalidator_test_template.h
@@ -20,8 +20,7 @@
// // Create the Invalidator implementation with the given parameters.
// void CreateInvalidator(
// const std::string& initial_state,
-// const base::WeakPtr<InvalidationStateTracker>&
-// invalidation_state_tracker) {
+// InvalidationStateTracker* invalidation_state_tracker) {
// ...
// }
//
@@ -100,14 +99,14 @@ class InvalidatorTest : public testing::Test {
}
Invalidator* CreateAndInitializeInvalidator() {
- this->delegate_.CreateInvalidator("fake_invalidator_client_id",
- "fake_initial_state",
- this->fake_tracker_.AsWeakPtr());
- Invalidator* const invalidator = this->delegate_.GetInvalidator();
+ delegate_.CreateInvalidator("fake_invalidator_client_id",
+ "fake_initial_state",
+ &fake_tracker_);
+ Invalidator* const invalidator = delegate_.GetInvalidator();
- this->delegate_.WaitForInvalidator();
+ delegate_.WaitForInvalidator();
invalidator->UpdateCredentials("foo@bar.com", "fake_token");
- this->delegate_.WaitForInvalidator();
+ delegate_.WaitForInvalidator();
return invalidator;
}

Powered by Google App Engine
This is Rietveld 408576698