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

Side by Side Diff: chrome/test/startup/startup_test.cc

Issue 42306: Make startup_tests build and work on Mac. (Closed)
Patch Set: add app dependency Created 11 years, 9 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
« no previous file with comments | « chrome/common/chrome_constants.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 "base/file_path.h" 5 #include "base/file_path.h"
6 #include "base/file_util.h" 6 #include "base/file_util.h"
7 #include "base/path_service.h" 7 #include "base/path_service.h"
8 #include "base/platform_thread.h" 8 #include "base/platform_thread.h"
9 #include "base/string_util.h" 9 #include "base/string_util.h"
10 #include "base/test_file_util.h" 10 #include "base/test_file_util.h"
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 std::wstring file_url; 100 std::wstring file_url;
101 ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &file_url)); 101 ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &file_url));
102 file_util::AppendToPath(&file_url, L"empty.html"); 102 file_util::AppendToPath(&file_url, L"empty.html");
103 ASSERT_TRUE(file_util::PathExists(file_url)); 103 ASSERT_TRUE(file_util::PathExists(file_url));
104 launch_arguments_.AppendLooseValue(file_url); 104 launch_arguments_.AppendLooseValue(file_url);
105 105
106 pages_ = WideToUTF8(file_url); 106 pages_ = WideToUTF8(file_url);
107 } 107 }
108 }; 108 };
109 109
110 } // namespace
111
112 TEST_F(StartupTest, Perf) { 110 TEST_F(StartupTest, Perf) {
113 RunStartupTest("warm", "t", false /* not cold */, true /* important */); 111 RunStartupTest("warm", "t", false /* not cold */, true /* important */);
114 } 112 }
115 113
116 #if defined(OS_WIN) 114 #if defined(OS_WIN)
117 // TODO(port): Enable reference tests on other platforms. 115 // TODO(port): Enable reference tests on other platforms.
118 116
119 TEST_F(StartupReferenceTest, Perf) { 117 TEST_F(StartupReferenceTest, Perf) {
120 RunStartupTest("warm", "t_ref", false /* not cold */, 118 RunStartupTest("warm", "t_ref", false /* not cold */,
121 true /* important */); 119 true /* important */);
122 } 120 }
123 121
124 // TODO(mpcomplete): Should we have reference timings for all these? 122 // TODO(mpcomplete): Should we have reference timings for all these?
125 123
126 TEST_F(StartupTest, PerfCold) { 124 TEST_F(StartupTest, PerfCold) {
127 RunStartupTest("cold", "t", true /* cold */, false /* not important */); 125 RunStartupTest("cold", "t", true /* cold */, false /* not important */);
128 } 126 }
129 127
130 TEST_F(StartupFileTest, PerfGears) { 128 TEST_F(StartupFileTest, PerfGears) {
131 RunStartupTest("warm", "gears", false /* not cold */, 129 RunStartupTest("warm", "gears", false /* not cold */,
132 false /* not important */); 130 false /* not important */);
133 } 131 }
134 132
135 TEST_F(StartupFileTest, PerfColdGears) { 133 TEST_F(StartupFileTest, PerfColdGears) {
136 RunStartupTest("cold", "gears", true /* cold */, 134 RunStartupTest("cold", "gears", true /* cold */,
137 false /* not important */); 135 false /* not important */);
138 } 136 }
139 137
140 #endif // defined(OS_WIN) 138 #endif // defined(OS_WIN)
139
140 } // namespace
OLDNEW
« no previous file with comments | « chrome/common/chrome_constants.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698