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

Unified Diff: chrome/test/startup/startup_test.cc

Issue 3129003: remove toolstrips (Closed)
Patch Set: merge Created 10 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/test/startup/startup_test.cc
diff --git a/chrome/test/startup/startup_test.cc b/chrome/test/startup/startup_test.cc
index e33e11335b5ec0872cb4d796079568ccb258a95b..1422e4961a3e2a0b071da9653126a3108aecdc47 100644
--- a/chrome/test/startup/startup_test.cc
+++ b/chrome/test/startup/startup_test.cc
@@ -79,9 +79,6 @@ class StartupTest : public UITest {
data_dir = data_dir.AppendASCII("extensions").AppendASCII("profiles").
AppendASCII(profile);
set_template_user_data(data_dir);
-
- // For now, these tests still depend on toolstrips.
- launch_arguments_.AppendSwitch(switches::kEnableExtensionToolstrips);
}
// Runs a test which loads |tab_count| tabs on startup, either as command line
@@ -343,20 +340,6 @@ TEST_F(StartupTest, PerfExtensionEmpty) {
UITest::DEFAULT_THEME, 1, 0);
}
-TEST_F(StartupTest, PerfExtensionToolstrips1) {
- SetUpWithFileURL();
- SetUpWithExtensionsProfile("toolstrips1");
- RunStartupTest("warm", "extension_toolstrip1", WARM, NOT_IMPORTANT,
- UITest::DEFAULT_THEME, 1, 0);
-}
-
-TEST_F(StartupTest, PerfExtensionToolstrips50) {
- SetUpWithFileURL();
- SetUpWithExtensionsProfile("toolstrips50");
- RunStartupTest("warm", "extension_toolstrip50", WARM, NOT_IMPORTANT,
- UITest::DEFAULT_THEME, 1, 0);
-}
-
TEST_F(StartupTest, PerfExtensionContentScript1) {
SetUpWithFileURL();
SetUpWithExtensionsProfile("content_scripts1");
« no previous file with comments | « chrome/test/data/extensions/profiles/toolstrips50/Default/Preferences ('k') | chrome/test/test_browser_window.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698