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

Unified Diff: base/test/perf_test_suite.cc

Issue 3026055: Cleanup in base. This moves the implementation (and a bunch of header file... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: 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: base/test/perf_test_suite.cc
===================================================================
--- base/test/perf_test_suite.cc (revision 55077)
+++ base/test/perf_test_suite.cc (working copy)
@@ -1,49 +1,46 @@
-// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef BASE_TEST_PERF_TEST_SUITE_H_
-#define BASE_TEST_PERF_TEST_SUITE_H_
-#pragma once
+#include "base/test/perf_test_suite.h"
#include "base/command_line.h"
#include "base/debug_util.h"
#include "base/file_path.h"
+#include "base/path_service.h"
#include "base/perftimer.h"
#include "base/process_util.h"
-#include "base/string_util.h"
#include "base/test/test_suite.h"
+#include "testing/gtest/include/gtest/gtest.h"
-class PerfTestSuite : public TestSuite {
- public:
- PerfTestSuite(int argc, char** argv) : TestSuite(argc, argv) {
- }
+namespace base {
- virtual void Initialize() {
- TestSuite::Initialize();
+PerfTestSuite::PerfTestSuite(int argc, char** argv) : TestSuite(argc, argv) {
+}
- // Initialize the perf timer log
- FilePath log_path =
- CommandLine::ForCurrentProcess()->GetSwitchValuePath("log-file");
- if (log_path.empty()) {
- FilePath exe;
- PathService::Get(base::FILE_EXE, &exe);
- log_path = exe.ReplaceExtension(FILE_PATH_LITERAL("log"));
- log_path = log_path.InsertBeforeExtension(FILE_PATH_LITERAL("_perf"));
- }
- ASSERT_TRUE(InitPerfLog(log_path));
+void PerfTestSuite::Initialize() {
+ TestSuite::Initialize();
- // Raise to high priority to have more precise measurements. Since we don't
- // aim at 1% precision, it is not necessary to run at realtime level.
- if (!DebugUtil::BeingDebugged())
- base::RaiseProcessToHighPriority();
+ // Initialize the perf timer log
+ FilePath log_path =
+ CommandLine::ForCurrentProcess()->GetSwitchValuePath("log-file");
+ if (log_path.empty()) {
+ FilePath exe;
+ PathService::Get(FILE_EXE, &exe);
+ log_path = exe.ReplaceExtension(FILE_PATH_LITERAL("log"));
+ log_path = log_path.InsertBeforeExtension(FILE_PATH_LITERAL("_perf"));
}
+ ASSERT_TRUE(InitPerfLog(log_path));
- virtual void Shutdown() {
- TestSuite::Shutdown();
+ // Raise to high priority to have more precise measurements. Since we don't
+ // aim at 1% precision, it is not necessary to run at realtime level.
+ if (!DebugUtil::BeingDebugged())
+ RaiseProcessToHighPriority();
+}
- FinalizePerfLog();
- }
-};
+void PerfTestSuite::Shutdown() {
+ TestSuite::Shutdown();
+ FinalizePerfLog();
+}
-#endif // BASE_TEST_PERF_TEST_SUITE_H_
+} // namespace base

Powered by Google App Engine
This is Rietveld 408576698