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

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

Issue 1095843003: [cros] Modify oobe/login tests to work with webview (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review Created 5 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
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 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 const int steps; 78 const int steps;
79 const char* username; 79 const char* username;
80 const bool enroll_device; 80 const bool enroll_device;
81 }; 81 };
82 82
83 class BlockingLoginTest 83 class BlockingLoginTest
84 : public OobeBaseTest, 84 : public OobeBaseTest,
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 // TODO(nkostylev): Fix this test for webview. http://crbug.com/477402
90 set_use_webview(false);
91 }
89 92
90 void SetUpCommandLine(base::CommandLine* command_line) override { 93 void SetUpCommandLine(base::CommandLine* command_line) override {
91 OobeBaseTest::SetUpCommandLine(command_line); 94 OobeBaseTest::SetUpCommandLine(command_line);
92 95
93 command_line->AppendSwitchASCII(::switches::kAuthExtensionPath, 96 command_line->AppendSwitchASCII(::switches::kAuthExtensionPath,
94 "gaia_auth"); 97 "gaia_auth");
95 command_line->AppendSwitchASCII( 98 command_line->AppendSwitchASCII(
96 policy::switches::kDeviceManagementUrl, 99 policy::switches::kDeviceManagementUrl,
97 embedded_test_server()->GetURL("/device_management").spec()); 100 embedded_test_server()->GetURL("/device_management").spec());
98 } 101 }
(...skipping 235 matching lines...) Expand 10 before | Expand all | Expand 10 after
334 { 3, kUsernameOtherDomain, true }, 337 { 3, kUsernameOtherDomain, true },
335 { 4, kUsernameOtherDomain, true }, 338 { 4, kUsernameOtherDomain, true },
336 { 5, kUsernameOtherDomain, true }, 339 { 5, kUsernameOtherDomain, true },
337 }; 340 };
338 341
339 INSTANTIATE_TEST_CASE_P(BlockingLoginTestInstance, 342 INSTANTIATE_TEST_CASE_P(BlockingLoginTestInstance,
340 BlockingLoginTest, 343 BlockingLoginTest,
341 testing::ValuesIn(kBlockinLoginTestCases)); 344 testing::ValuesIn(kBlockinLoginTestCases));
342 345
343 } // namespace chromeos 346 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698