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

Side by Side Diff: chrome/browser/ui/sync/sync_promo_ui_unittest.cc

Issue 1257623002: Componentize FakeSigninManager and SigninManager prefs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Review comment Created 5 years, 5 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/ui/sync/sync_promo_ui.h" 5 #include "chrome/browser/ui/sync/sync_promo_ui.h"
6 6
7 #include "base/basictypes.h" 7 #include "base/basictypes.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
11 #include "chrome/browser/signin/fake_signin_manager.h" 11 #include "chrome/browser/signin/fake_signin_manager_builder.h"
12 #include "chrome/browser/signin/signin_manager_factory.h" 12 #include "chrome/browser/signin/signin_manager_factory.h"
13 #include "chrome/common/chrome_switches.h" 13 #include "chrome/common/chrome_switches.h"
14 #include "chrome/test/base/testing_profile.h" 14 #include "chrome/test/base/testing_profile.h"
15 #include "content/public/test/test_browser_thread_bundle.h" 15 #include "content/public/test/test_browser_thread_bundle.h"
16 #include "testing/gtest/include/gtest/gtest.h" 16 #include "testing/gtest/include/gtest/gtest.h"
17 17
18 class SyncPromoUITest : public testing::Test { 18 class SyncPromoUITest : public testing::Test {
19 public: 19 public:
20 SyncPromoUITest() {} 20 SyncPromoUITest() {}
21 21
22 // testing::Test: 22 // testing::Test:
23 void SetUp() override { 23 void SetUp() override {
24 testing::Test::SetUp(); 24 testing::Test::SetUp();
25 TestingProfile::Builder builder; 25 TestingProfile::Builder builder;
26 builder.AddTestingFactory(SigninManagerFactory::GetInstance(), 26 builder.AddTestingFactory(SigninManagerFactory::GetInstance(),
27 FakeSigninManagerBase::Build); 27 BuildFakeSigninManagerBase);
28 profile_ = builder.Build(); 28 profile_ = builder.Build();
29 } 29 }
30 30
31 protected: 31 protected:
32 void DisableSync() { 32 void DisableSync() {
33 base::CommandLine::ForCurrentProcess()->AppendSwitch( 33 base::CommandLine::ForCurrentProcess()->AppendSwitch(
34 switches::kDisableSync); 34 switches::kDisableSync);
35 } 35 }
36 36
37 content::TestBrowserThreadBundle thread_bundle_; 37 content::TestBrowserThreadBundle thread_bundle_;
(...skipping 13 matching lines...) Expand all
51 // Verifies that ShouldShowSyncPromo returns true if all conditions to 51 // Verifies that ShouldShowSyncPromo returns true if all conditions to
52 // show the promo are met. 52 // show the promo are met.
53 TEST_F(SyncPromoUITest, ShouldShowSyncPromoSyncEnabled) { 53 TEST_F(SyncPromoUITest, ShouldShowSyncPromoSyncEnabled) {
54 #if defined(OS_CHROMEOS) 54 #if defined(OS_CHROMEOS)
55 // No sync promo on CrOS. 55 // No sync promo on CrOS.
56 EXPECT_FALSE(SyncPromoUI::ShouldShowSyncPromo(profile_.get())); 56 EXPECT_FALSE(SyncPromoUI::ShouldShowSyncPromo(profile_.get()));
57 #else 57 #else
58 EXPECT_TRUE(SyncPromoUI::ShouldShowSyncPromo(profile_.get())); 58 EXPECT_TRUE(SyncPromoUI::ShouldShowSyncPromo(profile_.get()));
59 #endif 59 #endif
60 } 60 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698