gpu: support RG_88 GpuMemoryBuffer

gpu: support RG_88 GpuMemoryBuffer

9-16 bit video channel will use it.

BUG=445071, 624436
TEST=gl_tests --gtest_filter=GpuMemoryBufferTest*
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/961b0e1cfb0c951c9bcd824044e2fa95b611312e
Cr-Commit-Position: refs/heads/master@{#422741}

[dshwang, 2016-09-29 15:32:45]

Description was changed from

==========
gpu: support RG_88 GpuMemoryBuffer

9-16 bit video channel will use it.

BUG=445071, 624436
TEST=gl_tests --gtest_filter=GpuMemoryBufferTest*
==========

to

==========
gpu: support RG_88 GpuMemoryBuffer

9-16 bit video channel will use it.

BUG=445071, 624436
TEST=gl_tests --gtest_filter=GpuMemoryBufferTest*
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
==========

[inactive_dshwang_plz_cc_intel, 2016-09-29 15:32:52]

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

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

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

[dshwang, 2016-09-29 15:36:59]

dongseong.hwang@intel.com changed reviewers:
+ hubbe@chromium.org, piman@chromium.org, reveman@chromium.org,
spang@chromium.org

[dshwang, 2016-09-29 15:36:59]

reveman@: Please review changes in ui/
spang@: Please review changes in ui/ozone
piman@: Please review changes in content/ and gpu/

RG_88 GMB will used in https://codereview.chromium.org/2122573003/
In detail, GpuMemoryBuffer video decoder will decode YUV420P10 (10bit channel)
and YUV420P12 (12bit channel) on RG_88 GpuMemoryBuffer per channel.

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

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

[commit-bot: I haz the power, 2016-09-29 16:08:42]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[hubbe, 2016-09-29 17:22:57]

non-owner LGTM

[inactive_dshwang_plz_cc_intel, 2016-09-29 17:35:54]

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

[commit-bot: I haz the power, 2016-09-29 17:36:17]

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 17:46:44]

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

[commit-bot: I haz the power, 2016-09-29 17:46:45]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[reveman, 2016-09-29 21:57:53]

lgtm

[hubbe, 2016-09-30 18:10:37]

On 2016/09/29 21:57:53, reveman wrote:
> lgtm

I think you also need to add GL_RG_EXT in
gpu/command_buffer/client/gles2_implementation.cc CreateImageValidInternalFormat

[inactive_dshwang_plz_cc_intel, 2016-10-03 11:26:21]

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

[commit-bot: I haz the power, 2016-10-03 11:26:29]

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

[dshwang, 2016-10-03 11:27:03]

Description was changed from

==========
gpu: support RG_88 GpuMemoryBuffer

9-16 bit video channel will use it.

BUG=445071, 624436
TEST=gl_tests --gtest_filter=GpuMemoryBufferTest*
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

==========
gpu: support RG_88 GpuMemoryBuffer

9-16 bit video channel will use it.

BUG=445071, 624436
TEST=gl_tests --gtest_filter=GpuMemoryBufferTest*
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
==========

[dshwang, 2016-10-03 11:27:03]

dongseong.hwang@intel.com changed reviewers:
- piman@chromium.org

[dshwang, 2016-10-03 11:32:11]

dongseong.hwang@intel.com changed reviewers:
+ ccameron@chromium.org, meacer@chromium.org, zmo@chromium.org

[dshwang, 2016-10-03 11:32:12]

ccameron@: Please review changes in ui/ozone
zmo@: Please review changes in content/ and gpu/
meacer@: Please review changes in ui/gfx/mojo

On 2016/09/30 18:10:37, hubbe wrote:
> I think you also need to add GL_RG_EXT in
> gpu/command_buffer/client/gles2_implementation.cc
CreateImageValidInternalFormat

That's right. Done.

[commit-bot: I haz the power, 2016-10-03 12:41:45]

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

[commit-bot: I haz the power, 2016-10-03 12:41:46]

Dry run: This issue passed the CQ dry run.

[Zhenyao Mo, 2016-10-03 18:17:37]

On 2016/10/03 11:32:12, dshwang wrote:
> ccameron@: Please review changes in ui/ozone
> zmo@: Please review changes in content/ and gpu/
> meacer@: Please review changes in ui/gfx/mojo
> 
> On 2016/09/30 18:10:37, hubbe wrote:
> > I think you also need to add GL_RG_EXT in
> > gpu/command_buffer/client/gles2_implementation.cc
> CreateImageValidInternalFormat
> 
> That's right. Done.

lgtm

[meacer, 2016-10-03 22:15:33]

On 2016/10/03 18:17:37, Zhenyao Mo wrote:
> On 2016/10/03 11:32:12, dshwang wrote:
> > ccameron@: Please review changes in ui/ozone
> > zmo@: Please review changes in content/ and gpu/
> > meacer@: Please review changes in ui/gfx/mojo
> > 
> > On 2016/09/30 18:10:37, hubbe wrote:
> > > I think you also need to add GL_RG_EXT in
> > > gpu/command_buffer/client/gles2_implementation.cc
> > CreateImageValidInternalFormat
> > 
> > That's right. Done.
> 
> lgtm

ui/gfx/mojo lgtm

[ccameron, 2016-10-03 22:23:27]

ui/ozone OWNER lgtm stamp

[dshwang, 2016-10-04 10:11:05]

The CQ bit was checked by dongseong.hwang@intel.com

[dshwang, 2016-10-04 10:11:05]

The patchset sent to the CQ was uploaded after l-g-t-m from hubbe@chromium.org,
reveman@chromium.org
Link to the patchset: https://codereview.chromium.org/2376293003/#ps40001
(title: "resolve hubbe's review")

[commit-bot: I haz the power, 2016-10-04 10:11: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-10-04 10:15:49]

Description was changed from

==========
gpu: support RG_88 GpuMemoryBuffer

9-16 bit video channel will use it.

BUG=445071, 624436
TEST=gl_tests --gtest_filter=GpuMemoryBufferTest*
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

==========
gpu: support RG_88 GpuMemoryBuffer

9-16 bit video channel will use it.

BUG=445071, 624436
TEST=gl_tests --gtest_filter=GpuMemoryBufferTest*
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
==========

[commit-bot: I haz the power, 2016-10-04 10:15:51]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-04 10:17:47]

Description was changed from

==========
gpu: support RG_88 GpuMemoryBuffer

9-16 bit video channel will use it.

BUG=445071, 624436
TEST=gl_tests --gtest_filter=GpuMemoryBufferTest*
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

==========
gpu: support RG_88 GpuMemoryBuffer

9-16 bit video channel will use it.

BUG=445071, 624436
TEST=gl_tests --gtest_filter=GpuMemoryBufferTest*
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/961b0e1cfb0c951c9bcd824044e2fa95b611312e
Cr-Commit-Position: refs/heads/master@{#422741}
==========

[commit-bot: I haz the power, 2016-10-04 10:17:48]

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