Remove the command buffer method glGetImageivCHROMIUM.

Remove the command buffer method glGetImageivCHROMIUM.

It is no longer necessary, since the browser compositor no longer needs access
to the underlying GpuMemoryBuffers.

This CL is a revert of "Expose GpuMemoryBufferId through glGetImageivCHROMIUM."
> WebKit widgets that provide textures to TextureLayer need to also provide the
> GpuMemoryBufferId if the texture is backed by a GpuMemoryBuffer. This CL
> modifies the CHROMIUM Image extension to allow the GpuMemoryBuffer id to be
> obtained by a new function glGetImageivCHROMIUM.

BUG=608026
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/7cb1503e57864170612b6a7ad6dc1b8c52907c44
Cr-Commit-Position: refs/heads/master@{#404502}

[erikchen, 2016-07-08 16:28:19]

Description was changed from

==========
Remove the command buffer method glGetImageivCHROMIUM.

It is no longer necessary, since the browser compositor no longer needs access
to the underlying GpuMemoryBuffers.

BUG=
==========

to

==========
Remove the command buffer method glGetImageivCHROMIUM.

It is no longer necessary, since the browser compositor no longer needs access
to the underlying GpuMemoryBuffers.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[erikchen, 2016-07-08 16:28:20]

Description was changed from

==========
Remove the command buffer method glGetImageivCHROMIUM.

It is no longer necessary, since the browser compositor no longer needs access
to the underlying GpuMemoryBuffers.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel;tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Remove the command buffer method glGetImageivCHROMIUM.

It is no longer necessary, since the browser compositor no longer needs access
to the underlying GpuMemoryBuffers.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[erikchen, 2016-07-08 16:28:31]

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

[commit-bot: I haz the power, 2016-07-08 16:28:58]

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

[erikchen, 2016-07-08 16:31:38]

Description was changed from

==========
Remove the command buffer method glGetImageivCHROMIUM.

It is no longer necessary, since the browser compositor no longer needs access
to the underlying GpuMemoryBuffers.

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Remove the command buffer method glGetImageivCHROMIUM.

It is no longer necessary, since the browser compositor no longer needs access
to the underlying GpuMemoryBuffers.

This CL is a revert of "Expose GpuMemoryBufferId through glGetImageivCHROMIUM."
> WebKit widgets that provide textures to TextureLayer need to also provide the
> GpuMemoryBufferId if the texture is backed by a GpuMemoryBuffer. This CL
> modifies the CHROMIUM Image extension to allow the GpuMemoryBuffer id to be
> obtained by a new function glGetImageivCHROMIUM.

BUG=608026
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[erikchen, 2016-07-08 16:35:34]

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

[commit-bot: I haz the power, 2016-07-08 16:35:54]

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-07-08 16:52:44]

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

[commit-bot: I haz the power, 2016-07-08 16:52:45]

Dry run: Try jobs failed on following builders:
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[erikchen, 2016-07-08 17:58:49]

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

[commit-bot: I haz the power, 2016-07-08 17:59:23]

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-07-08 19:29:04]

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

[commit-bot: I haz the power, 2016-07-08 19:29:05]

Dry run: This issue passed the CQ dry run.

[erikchen, 2016-07-08 19:44:51]

erikchen@chromium.org changed reviewers:
+ piman@chromium.org

[erikchen, 2016-07-08 19:44:52]

piman: Please review everything but third_party/WebKit/Source/platform/graphics
kbr: Please review third_party/WebKit/Source/platform/graphics

[erikchen, 2016-07-08 19:45:16]

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

[erikchen, 2016-07-08 19:45:17]

kbr: Please review third_party/WebKit/Source/platform/graphics

[erikchen, 2016-07-08 19:45:46]

erikchen@chromium.org changed reviewers:
+ sky@chromium.org

[erikchen, 2016-07-08 19:45:46]

sky: Please review services/ui/

[piman, 2016-07-08 20:11:43]

lgtm

[sky, 2016-07-08 20:49:28]

LGTM

[Ken Russell (switch to Gerrit), 2016-07-08 21:43:22]

Great! Thanks for following up and cleaning this up. LGTM

[erikchen, 2016-07-08 21:45:09]

The CQ bit was checked by erikchen@chromium.org

[commit-bot: I haz the power, 2016-07-08 21:45:36]

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-07-08 21:53:27]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-08 21:55:57]

Description was changed from

==========
Remove the command buffer method glGetImageivCHROMIUM.

It is no longer necessary, since the browser compositor no longer needs access
to the underlying GpuMemoryBuffers.

This CL is a revert of "Expose GpuMemoryBufferId through glGetImageivCHROMIUM."
> WebKit widgets that provide textures to TextureLayer need to also provide the
> GpuMemoryBufferId if the texture is backed by a GpuMemoryBuffer. This CL
> modifies the CHROMIUM Image extension to allow the GpuMemoryBuffer id to be
> obtained by a new function glGetImageivCHROMIUM.

BUG=608026
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Remove the command buffer method glGetImageivCHROMIUM.

It is no longer necessary, since the browser compositor no longer needs access
to the underlying GpuMemoryBuffers.

This CL is a revert of "Expose GpuMemoryBufferId through glGetImageivCHROMIUM."
> WebKit widgets that provide textures to TextureLayer need to also provide the
> GpuMemoryBufferId if the texture is backed by a GpuMemoryBuffer. This CL
> modifies the CHROMIUM Image extension to allow the GpuMemoryBuffer id to be
> obtained by a new function glGetImageivCHROMIUM.

BUG=608026
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/7cb1503e57864170612b6a7ad6dc1b8c52907c44
Cr-Commit-Position: refs/heads/master@{#404502}
==========

[commit-bot: I haz the power, 2016-07-08 21:55:58]

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