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

Side by Side Diff: chrome/browser/chromeos/arc/arc_session_manager_browsertest.cc

Issue 2957983002: Remove pointless InProcessBrowserTest calls. (Closed)
Patch Set: build Created 3 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 <memory> 5 #include <memory>
6 #include <string> 6 #include <string>
7 7
8 #include "base/auto_reset.h" 8 #include "base/auto_reset.h"
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/files/file_path.h" 10 #include "base/files/file_path.h"
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
97 }; 97 };
98 98
99 class ArcSessionManagerTest : public InProcessBrowserTest { 99 class ArcSessionManagerTest : public InProcessBrowserTest {
100 protected: 100 protected:
101 ArcSessionManagerTest() {} 101 ArcSessionManagerTest() {}
102 102
103 // InProcessBrowserTest: 103 // InProcessBrowserTest:
104 ~ArcSessionManagerTest() override {} 104 ~ArcSessionManagerTest() override {}
105 105
106 void SetUpCommandLine(base::CommandLine* command_line) override { 106 void SetUpCommandLine(base::CommandLine* command_line) override {
107 InProcessBrowserTest::SetUpCommandLine(command_line);
108 arc::SetArcAvailableCommandLineForTesting(command_line); 107 arc::SetArcAvailableCommandLineForTesting(command_line);
109 } 108 }
110 109
111 void SetUpInProcessBrowserTestFixture() override { 110 void SetUpInProcessBrowserTestFixture() override {
112 // Start test device management server. 111 // Start test device management server.
113 test_server_.reset(new policy::LocalPolicyTestServer()); 112 test_server_.reset(new policy::LocalPolicyTestServer());
114 ASSERT_TRUE(test_server_->Start()); 113 ASSERT_TRUE(test_server_->Start());
115 114
116 // Specify device management server URL. 115 // Specify device management server URL.
117 std::string url = test_server_->GetServiceURL().spec(); 116 std::string url = test_server_->GetServiceURL().spec();
(...skipping 121 matching lines...) Expand 10 before | Expand all | Expand 10 after
239 238
240 IN_PROC_BROWSER_TEST_F(ArcSessionManagerTest, ManagedAndroidAccount) { 239 IN_PROC_BROWSER_TEST_F(ArcSessionManagerTest, ManagedAndroidAccount) {
241 EnableArc(); 240 EnableArc();
242 token_service()->IssueTokenForAllPendingRequests(kManagedAuthToken, 241 token_service()->IssueTokenForAllPendingRequests(kManagedAuthToken,
243 base::Time::Max()); 242 base::Time::Max());
244 ArcPlayStoreDisabledWaiter().Wait(); 243 ArcPlayStoreDisabledWaiter().Wait();
245 EXPECT_FALSE(IsArcPlayStoreEnabledForProfile(profile())); 244 EXPECT_FALSE(IsArcPlayStoreEnabledForProfile(profile()));
246 } 245 }
247 246
248 } // namespace arc 247 } // namespace arc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698