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

Issue 7068005: Revert 86568 - Add initial framework for performance tests that measure frame rate. (Closed)

Created:
9 years, 7 months ago by ukai
Modified:
9 years, 7 months ago
CC:
chromium-reviews, Paweł Hajdan Jr.
Visibility:
Public.

Description

Revert 86568 - Add initial framework for performance tests that measure frame rate. Add the "blank page" test, which just scrolls a page with a very large body. The actual frame rate measurement is based on the webkitRequestAnimationFrame callback. The framework records how frequently that API generates a callback. If it generates a callback at 60 FPS, then we know nothing is slowing us down. This of course assumes that webkitRequestAnimationFrame is well implemented. The advantage of this test framework over one that directly reads a framerate from Chromium is that this one works outside of the test harness. You can just navigate your browser to the test file and run the test manually. R=brettw,nduca Review URL: http://codereview.chromium.org/7038052 TBR=darin@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=86572

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -280 lines) Patch
M chrome/chrome_tests.gypi View 1 chunk +0 lines, -44 lines 0 comments Download
D chrome/test/data/perf/frame_rate/blank/test.html View 1 chunk +0 lines, -4 lines 0 comments Download
D chrome/test/data/perf/frame_rate/head.js View 1 chunk +0 lines, -100 lines 0 comments Download
D chrome/test/perf/frame_rate/frame_rate_tests.cc View 1 chunk +0 lines, -110 lines 0 comments Download
M chrome/test/ui/javascript_test_util.cc View 2 chunks +9 lines, -21 lines 0 comments Download
M chrome/test/ui/ui_perf_test.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
ukai
9 years, 7 months ago (2011-05-25 05:25:30 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698