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

Unified Diff: base/test/perf_test_suite.h

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.h
===================================================================
--- base/test/perf_test_suite.h (revision 55077)
+++ base/test/perf_test_suite.h (working copy)
@@ -1,4 +1,4 @@
-// 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.
@@ -6,44 +6,22 @@
#define BASE_TEST_PERF_TEST_SUITE_H_
#pragma once
-#include "base/command_line.h"
-#include "base/debug_util.h"
-#include "base/file_path.h"
-#include "base/perftimer.h"
-#include "base/process_util.h"
-#include "base/string_util.h"
+#include "base/basictypes.h"
#include "base/test/test_suite.h"
+namespace base {
+
class PerfTestSuite : public TestSuite {
public:
- PerfTestSuite(int argc, char** argv) : TestSuite(argc, argv) {
- }
+ PerfTestSuite(int argc, char** argv);
- virtual void Initialize() {
- TestSuite::Initialize();
+ virtual void Initialize();
+ virtual void Shutdown();
- // 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));
-
- // 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();
- }
-
- virtual void Shutdown() {
- TestSuite::Shutdown();
-
- FinalizePerfLog();
- }
+ private:
+ DISALLOW_COPY_AND_ASSIGN(PerfTestSuite);
};
+} // namespace base
+
#endif // BASE_TEST_PERF_TEST_SUITE_H_

Powered by Google App Engine
This is Rietveld 408576698