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

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

Issue 535683002: Fix use-after-free in HDDDTC shutdown (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix tests Created 6 years, 3 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
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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 <set> 5 #include <set>
6 #include <string> 6 #include <string>
7 #include <utility> 7 #include <utility>
8 #include <vector> 8 #include <vector>
9 9
10 #include "testing/gtest/include/gtest/gtest.h" 10 #include "testing/gtest/include/gtest/gtest.h"
(...skipping 344 matching lines...) Expand 10 before | Expand all | Expand 10 after
355 return AutocompleteSyncableService::FromWebDataService(wds)->AsWeakPtr(); 355 return AutocompleteSyncableService::FromWebDataService(wds)->AsWeakPtr();
356 } 356 }
357 357
358 ACTION_P(ReturnNewDataTypeManagerWithDebugListener, debug_listener) { 358 ACTION_P(ReturnNewDataTypeManagerWithDebugListener, debug_listener) {
359 return new sync_driver::DataTypeManagerImpl( 359 return new sync_driver::DataTypeManagerImpl(
360 base::Closure(), 360 base::Closure(),
361 debug_listener, 361 debug_listener,
362 arg1, 362 arg1,
363 arg2, 363 arg2,
364 arg3, 364 arg3,
365 arg4, 365 arg4);
366 arg5);
367 } 366 }
368 367
369 ACTION_P(MakeAutofillProfileSyncComponents, wds) { 368 ACTION_P(MakeAutofillProfileSyncComponents, wds) {
370 EXPECT_TRUE(BrowserThread::CurrentlyOn(BrowserThread::DB)); 369 EXPECT_TRUE(BrowserThread::CurrentlyOn(BrowserThread::DB));
371 if (!BrowserThread::CurrentlyOn(BrowserThread::DB)) 370 if (!BrowserThread::CurrentlyOn(BrowserThread::DB))
372 return base::WeakPtr<syncer::SyncableService>(); 371 return base::WeakPtr<syncer::SyncableService>();
373 return AutofillProfileSyncableService::FromWebDataService(wds)->AsWeakPtr(); 372 return AutofillProfileSyncableService::FromWebDataService(wds)->AsWeakPtr();
374 } 373 }
375 374
376 class AbstractAutofillFactory { 375 class AbstractAutofillFactory {
(...skipping 173 matching lines...) Expand 10 before | Expand all | Expand 10 after
550 549
551 ProfileSyncComponentsFactoryMock* components = 550 ProfileSyncComponentsFactoryMock* components =
552 sync_service_->components_factory_mock(); 551 sync_service_->components_factory_mock();
553 DataTypeController* data_type_controller = 552 DataTypeController* data_type_controller =
554 factory->CreateDataTypeController(components, profile_, sync_service_); 553 factory->CreateDataTypeController(components, profile_, sync_service_);
555 factory->SetExpectation(components, 554 factory->SetExpectation(components,
556 sync_service_, 555 sync_service_,
557 web_data_service_.get(), 556 web_data_service_.get(),
558 data_type_controller); 557 data_type_controller);
559 558
560 EXPECT_CALL(*components, CreateDataTypeManager(_, _, _, _, _, _)). 559 EXPECT_CALL(*components, CreateDataTypeManager(_, _, _, _, _)).
561 WillOnce(ReturnNewDataTypeManagerWithDebugListener( 560 WillOnce(ReturnNewDataTypeManagerWithDebugListener(
562 syncer::MakeWeakHandle(debug_ptr_factory_.GetWeakPtr()))); 561 syncer::MakeWeakHandle(debug_ptr_factory_.GetWeakPtr())));
563 562
564 EXPECT_CALL(*personal_data_manager_, IsDataLoaded()). 563 EXPECT_CALL(*personal_data_manager_, IsDataLoaded()).
565 WillRepeatedly(Return(true)); 564 WillRepeatedly(Return(true));
566 565
567 // We need tokens to get the tests going 566 // We need tokens to get the tests going
568 ProfileOAuth2TokenServiceFactory::GetForProfile(profile_) 567 ProfileOAuth2TokenServiceFactory::GetForProfile(profile_)
569 ->UpdateCredentials("test_user@gmail.com", "oauth2_login_token"); 568 ->UpdateCredentials("test_user@gmail.com", "oauth2_login_token");
570 569
(...skipping 810 matching lines...) Expand 10 before | Expand all | Expand 10 after
1381 std::vector<AutofillEntry> sync_entries; 1380 std::vector<AutofillEntry> sync_entries;
1382 std::vector<AutofillProfile> sync_profiles; 1381 std::vector<AutofillProfile> sync_profiles;
1383 ASSERT_TRUE(GetAutofillEntriesFromSyncDB(&sync_entries, &sync_profiles)); 1382 ASSERT_TRUE(GetAutofillEntriesFromSyncDB(&sync_entries, &sync_profiles));
1384 EXPECT_EQ(3U, sync_entries.size()); 1383 EXPECT_EQ(3U, sync_entries.size());
1385 EXPECT_EQ(0U, sync_profiles.size()); 1384 EXPECT_EQ(0U, sync_profiles.size());
1386 for (size_t i = 0; i < sync_entries.size(); i++) { 1385 for (size_t i = 0; i < sync_entries.size(); i++) {
1387 DVLOG(1) << "Entry " << i << ": " << sync_entries[i].key().name() 1386 DVLOG(1) << "Entry " << i << ": " << sync_entries[i].key().name()
1388 << ", " << sync_entries[i].key().value(); 1387 << ", " << sync_entries[i].key().value();
1389 } 1388 }
1390 } 1389 }
OLDNEW
« no previous file with comments | « chrome/browser/sync/profile_sync_service.cc ('k') | chrome/browser/sync/profile_sync_service_startup_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698