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

Unified Diff: base/test/perf_test_suite.cc

Issue 2358063002: Preparing components_perftests (Closed)
Patch Set: Attempt to guess how generate_perf_json.py works Created 4 years, 3 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
diff --git a/base/test/perf_test_suite.cc b/base/test/perf_test_suite.cc
index 58cc4f9165da97667da5a6edf55ab0252f45eb1b..bb8512c21a7801ac5003e0a6e406381abfcbd1d6 100644
--- a/base/test/perf_test_suite.cc
+++ b/base/test/perf_test_suite.cc
@@ -4,14 +4,10 @@
#include "base/test/perf_test_suite.h"
-#include "base/command_line.h"
#include "base/debug/debugger.h"
-#include "base/files/file_path.h"
-#include "base/path_service.h"
#include "base/process/launch.h"
#include "base/strings/string_util.h"
#include "base/test/perf_log.h"
-#include "build/build_config.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace base {
@@ -22,19 +18,7 @@ void PerfTestSuite::Initialize() {
TestSuite::Initialize();
// Initialize the perf timer log
- FilePath log_path =
- CommandLine::ForCurrentProcess()->GetSwitchValuePath("log-file");
- if (log_path.empty()) {
- PathService::Get(FILE_EXE, &log_path);
-#if defined(OS_ANDROID)
- base::FilePath tmp_dir;
- PathService::Get(base::DIR_CACHE, &tmp_dir);
- log_path = tmp_dir.Append(log_path.BaseName());
-#endif
- log_path = log_path.ReplaceExtension(FILE_PATH_LITERAL("log"));
- log_path = log_path.InsertBeforeExtension(FILE_PATH_LITERAL("_perf"));
- }
- ASSERT_TRUE(InitPerfLog(log_path));
+ ASSERT_TRUE(InitPerfLog());
// 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.

Powered by Google App Engine
This is Rietveld 408576698