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

Side by Side Diff: chrome/browser/sync/test_profile_sync_service.cc

Issue 26594002: Chrome sync: Put WeakPtrFactory members at the end (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/sync/test_profile_sync_service.h" 5 #include "chrome/browser/sync/test_profile_sync_service.h"
6 6
7 #include "chrome/browser/chrome_notification_types.h" 7 #include "chrome/browser/chrome_notification_types.h"
8 #include "chrome/browser/signin/signin_manager.h" 8 #include "chrome/browser/signin/signin_manager.h"
9 #include "chrome/browser/signin/signin_manager_factory.h" 9 #include "chrome/browser/signin/signin_manager_factory.h"
10 #include "chrome/browser/sync/glue/data_type_controller.h" 10 #include "chrome/browser/sync/glue/data_type_controller.h"
(...skipping 25 matching lines...) Expand all
36 Profile* profile, 36 Profile* profile,
37 const base::WeakPtr<SyncPrefs>& sync_prefs, 37 const base::WeakPtr<SyncPrefs>& sync_prefs,
38 syncer::TestIdFactory& id_factory, 38 syncer::TestIdFactory& id_factory,
39 base::Closure& callback, 39 base::Closure& callback,
40 bool set_initial_sync_ended_on_init, 40 bool set_initial_sync_ended_on_init,
41 bool synchronous_init, 41 bool synchronous_init,
42 bool fail_initial_download, 42 bool fail_initial_download,
43 syncer::StorageOption storage_option) 43 syncer::StorageOption storage_option)
44 : browser_sync::SyncBackendHost( 44 : browser_sync::SyncBackendHost(
45 profile->GetDebugName(), profile, sync_prefs), 45 profile->GetDebugName(), profile, sync_prefs),
46 weak_ptr_factory_(this),
47 id_factory_(id_factory), 46 id_factory_(id_factory),
48 callback_(callback), 47 callback_(callback),
49 fail_initial_download_(fail_initial_download), 48 fail_initial_download_(fail_initial_download),
50 set_initial_sync_ended_on_init_(set_initial_sync_ended_on_init), 49 set_initial_sync_ended_on_init_(set_initial_sync_ended_on_init),
51 synchronous_init_(synchronous_init), 50 synchronous_init_(synchronous_init),
52 storage_option_(storage_option) {} 51 storage_option_(storage_option),
52 weak_ptr_factory_(this) {}
53 53
54 SyncBackendHostForProfileSyncTest::~SyncBackendHostForProfileSyncTest() {} 54 SyncBackendHostForProfileSyncTest::~SyncBackendHostForProfileSyncTest() {}
55 55
56 namespace { 56 namespace {
57 57
58 scoped_ptr<syncer::HttpPostProviderFactory> MakeTestHttpBridgeFactory() { 58 scoped_ptr<syncer::HttpPostProviderFactory> MakeTestHttpBridgeFactory() {
59 return scoped_ptr<syncer::HttpPostProviderFactory>( 59 return scoped_ptr<syncer::HttpPostProviderFactory>(
60 new browser_sync::TestHttpBridgeFactory()); 60 new browser_sync::TestHttpBridgeFactory());
61 } 61 }
62 62
(...skipping 286 matching lines...) Expand 10 before | Expand all | Expand 10 after
349 void FakeOAuth2TokenService::PersistCredentials( 349 void FakeOAuth2TokenService::PersistCredentials(
350 const std::string& account_id, 350 const std::string& account_id,
351 const std::string& refresh_token) { 351 const std::string& refresh_token) {
352 // Disabling the token persistence. 352 // Disabling the token persistence.
353 } 353 }
354 354
355 void FakeOAuth2TokenService::ClearPersistedCredentials( 355 void FakeOAuth2TokenService::ClearPersistedCredentials(
356 const std::string& account_id) { 356 const std::string& account_id) {
357 // Disabling the token persistence. 357 // Disabling the token persistence.
358 } 358 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698