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

Issue 9580021: Add test for RAF-spinning with no damage (Closed)

Created:
8 years, 9 months ago by jbates
Modified:
8 years, 9 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Add test to for RAF-spinning in accelerated mode with no damage. R=jamesr,zmo BUG=114914 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=125891

Patch Set 1 #

Patch Set 2 : adjust time to hopefully make linux fail #

Patch Set 3 : added trim feature to TraceAnalyzer to help with flakiness #

Patch Set 4 : just merge, no changes #

Patch Set 5 : test with TimeDelta render_widget #

Patch Set 6 : testing with trace output #

Patch Set 7 : testing AnimateIfNeeded task leak fix #

Patch Set 8 : update / merge #

Patch Set 9 : removed animation_task_posted_ bool #

Patch Set 10 : removed render_widget changes #

Patch Set 11 : marking test flaky for initial commit to be safe #

Unified diffs Side-by-side diffs Delta from patch set Stats (+155 lines, -21 lines) Patch
M base/test/trace_event_analyzer.h View 1 2 1 chunk +13 lines, -1 line 0 comments Download
M base/test/trace_event_analyzer.cc View 1 2 2 chunks +21 lines, -4 lines 0 comments Download
M base/test/trace_event_analyzer_unittest.cc View 1 2 1 chunk +37 lines, -4 lines 0 comments Download
A chrome/test/data/gpu/feature_raf_no_damage.html View 1 2 3 4 5 6 7 8 9 10 1 chunk +37 lines, -0 lines 0 comments Download
M chrome/test/gpu/gpu_feature_browsertest.cc View 1 2 3 4 5 6 7 8 9 10 4 chunks +45 lines, -10 lines 0 comments Download
M chrome/test/perf/rendering/latency_tests.cc View 1 2 3 4 5 6 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/perf/rendering/throughput_tests.cc View 1 2 3 4 5 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
jbates
If this is working, all the try jobs should fail!
8 years, 9 months ago (2012-03-02 20:51:16 UTC) #1
jbates
On 2012/03/02 20:51:16, jbates wrote: > If this is working, all the try jobs should ...
8 years, 9 months ago (2012-03-02 20:51:49 UTC) #2
Zhenyao Mo
LGTM
8 years, 9 months ago (2012-03-02 22:01:13 UTC) #3
jbates
phajdan: base/test OWNERS
8 years, 9 months ago (2012-03-05 18:08:35 UTC) #4
Paweł Hajdan Jr.
LGTM
8 years, 9 months ago (2012-03-06 20:25:56 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jbates@chromium.org/9580021/37001
8 years, 9 months ago (2012-03-09 18:09:36 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jbates@chromium.org/9580021/43003
8 years, 9 months ago (2012-03-09 18:45:41 UTC) #7
commit-bot: I haz the power
8 years, 9 months ago (2012-03-09 20:43:14 UTC) #8
Change committed as 125891

Powered by Google App Engine
This is Rietveld 408576698