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

Issue 2485213002: Updating src side json for 4 core 4GB and full suite on 2 core 4 GB (Closed)

Created:
4 years, 1 month ago by eyaich1
Modified:
4 years, 1 month ago
Reviewers:
dtu
CC:
chromium-reviews, telemetry-reviews_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Updating src side json for 4 core 4GB and full suite on 2 core 4 GB BUG=chromium:663370 Committed: https://crrev.com/2fa7a0c729da95dc10e58714477036684d5044da Cr-Commit-Position: refs/heads/master@{#430952}

Patch Set 1 #

Patch Set 2 : Adding in 2 core changes #

Patch Set 3 : Full suite of benchmarks on 2 core #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+10491 lines, -43 lines) Patch
M testing/buildbot/chromium.perf.fyi.json View 1 2 11 chunks +10439 lines, -30 lines 0 comments Download
M tools/perf/generate_perf_json.py View 1 2 7 chunks +52 lines, -13 lines 1 comment Download

Messages

Total messages: 12 (6 generated)
eyaich1
4 years, 1 month ago (2016-11-08 18:53:33 UTC) #2
eyaich1
4 years, 1 month ago (2016-11-08 18:53:34 UTC) #4
dtu
lgtm https://codereview.chromium.org/2485213002/diff/40001/tools/perf/generate_perf_json.py File tools/perf/generate_perf_json.py (right): https://codereview.chromium.org/2485213002/diff/40001/tools/perf/generate_perf_json.py#newcode464 tools/perf/generate_perf_json.py:464: #num_shards = len(dimension['device_ids']) leftover code?
4 years, 1 month ago (2016-11-09 02:35:18 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2485213002/40001
4 years, 1 month ago (2016-11-09 15:46:08 UTC) #8
commit-bot: I haz the power
Committed patchset #3 (id:40001)
4 years, 1 month ago (2016-11-09 16:55:16 UTC) #10
commit-bot: I haz the power
4 years, 1 month ago (2016-11-09 17:04:18 UTC) #12
Message was sent while issue was closed.
Patchset 3 (id:??) landed as
https://crrev.com/2fa7a0c729da95dc10e58714477036684d5044da
Cr-Commit-Position: refs/heads/master@{#430952}

Powered by Google App Engine
This is Rietveld 408576698