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

Issue 9320001: Added a new pyauto-based flash video performance test to perf.py. (Closed)

Created:
8 years, 10 months ago by dennis_jeffrey
Modified:
8 years, 10 months ago
Reviewers:
Nirnimesh
CC:
chromium-reviews, dennis_jeffrey, John Grabowski, anantha, dyu1, Nirnimesh, trond, arscott
Visibility:
Public.

Description

Added a new pyauto-based flash video performance test to perf.py. This test must be run manually for now, and will be used in its current state only temporarily to compare results across different ChromeOS devices. BUG=None TEST=Verified the test runs successfully on a local Alex device.

Patch Set 1 #

Total comments: 2

Patch Set 2 : Addressed review comment. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -0 lines) Patch
M chrome/test/functional/perf.py View 1 1 chunk +43 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
dennis_jeffrey
Nirnimesh, arscott@ had written a flash test for 1080p video, which we want to use ...
8 years, 10 months ago (2012-02-01 02:00:53 UTC) #1
Nirnimesh
http://codereview.chromium.org/9320001/diff/1/chrome/test/functional/perf.py File chrome/test/functional/perf.py (right): http://codereview.chromium.org/9320001/diff/1/chrome/test/functional/perf.py#newcode924 chrome/test/functional/perf.py:924: webpage_url = 'http://www.corp.google.com/~arscott/fl/FlashVideoTests.html' It's not a good idea to ...
8 years, 10 months ago (2012-02-01 02:04:01 UTC) #2
dennis_jeffrey
http://codereview.chromium.org/9320001/diff/1/chrome/test/functional/perf.py File chrome/test/functional/perf.py (right): http://codereview.chromium.org/9320001/diff/1/chrome/test/functional/perf.py#newcode924 chrome/test/functional/perf.py:924: webpage_url = 'http://www.corp.google.com/~arscott/fl/FlashVideoTests.html' On 2012/02/01 02:04:01, Nirnimesh wrote: > ...
8 years, 10 months ago (2012-02-01 02:13:31 UTC) #3
Nirnimesh
8 years, 10 months ago (2012-02-01 02:14:31 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698