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

Unified Diff: tools/perf/core/path_util.py

Issue 1568103002: Update tools/perf to use cloud_storage from catapult. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Erik comment Created 4 years, 11 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: tools/perf/core/path_util.py
diff --git a/tools/perf/core/path_util.py b/tools/perf/core/path_util.py
index b3aee1e77a91ba0f7b06c5d6348616498d0606a2..8b7f9ee5cdca4e746c9153b791f850c7a82a7c33 100644
--- a/tools/perf/core/path_util.py
+++ b/tools/perf/core/path_util.py
@@ -3,21 +3,42 @@
# found in the LICENSE file.
import os
+import sys
def GetChromiumSrcDir():
return os.path.dirname(os.path.dirname(os.path.dirname(os.path.dirname(
os.path.abspath(__file__)))))
+
+def GetCatapultBaseDir():
+ return os.path.join(
+ GetChromiumSrcDir(), 'third_party', 'catapult', 'catapult_base')
+
+
def GetTelemetryDir():
return os.path.join(GetChromiumSrcDir(), 'tools', 'telemetry')
+
def GetPerfDir():
return os.path.join(GetChromiumSrcDir(), 'tools', 'perf')
+
def GetPerfStorySetsDir():
return os.path.join(GetPerfDir(), 'page_sets')
+
def GetPerfBenchmarksDir():
return os.path.join(GetPerfDir(), 'benchmarks')
+
+def AddTelemetryToPath():
+ telemetry_path = GetTelemetryDir()
+ if telemetry_path not in sys.path:
+ sys.path.insert(1, telemetry_path)
+
+
+def AddCatapultBaseToPath():
+ catapult_base_path = GetCatapultBaseDir()
+ if catapult_base_path not in sys.path:
+ sys.path.insert(1, catapult_base_path)
eakuefner 2016/01/08 19:54:54 If you do client_config in this CL, you can add he
aiolos (Not reviewing) 2016/01/08 20:25:27 Do we actually need that? That path gets passed in
eakuefner 2016/01/08 20:31:22 Oh, I see. I didn't realize that ChromiumConfig wo

Powered by Google App Engine
This is Rietveld 408576698