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

Side by Side Diff: chrome/browser/extensions/extension_startup_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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 <stddef.h> 5 #include <stddef.h>
6 #include <stdint.h> 6 #include <stdint.h>
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/base_switches.h" 10 #include "base/base_switches.h"
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 102
103 base::CopyFile(src_dir.Append(chrome::kPreferencesFilename), 103 base::CopyFile(src_dir.Append(chrome::kPreferencesFilename),
104 preferences_file_); 104 preferences_file_);
105 base::CopyDirectory(src_dir.AppendASCII("Extensions"), profile_dir, 105 base::CopyDirectory(src_dir.AppendASCII("Extensions"), profile_dir,
106 true); // recursive 106 true); // recursive
107 } 107 }
108 return true; 108 return true;
109 } 109 }
110 110
111 void SetUpInProcessBrowserTestFixture() override { 111 void SetUpInProcessBrowserTestFixture() override {
112 InProcessBrowserTest::SetUpInProcessBrowserTestFixture();
113
114 // Bots are on a domain, turn off the domain check for settings hardening in 112 // Bots are on a domain, turn off the domain check for settings hardening in
115 // order to be able to test all SettingsEnforcement groups. 113 // order to be able to test all SettingsEnforcement groups.
116 chrome_prefs::DisableDomainCheckForTesting(); 114 chrome_prefs::DisableDomainCheckForTesting();
117 } 115 }
118 116
119 void TearDown() override { 117 void TearDown() override {
120 EXPECT_TRUE(base::DeleteFile(preferences_file_, false)); 118 EXPECT_TRUE(base::DeleteFile(preferences_file_, false));
121 119
122 // TODO(phajdan.jr): Check return values of the functions below, carefully. 120 // TODO(phajdan.jr): Check return values of the functions below, carefully.
123 base::DeleteFile(user_scripts_dir_, true); 121 base::DeleteFile(user_scripts_dir_, true);
(...skipping 288 matching lines...) Expand 10 before | Expand all | Expand 10 after
412 } else if (extension->name() == "App Test") { 410 } else if (extension->name() == "App Test") {
413 is_app_test_extension_enabled = true; 411 is_app_test_extension_enabled = true;
414 } else { 412 } else {
415 EXPECT_TRUE( 413 EXPECT_TRUE(
416 extensions::Manifest::IsComponentLocation(extension->location())); 414 extensions::Manifest::IsComponentLocation(extension->location()));
417 } 415 }
418 } 416 }
419 EXPECT_TRUE(is_app_dot_com_extension_enabled); 417 EXPECT_TRUE(is_app_dot_com_extension_enabled);
420 EXPECT_TRUE(is_app_test_extension_enabled); 418 EXPECT_TRUE(is_app_test_extension_enabled);
421 } 419 }
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_browsertest.cc ('k') | chrome/browser/first_run/first_run_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698