Ported pixel_test to new gpu_integration_test harness.

Ported pixel_test to new gpu_integration_test harness.

Ported cloud_storage_test_base as cloud_storage_integration_test_base.

Ported pixel tests' page set into pixel_test_pages.py. Eliminated the
use of separate JSON files for manually-specified expected colors.

The browser is restarted between pixel test pages which require
different browser command line arguments. The new harness's test name
sorting increases the number of restarts, and therefore the execution
time, compared to the old harness. This will be fixed in a follow-on CL.

Deleted the old pixel.py harness to avoid confusion.

BUG=352807
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

Committed: https://crrev.com/def2386f8a09d45f74207b2b47a070c527cff3bf
Cr-Commit-Position: refs/heads/master@{#421951}

[Ken Russell (switch to Gerrit), 2016-09-26 00:38:16]

Description was changed from

==========
Ported pixel_test to new gpu_integration_test harness.

Ported cloud_storage_test_base as cloud_storage_integration_test_base.

Ported pixel tests' page set into pixel_test_pages.py. Eliminated the
use of separate JSON files for manually-specified expected colors.

The browser is restarted between pixel test pages which require
different browser command line arguments. The new harness's test name
sorting increases the number of restarts, and therefore the execution
time, compared to the old harness. This will be fixed in a follow-on CL.

BUG=352807
==========

to

==========
Ported pixel_test to new gpu_integration_test harness.

Ported cloud_storage_test_base as cloud_storage_integration_test_base.

Ported pixel tests' page set into pixel_test_pages.py. Eliminated the
use of separate JSON files for manually-specified expected colors.

The browser is restarted between pixel test pages which require
different browser command line arguments. The new harness's test name
sorting increases the number of restarts, and therefore the execution
time, compared to the old harness. This will be fixed in a follow-on CL.

BUG=352807
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

[Ken Russell (switch to Gerrit), 2016-09-29 04:39:56]

Description was changed from

==========
Ported pixel_test to new gpu_integration_test harness.

Ported cloud_storage_test_base as cloud_storage_integration_test_base.

Ported pixel tests' page set into pixel_test_pages.py. Eliminated the
use of separate JSON files for manually-specified expected colors.

The browser is restarted between pixel test pages which require
different browser command line arguments. The new harness's test name
sorting increases the number of restarts, and therefore the execution
time, compared to the old harness. This will be fixed in a follow-on CL.

BUG=352807
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

to

==========
Ported pixel_test to new gpu_integration_test harness.

Ported cloud_storage_test_base as cloud_storage_integration_test_base.

Ported pixel tests' page set into pixel_test_pages.py. Eliminated the
use of separate JSON files for manually-specified expected colors.

The browser is restarted between pixel test pages which require
different browser command line arguments. The new harness's test name
sorting increases the number of restarts, and therefore the execution
time, compared to the old harness. This will be fixed in a follow-on CL.

Deleted the old pixel.py harness to avoid confusion.

BUG=352807
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

[Ken Russell (switch to Gerrit), 2016-09-29 04:43:01]

kbr@chromium.org changed reviewers:
+ zmo@chromium.org

[Ken Russell (switch to Gerrit), 2016-09-29 04:43:02]

Mo: let's do an in-person review of this port tomorrow. It's basically
boilerplate. Until the trace_test is ported (after this, should hopefully be
soon, and easy), there are two copies of the pixel test pages, one in
page_sets/pixel_tests.py, and one in gpu_tests/pixel_test_pages.py.

Others: FYI. Xida, Olivia in particular: once this lands, any new pixel test
pages should be added to pixel_test_pages.py. The naming convention for
suppressions also changes slightly.

[Ken Russell (switch to Gerrit), 2016-09-29 04:45:09]

The CQ bit was checked by kbr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-29 04:45:24]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-29 05:27:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-29 05:27:08]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Ken Russell (switch to Gerrit), 2016-09-29 08:41:33]

The CQ bit was checked by kbr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-29 08:41:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-29 09:39:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-29 09:39:32]

Dry run: This issue passed the CQ dry run.

[Zhenyao Mo, 2016-09-29 22:09:21]

On 2016/09/29 04:43:02, Ken Russell wrote:
> Mo: let's do an in-person review of this port tomorrow. It's basically
> boilerplate. Until the trace_test is ported (after this, should hopefully be
> soon, and easy), there are two copies of the pixel test pages, one in
> page_sets/pixel_tests.py, and one in gpu_tests/pixel_test_pages.py.
> 
> Others: FYI. Xida, Olivia in particular: once this lands, any new pixel test
> pages should be added to pixel_test_pages.py. The naming convention for
> suppressions also changes slightly.

LGTM

[Ken Russell (switch to Gerrit), 2016-09-29 22:11:52]

The CQ bit was checked by kbr@chromium.org

[commit-bot: I haz the power, 2016-09-29 22:12:18]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-29 22:19:44]

Description was changed from

==========
Ported pixel_test to new gpu_integration_test harness.

Ported cloud_storage_test_base as cloud_storage_integration_test_base.

Ported pixel tests' page set into pixel_test_pages.py. Eliminated the
use of separate JSON files for manually-specified expected colors.

The browser is restarted between pixel test pages which require
different browser command line arguments. The new harness's test name
sorting increases the number of restarts, and therefore the execution
time, compared to the old harness. This will be fixed in a follow-on CL.

Deleted the old pixel.py harness to avoid confusion.

BUG=352807
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

to

==========
Ported pixel_test to new gpu_integration_test harness.

Ported cloud_storage_test_base as cloud_storage_integration_test_base.

Ported pixel tests' page set into pixel_test_pages.py. Eliminated the
use of separate JSON files for manually-specified expected colors.

The browser is restarted between pixel test pages which require
different browser command line arguments. The new harness's test name
sorting increases the number of restarts, and therefore the execution
time, compared to the old harness. This will be fixed in a follow-on CL.

Deleted the old pixel.py harness to avoid confusion.

BUG=352807
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

[commit-bot: I haz the power, 2016-09-29 22:19:45]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-29 22:22:18]

Description was changed from

==========
Ported pixel_test to new gpu_integration_test harness.

Ported cloud_storage_test_base as cloud_storage_integration_test_base.

Ported pixel tests' page set into pixel_test_pages.py. Eliminated the
use of separate JSON files for manually-specified expected colors.

The browser is restarted between pixel test pages which require
different browser command line arguments. The new harness's test name
sorting increases the number of restarts, and therefore the execution
time, compared to the old harness. This will be fixed in a follow-on CL.

Deleted the old pixel.py harness to avoid confusion.

BUG=352807
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
==========

to

==========
Ported pixel_test to new gpu_integration_test harness.

Ported cloud_storage_test_base as cloud_storage_integration_test_base.

Ported pixel tests' page set into pixel_test_pages.py. Eliminated the
use of separate JSON files for manually-specified expected colors.

The browser is restarted between pixel test pages which require
different browser command line arguments. The new harness's test name
sorting increases the number of restarts, and therefore the execution
time, compared to the old harness. This will be fixed in a follow-on CL.

Deleted the old pixel.py harness to avoid confusion.

BUG=352807
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

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

[commit-bot: I haz the power, 2016-09-29 22:22:19]

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