add range check to service-side MapBufferRange

add range check to service-side MapBufferRange

BUG=650123
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

Committed: https://crrev.com/6bf82374fb3a17659851bc4e830c063577ba80fd
Cr-Commit-Position: refs/heads/master@{#422326}

[Kai Ninomiya, 2016-09-28 21:57:04]

Description was changed from

==========
add range check to service-side MapBufferRange

BUG=650123
==========

to

==========
add range check to service-side MapBufferRange

BUG=650123
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
==========

[Kai Ninomiya, 2016-09-28 21:57:51]

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

[Kai Ninomiya, 2016-09-28 21:58:18]

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

[Kai Ninomiya, 2016-09-28 21:58:19]

PTAL

[commit-bot: I haz the power, 2016-09-28 21:58:19]

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

[Kai Ninomiya, 2016-09-28 21:58:50]

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

[Kai Ninomiya, 2016-09-28 21:58:50]

PTAL

[Kai Ninomiya, 2016-09-28 22:21:31]

The CQ bit was unchecked by kainino@chromium.org

[Kai Ninomiya, 2016-09-29 01:08:37]

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

[commit-bot: I haz the power, 2016-09-29 01:09:15]

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 02:40:30]

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

[commit-bot: I haz the power, 2016-09-29 02:40:30]

Dry run: This issue passed the CQ dry run.

[Ken Russell (switch to Gerrit), 2016-09-30 01:06:07]

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

[Ken Russell (switch to Gerrit), 2016-09-30 01:06:07]

Kai, thank you for tracking this down and fixing it. It looks great overall.

Could you also please remove the flaky expectation for
conformance2/buffers/getBufferSubData.html from
src/content/test/gpu/gpu_tests/webgl2_conformance_expectations.py ?

Mo, could you please review this too?

[Zhenyao Mo, 2016-09-30 20:02:37]

On 2016/09/30 01:06:07, Ken Russell wrote:
> Kai, thank you for tracking this down and fixing it. It looks great overall.
> 
> Could you also please remove the flaky expectation for
> conformance2/buffers/getBufferSubData.html from
> src/content/test/gpu/gpu_tests/webgl2_conformance_expectations.py ?
> 
> Mo, could you please review this too?

lgtm

[Kai Ninomiya, 2016-09-30 21:19:20]

The CQ bit was checked by kainino@chromium.org

[Kai Ninomiya, 2016-09-30 21:19:21]

The patchset sent to the CQ was uploaded after l-g-t-m from zmo@chromium.org
Link to the patchset: https://codereview.chromium.org/2380683003/#ps40001
(title: "remove flaky expectation")

[Kai Ninomiya, 2016-09-30 21:19:38]

The CQ bit was unchecked by kainino@chromium.org

[Kai Ninomiya, 2016-09-30 21:22:57]

The CQ bit was checked by kainino@chromium.org

[commit-bot: I haz the power, 2016-09-30 21:23:39]

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

[Zhenyao Mo, 2016-10-01 18:56:07]

The CQ bit was checked by zmo@chromium.org

[Zhenyao Mo, 2016-10-01 18:56:08]

The patchset sent to the CQ was uploaded after l-g-t-m from zmo@chromium.org
Link to the patchset: https://codereview.chromium.org/2380683003/#ps60001
(title: "rebase over reformat")

[commit-bot: I haz the power, 2016-10-01 18:56:22]

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-10-01 19:45:51]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-01 19:47:13]

Description was changed from

==========
add range check to service-side MapBufferRange

BUG=650123
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
==========

to

==========
add range check to service-side MapBufferRange

BUG=650123
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

Committed: https://crrev.com/6bf82374fb3a17659851bc4e830c063577ba80fd
Cr-Commit-Position: refs/heads/master@{#422326}
==========

[commit-bot: I haz the power, 2016-10-01 19:47:14]

Patchset 4 (id:??) landed as
https://crrev.com/6bf82374fb3a17659851bc4e830c063577ba80fd
Cr-Commit-Position: refs/heads/master@{#422326}