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

Side by Side Diff: chrome/browser/chromeos/policy/blocking_login_browsertest.cc

Issue 819133004: 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 5 years, 12 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 <string> 5 #include <string>
6 #include <vector> 6 #include <vector>
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/run_loop.h" 10 #include "base/run_loop.h"
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
80 const bool enroll_device; 80 const bool enroll_device;
81 }; 81 };
82 82
83 class BlockingLoginTest 83 class BlockingLoginTest
84 : public InProcessBrowserTest, 84 : public InProcessBrowserTest,
85 public content::NotificationObserver, 85 public content::NotificationObserver,
86 public testing::WithParamInterface<BlockingLoginTestParam> { 86 public testing::WithParamInterface<BlockingLoginTestParam> {
87 public: 87 public:
88 BlockingLoginTest() : profile_added_(NULL) {} 88 BlockingLoginTest() : profile_added_(NULL) {}
89 89
90 virtual void SetUpCommandLine(CommandLine* command_line) override { 90 virtual void SetUpCommandLine(base::CommandLine* command_line) override {
91 // Initialize the test server early, so that we can use its base url for 91 // Initialize the test server early, so that we can use its base url for
92 // the command line flags. 92 // the command line flags.
93 ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady()); 93 ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
94 94
95 // Use the login manager screens and the gaia auth extension. 95 // Use the login manager screens and the gaia auth extension.
96 command_line->AppendSwitch(switches::kLoginManager); 96 command_line->AppendSwitch(switches::kLoginManager);
97 command_line->AppendSwitch(switches::kForceLoginManagerInTests); 97 command_line->AppendSwitch(switches::kForceLoginManagerInTests);
98 command_line->AppendSwitchASCII(switches::kLoginProfile, "user"); 98 command_line->AppendSwitchASCII(switches::kLoginProfile, "user");
99 command_line->AppendSwitchASCII(::switches::kAuthExtensionPath, 99 command_line->AppendSwitchASCII(::switches::kAuthExtensionPath,
100 "gaia_auth"); 100 "gaia_auth");
(...skipping 246 matching lines...) Expand 10 before | Expand all | Expand 10 after
347 { 3, kUsernameOtherDomain, true }, 347 { 3, kUsernameOtherDomain, true },
348 { 4, kUsernameOtherDomain, true }, 348 { 4, kUsernameOtherDomain, true },
349 { 5, kUsernameOtherDomain, true }, 349 { 5, kUsernameOtherDomain, true },
350 }; 350 };
351 351
352 INSTANTIATE_TEST_CASE_P(BlockingLoginTestInstance, 352 INSTANTIATE_TEST_CASE_P(BlockingLoginTestInstance,
353 BlockingLoginTest, 353 BlockingLoginTest,
354 testing::ValuesIn(kBlockinLoginTestCases)); 354 testing::ValuesIn(kBlockinLoginTestCases));
355 355
356 } // namespace chromeos 356 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698