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

Side by Side Diff: chrome/test/live_sync/single_client_live_extensions_sync_test.cc

Issue 6873061: [Sync] Revamp and re-enable themes/extensions sync integration tests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync to head Created 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 "base/basictypes.h" 5 #include "base/basictypes.h"
6 #include "chrome/browser/profiles/profile.h"
7 #include "chrome/browser/sync/profile_sync_service_harness.h" 6 #include "chrome/browser/sync/profile_sync_service_harness.h"
8 #include "chrome/common/extensions/extension.h"
9 #include "chrome/test/live_sync/live_extensions_sync_test.h" 7 #include "chrome/test/live_sync/live_extensions_sync_test.h"
10 8
11 class SingleClientLiveExtensionsSyncTest : public LiveExtensionsSyncTest { 9 class SingleClientLiveExtensionsSyncTest : public LiveExtensionsSyncTest {
12 public: 10 public:
13 SingleClientLiveExtensionsSyncTest() 11 SingleClientLiveExtensionsSyncTest()
14 : LiveExtensionsSyncTest(SINGLE_CLIENT) {} 12 : LiveExtensionsSyncTest(SINGLE_CLIENT) {}
15 13
16 virtual ~SingleClientLiveExtensionsSyncTest() {} 14 virtual ~SingleClientLiveExtensionsSyncTest() {}
17 15
18 private: 16 private:
19 DISALLOW_COPY_AND_ASSIGN(SingleClientLiveExtensionsSyncTest); 17 DISALLOW_COPY_AND_ASSIGN(SingleClientLiveExtensionsSyncTest);
20 }; 18 };
21 19
22 IN_PROC_BROWSER_TEST_F(SingleClientLiveExtensionsSyncTest, 20 IN_PROC_BROWSER_TEST_F(SingleClientLiveExtensionsSyncTest,
23 StartWithNoExtensions) { 21 StartWithNoExtensions) {
24 ASSERT_TRUE(SetupSync()); 22 ASSERT_TRUE(SetupSync());
25 23
26 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier()); 24 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier());
27 } 25 }
28 26
29 // TODO(rsimha): Enable after http://crbug.com/70028 is fixed.
30 IN_PROC_BROWSER_TEST_F(SingleClientLiveExtensionsSyncTest, 27 IN_PROC_BROWSER_TEST_F(SingleClientLiveExtensionsSyncTest,
31 DISABLED_StartWithSomeExtensions) { 28 StartWithSomeExtensions) {
32 ASSERT_TRUE(SetupClients()); 29 ASSERT_TRUE(SetupClients());
33 30
34 const int kNumExtensions = 5; 31 const int kNumExtensions = 5;
35 for (int i = 0; i < kNumExtensions; ++i) { 32 for (int i = 0; i < kNumExtensions; ++i) {
36 InstallExtension(GetProfile(0), GetExtension(i)); 33 InstallExtension(GetProfile(0), i);
37 InstallExtension(verifier(), GetExtension(i)); 34 InstallExtension(verifier(), i);
38 } 35 }
39 36
40 ASSERT_TRUE(SetupSync()); 37 ASSERT_TRUE(SetupSync());
41 38
42 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier()); 39 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier());
43 } 40 }
44 41
45 IN_PROC_BROWSER_TEST_F(SingleClientLiveExtensionsSyncTest, 42 IN_PROC_BROWSER_TEST_F(SingleClientLiveExtensionsSyncTest,
46 InstallSomeExtensions) { 43 InstallSomeExtensions) {
47 ASSERT_TRUE(SetupSync()); 44 ASSERT_TRUE(SetupSync());
48 45
49 const int kNumExtensions = 5; 46 const int kNumExtensions = 5;
50 for (int i = 0; i < kNumExtensions; ++i) { 47 for (int i = 0; i < kNumExtensions; ++i) {
51 InstallExtension(GetProfile(0), GetExtension(i)); 48 InstallExtension(GetProfile(0), i);
52 InstallExtension(verifier(), GetExtension(i)); 49 InstallExtension(verifier(), i);
53 } 50 }
54 51
55 ASSERT_TRUE(GetClient(0)->AwaitSyncCycleCompletion( 52 ASSERT_TRUE(GetClient(0)->AwaitSyncCycleCompletion(
56 "Waiting for extension changes.")); 53 "Waiting for extension changes."));
57 54
58 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier()); 55 ASSERT_TRUE(AllProfilesHaveSameExtensionsAsVerifier());
59 } 56 }
OLDNEW
« no previous file with comments | « chrome/test/live_sync/live_themes_sync_test.cc ('k') | chrome/test/live_sync/single_client_live_themes_sync_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698