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

Issue 578193002: telemetry: re-enable repaint.gpu_rasterization.key_mobile_sites (Closed)

Created:
6 years, 3 months ago by ernstm
Modified:
6 years, 2 months ago
Reviewers:
tonyg
CC:
chromium-reviews, telemetry+watch_chromium.org, bsalomon
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

telemetry: re-enable repaint.gpu_rasterization.key_mobile_sites R=tonyg@chromium.org BUG=412001 Committed: https://crrev.com/a45c079d65481b2c17447ebfc989f8049ece30df Cr-Commit-Position: refs/heads/master@{#298377}

Patch Set 1 #

Total comments: 2

Patch Set 2 : enable on Android only #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M tools/perf/benchmarks/repaint.py View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 20 (7 generated)
ernstm
PTAL
6 years, 3 months ago (2014-09-18 05:18:54 UTC) #1
ernstm
On 2014/09/18 05:18:54, ernstm wrote: > PTAL ping
6 years, 3 months ago (2014-09-22 23:42:48 UTC) #2
tonyg
https://codereview.chromium.org/578193002/diff/1/tools/perf/benchmarks/repaint.py File tools/perf/benchmarks/repaint.py (right): https://codereview.chromium.org/578193002/diff/1/tools/perf/benchmarks/repaint.py#newcode19 tools/perf/benchmarks/repaint.py:19: class RepaintGpuRasterizationKeyMobileSites(benchmark.Benchmark): This'll run on desktop now too - ...
6 years, 3 months ago (2014-09-23 00:11:59 UTC) #3
ernstm
https://codereview.chromium.org/578193002/diff/1/tools/perf/benchmarks/repaint.py File tools/perf/benchmarks/repaint.py (right): https://codereview.chromium.org/578193002/diff/1/tools/perf/benchmarks/repaint.py#newcode19 tools/perf/benchmarks/repaint.py:19: class RepaintGpuRasterizationKeyMobileSites(benchmark.Benchmark): On 2014/09/23 00:11:58, tonyg wrote: > This'll ...
6 years, 2 months ago (2014-10-06 17:38:20 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/578193002/20001
6 years, 2 months ago (2014-10-06 17:40:05 UTC) #6
commit-bot: I haz the power
Try jobs failed on following builders: linux_gpu_triggered_tests on tryserver.chromium.gpu (http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu_triggered_tests/builds/65069)
6 years, 2 months ago (2014-10-06 20:42:46 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/578193002/20001
6 years, 2 months ago (2014-10-06 20:50:43 UTC) #10
commit-bot: I haz the power
Try jobs failed on following builders: linux_gpu_triggered_tests on tryserver.chromium.gpu (http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu_triggered_tests/builds/65069)
6 years, 2 months ago (2014-10-06 21:58:07 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/578193002/20001
6 years, 2 months ago (2014-10-06 23:59:14 UTC) #14
commit-bot: I haz the power
Try jobs failed on following builders: mac_gpu_triggered_tests on tryserver.chromium.gpu (http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu_triggered_tests/builds/55104)
6 years, 2 months ago (2014-10-07 01:13:03 UTC) #16
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/578193002/20001
6 years, 2 months ago (2014-10-07 04:54:42 UTC) #18
commit-bot: I haz the power
Committed patchset #2 (id:20001) as d8c1b2f2198a2a38343a18a27aff4b69d7373071
6 years, 2 months ago (2014-10-07 04:56:58 UTC) #19
commit-bot: I haz the power
6 years, 2 months ago (2014-10-07 04:58:14 UTC) #20
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/a45c079d65481b2c17447ebfc989f8049ece30df
Cr-Commit-Position: refs/heads/master@{#298377}

Powered by Google App Engine
This is Rietveld 408576698