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

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

Issue 816403003: Make callers of CommandLine use it via the base:: namespace. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix Created 6 years 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/profile_signin_confirmation_helper.h" 5 #include "chrome/browser/ui/sync/profile_signin_confirmation_helper.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/command_line.h" 8 #include "base/command_line.h"
9 #include "base/run_loop.h" 9 #include "base/run_loop.h"
10 #include "chrome/browser/first_run/first_run.h" 10 #include "chrome/browser/first_run/first_run.h"
11 #include "chrome/browser/ui/browser.h" 11 #include "chrome/browser/ui/browser.h"
12 #include "chrome/common/chrome_switches.h" 12 #include "chrome/common/chrome_switches.h"
13 #include "chrome/test/base/in_process_browser_test.h" 13 #include "chrome/test/base/in_process_browser_test.h"
14 #include "content/public/test/test_launcher.h" 14 #include "content/public/test/test_launcher.h"
15 #include "testing/gtest/include/gtest/gtest.h" 15 #include "testing/gtest/include/gtest/gtest.h"
16 16
17 class ProfileSigninConfirmationHelperBrowserTest : public InProcessBrowserTest { 17 class ProfileSigninConfirmationHelperBrowserTest : public InProcessBrowserTest {
18 public: 18 public:
19 ProfileSigninConfirmationHelperBrowserTest() {} 19 ProfileSigninConfirmationHelperBrowserTest() {}
20 20
21 void SetUpCommandLine(CommandLine* command_line) override { 21 void SetUpCommandLine(base::CommandLine* command_line) override {
22 // Force the first-run flow to trigger autoimport. 22 // Force the first-run flow to trigger autoimport.
23 InProcessBrowserTest::SetUpCommandLine(command_line); 23 InProcessBrowserTest::SetUpCommandLine(command_line);
24 command_line->AppendSwitch(switches::kForceFirstRun); 24 command_line->AppendSwitch(switches::kForceFirstRun);
25 } 25 }
26 26
27 private: 27 private:
28 DISALLOW_COPY_AND_ASSIGN(ProfileSigninConfirmationHelperBrowserTest); 28 DISALLOW_COPY_AND_ASSIGN(ProfileSigninConfirmationHelperBrowserTest);
29 }; 29 };
30 30
31 // http://crbug.com/321302 31 // http://crbug.com/321302
(...skipping 13 matching lines...) Expand all
45 // http://crbug.com/321302 45 // http://crbug.com/321302
46 #if defined(GOOGLE_CHROME_BUILD) && (defined(OS_MACOSX) || defined(OS_LINUX)) 46 #if defined(GOOGLE_CHROME_BUILD) && (defined(OS_MACOSX) || defined(OS_LINUX))
47 #define MAYBE_HasNoSyncedExtensions DISABLED_HasNoSyncedExtensions 47 #define MAYBE_HasNoSyncedExtensions DISABLED_HasNoSyncedExtensions
48 #else 48 #else
49 #define MAYBE_HasNoSyncedExtensions HasNoSyncedExtensions 49 #define MAYBE_HasNoSyncedExtensions HasNoSyncedExtensions
50 #endif 50 #endif
51 IN_PROC_BROWSER_TEST_F(ProfileSigninConfirmationHelperBrowserTest, 51 IN_PROC_BROWSER_TEST_F(ProfileSigninConfirmationHelperBrowserTest,
52 MAYBE_HasNoSyncedExtensions) { 52 MAYBE_HasNoSyncedExtensions) {
53 EXPECT_FALSE(ui::HasSyncedExtensions(browser()->profile())); 53 EXPECT_FALSE(ui::HasSyncedExtensions(browser()->profile()));
54 } 54 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698