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

Unified Diff: ct/go/worker_scripts/run_chromium_perf_swarming/main.go

Issue 1870173002: [CT] Add desktop device when running run_benchmark (Closed) Base URL: https://skia.googlesource.com/buildbot@master
Patch Set: Created 4 years, 8 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
« no previous file with comments | « ct/go/worker_scripts/run_chromium_perf/main.go ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ct/go/worker_scripts/run_chromium_perf_swarming/main.go
diff --git a/ct/go/worker_scripts/run_chromium_perf_swarming/main.go b/ct/go/worker_scripts/run_chromium_perf_swarming/main.go
index 3eaaa8fc4a928db2e2dc33a6d11971d4b5494803..4367a9e83de5bedc818ca5028072c42ab9e1cc0f 100644
--- a/ct/go/worker_scripts/run_chromium_perf_swarming/main.go
+++ b/ct/go/worker_scripts/run_chromium_perf_swarming/main.go
@@ -143,7 +143,7 @@ func runBenchmark(fileInfoName, pathToPagesets, localOutputDir, chromiumBuildNam
// Need to capture output for all benchmarks.
outputDirArgValue := filepath.Join(localOutputDir, pagesetName)
args = append(args, "--output-dir="+outputDirArgValue)
- // Figure out which browser should be used.
+ // Figure out which browser and device should be used.
if *targetPlatform == util.PLATFORM_ANDROID {
if err := util.InstallChromeAPK(chromiumBuildName); err != nil {
return fmt.Errorf("Error while installing APK: %s", err)
@@ -151,6 +151,7 @@ func runBenchmark(fileInfoName, pathToPagesets, localOutputDir, chromiumBuildNam
args = append(args, "--browser=android-chromium")
} else {
args = append(args, "--browser=exact", "--browser-executable="+chromiumBinary)
+ args = append(args, "--device=desktop")
}
// Split benchmark args if not empty and append to args.
if *benchmarkExtraArgs != "" {
« no previous file with comments | « ct/go/worker_scripts/run_chromium_perf/main.go ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698