Added reduced WebGL case from Animometer to tough_webgl_cases.

Added reduced WebGL case from Animometer to tough_webgl_cases.

Updated links to a few of the earlier test cases.

BUG=608576
Committed: https://crrev.com/c293972b740368327c4c6e7f16fc749c451a2998
Cr-Commit-Position: refs/heads/master@{#393115}

[Ken Russell (switch to Gerrit), 2016-05-11 18:46:23]

Ned, Kari, Ethan: please review.

I re-recorded this page set locally a couple of times. Let me know if I should
manually delete those intermediate recordings from tough_webgl_cases.json.

I also re-recorded this with the --upload option to record_wpr but am not sure
how to verify that. "gsutil ls" in the public telemetry bucket contains a lot of
hashes but nothing obviously related to tough_webgl_cases.

Thanks.

[Ken Russell (switch to Gerrit), 2016-05-11 18:47:03]

kbr@chromium.org changed reviewers:
+ aiolos@chromium.org, eakuefner@chromium.org, nednguyen@google.com

[Ken Russell (switch to Gerrit), 2016-05-11 18:47:03]

(Lost the reviewers when I accidentally closed and re-opened the tab -- please
see the comment above. Thanks)

[Ken Russell (switch to Gerrit), 2016-05-11 18:48:09]

Description was changed from

==========
Added reduced WebGL case from Animometer to tough_webgl_cases.

BUG=608576
==========

to

==========
Added reduced WebGL case from Animometer to tough_webgl_cases.

Updated links to a few of the earlier test cases.

BUG=608576
==========

[nednguyen, 2016-05-11 18:53:11]

nednguyen@google.com changed reviewers:
- eakuefner@chromium.org, nednguyen@google.com

[aiolos (Not reviewing), 2016-05-11 20:51:18]

On 2016/05/11 18:46:23, Ken Russell wrote:
> Ned, Kari, Ethan: please review.
> 
> I re-recorded this page set locally a couple of times. Let me know if I should
> manually delete those intermediate recordings from tough_webgl_cases.json.
> 
> I also re-recorded this with the --upload option to record_wpr but am not sure
> how to verify that. "gsutil ls" in the public telemetry bucket contains a lot
of
> hashes but nothing obviously related to tough_webgl_cases.
> 

The hashes match those stored in the sha1 files. You can run `gsutil ls
gs://<bucket>/<hash>` for the hash in the sha1 files to check whether the file
successfully uploaded. There is a catapult script that does that automatically
for all the files in a dependency manager config, but I'm not sure if someone
has done the same for the page set sha1 files. Ned would probably know.

Also, you will want to include the sha1 file you generated. It should be
tools/perf/page_sets/data/tough_webgl_cases_004.wpr.sha1

> Thanks.

[Ken Russell (switch to Gerrit), 2016-05-11 21:55:39]

On 2016/05/11 20:51:18, aiolos(slow reviews) wrote:
> On 2016/05/11 18:46:23, Ken Russell wrote:
> > Ned, Kari, Ethan: please review.
> > 
> > I re-recorded this page set locally a couple of times. Let me know if I
should
> > manually delete those intermediate recordings from tough_webgl_cases.json.
> > 
> > I also re-recorded this with the --upload option to record_wpr but am not
sure
> > how to verify that. "gsutil ls" in the public telemetry bucket contains a
lot
> of
> > hashes but nothing obviously related to tough_webgl_cases.
> > 
> 
> The hashes match those stored in the sha1 files. You can run `gsutil ls
> gs://<bucket>/<hash>` for the hash in the sha1 files to check whether the file
> successfully uploaded. There is a catapult script that does that automatically
> for all the files in a dependency manager config, but I'm not sure if someone
> has done the same for the page set sha1 files. Ned would probably know.
> 
> Also, you will want to include the sha1 file you generated. It should be
> tools/perf/page_sets/data/tough_webgl_cases_004.wpr.sha1

Thanks. Revised, and verified that the new SHA1 is in the public bucket.

[Ken Russell (switch to Gerrit), 2016-05-11 21:55:52]

+vmiura as FYI.

[aiolos (Not reviewing), 2016-05-11 22:03:04]

lgtm

https://codereview.chromium.org/1967213002/diff/20001/tools/perf/page_sets/da...
File tools/perf/page_sets/data/tough_webgl_cases.json (right):

https://codereview.chromium.org/1967213002/diff/20001/tools/perf/page_sets/da...
tools/perf/page_sets/data/tough_webgl_cases.json:16:
"http://montagestudio.com/demos/eco-homes/",
Did you get an error message for why these pages failed to re-record? The sites
still seems to be up and running. You should probably file a bug on wpr for
those urls.

[Ken Russell (switch to Gerrit), 2016-05-11 22:11:50]

https://codereview.chromium.org/1967213002/diff/20001/tools/perf/page_sets/da...
File tools/perf/page_sets/data/tough_webgl_cases.json (right):

https://codereview.chromium.org/1967213002/diff/20001/tools/perf/page_sets/da...
tools/perf/page_sets/data/tough_webgl_cases.json:16:
"http://montagestudio.com/demos/eco-homes/",
On 2016/05/11 22:03:03, aiolos(slow reviews) wrote:
> Did you get an error message for why these pages failed to re-record? The
sites
> still seems to be up and running. You should probably file a bug on wpr for
> those urls.

These two were removed from the page set a while back because they weren't
producing stable numbers. I assume that this is why the JSON file doesn't
contain them any more. Please correct me if I'm misunderstanding.

[Ken Russell (switch to Gerrit), 2016-05-11 22:11:53]

The CQ bit was checked by kbr@chromium.org

[commit-bot: I haz the power, 2016-05-11 22:12:14]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1967213002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1967213002/20001

[commit-bot: I haz the power, 2016-05-12 00:19:19]

Description was changed from

==========
Added reduced WebGL case from Animometer to tough_webgl_cases.

Updated links to a few of the earlier test cases.

BUG=608576
==========

to

==========
Added reduced WebGL case from Animometer to tough_webgl_cases.

Updated links to a few of the earlier test cases.

BUG=608576
==========

[commit-bot: I haz the power, 2016-05-12 00:19:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-12 00:21:58]

Description was changed from

==========
Added reduced WebGL case from Animometer to tough_webgl_cases.

Updated links to a few of the earlier test cases.

BUG=608576
==========

to

==========
Added reduced WebGL case from Animometer to tough_webgl_cases.

Updated links to a few of the earlier test cases.

BUG=608576

Committed: https://crrev.com/c293972b740368327c4c6e7f16fc749c451a2998
Cr-Commit-Position: refs/heads/master@{#393115}
==========

[commit-bot: I haz the power, 2016-05-12 00:21:59]

Patchset 2 (id:??) landed as
https://crrev.com/c293972b740368327c4c6e7f16fc749c451a2998
Cr-Commit-Position: refs/heads/master@{#393115}

[vmiura, 2016-05-16 21:28:28]

Thanks for adding this test.

It looks to me like with your optimization, the frame_time may now be clipped to
vsync 16.67ms.  May want to up the complexity, so that we can notice any minor
regressions or improvements.