Add supported sized formats by CHROMIUM_ycbcr_420v/422_image

Add supported sized formats by CHROMIUM_ycbcr_420v/422_image

BUG=657532
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/cbdb34d5124364d85882f4f75d6b43377bfda0c0
Cr-Commit-Position: refs/heads/master@{#428930}

[qiankun, 2016-10-31 05:49:53]

Description was changed from

==========
Add supported sized formats by CHROMIUM_ycbcr_420v/422_image

BUG=657532
==========

to

==========
Add supported sized formats by CHROMIUM_ycbcr_420v/422_image

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

[qiankun, 2016-10-31 15:56:57]

qiankun.miao@intel.com changed reviewers:
+ dongseong.hwang@intel.com, kbr@chromium.org, yunchao.he@intel.com,
zmo@chromium.org

[qiankun, 2016-10-31 15:56:58]

This fixes the WebRTC regression. I think it's the real fix other than the
workaround dongseong added in https://codereview.chromium.org/2448083003/. PTAL.

[qiankun, 2016-10-31 16:01:47]

On 2016/10/31 15:56:58, qiankun wrote:
> This fixes the WebRTC regression. I think it's the real fix other than the
> workaround dongseong added in https://codereview.chromium.org/2448083003/.
PTAL.

The regression is still existed in ToT Chromium on Mac. It's fixed by this CL.

[dshwang, 2016-10-31 16:54:33]

On 2016/10/31 16:01:47, qiankun wrote:
> On 2016/10/31 15:56:58, qiankun wrote:
> > This fixes the WebRTC regression. I think it's the real fix other than the
> > workaround dongseong added in https://codereview.chromium.org/2448083003/.
> PTAL.
> 
> The regression is still existed in ToT Chromium on Mac. It's fixed by this CL.

That's right. looks good to me.
Please remove MacOS hack as you fix the root bug.
The hack is defined(OS_MACOSX) block in
https://cs.chromium.org/chromium/src/gpu/command_buffer/service/gles2_cmd_dec...

[Zhenyao Mo, 2016-10-31 18:12:30]

lgtm

[qiankun, 2016-11-01 02:26:08]

On 2016/10/31 16:54:33, dshwang wrote:
> On 2016/10/31 16:01:47, qiankun wrote:
> > On 2016/10/31 15:56:58, qiankun wrote:
> > > This fixes the WebRTC regression. I think it's the real fix other than the
> > > workaround dongseong added in https://codereview.chromium.org/2448083003/.
> > PTAL.
> > 
> > The regression is still existed in ToT Chromium on Mac. It's fixed by this
CL.
> 
> That's right. looks good to me.
> Please remove MacOS hack as you fix the root bug.
> The hack is defined(OS_MACOSX) block in
>
https://cs.chromium.org/chromium/src/gpu/command_buffer/service/gles2_cmd_dec...

The hack was removed.

[qiankun, 2016-11-01 02:26:22]

The CQ bit was checked by qiankun.miao@intel.com

[qiankun, 2016-11-01 02:26:22]

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/2458423002/#ps20001
(title: "remove internal_format hack for Mac OSX")

[commit-bot: I haz the power, 2016-11-01 02:26:47]

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-11-01 03:24:01]

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

[commit-bot: I haz the power, 2016-11-01 03:24:02]

Try jobs failed on following builders:
  linux_optional_gpu_tests_rel on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_optional_...)

[Ken Russell (switch to Gerrit), 2016-11-01 03:38:11]

On 2016/11/01 03:24:02, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   linux_optional_gpu_tests_rel on master.tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_optional_...)

Sorry, that's http://crbug.com/659165 , being fixed. Re-CQ'ing.

[Ken Russell (switch to Gerrit), 2016-11-01 03:38:16]

The CQ bit was checked by kbr@chromium.org

[commit-bot: I haz the power, 2016-11-01 03:38:31]

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

[qiankun, 2016-11-01 03:46:00]

On 2016/11/01 03:38:11, Ken Russell wrote:
> On 2016/11/01 03:24:02, commit-bot: I haz the power wrote:
> > Try jobs failed on following builders:
> >   linux_optional_gpu_tests_rel on master.tryserver.chromium.linux
(JOB_FAILED,
> >
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_optional_...)
> 
> Sorry, that's http://crbug.com/659165 , being fixed. Re-CQ'ing.

Thank you, ken. I am just searching the bug.

[commit-bot: I haz the power, 2016-11-01 03:55:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-01 03:57:53]

Description was changed from

==========
Add supported sized formats by CHROMIUM_ycbcr_420v/422_image

BUG=657532
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 supported sized formats by CHROMIUM_ycbcr_420v/422_image

BUG=657532
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/cbdb34d5124364d85882f4f75d6b43377bfda0c0
Cr-Commit-Position: refs/heads/master@{#428930}
==========

[commit-bot: I haz the power, 2016-11-01 03:57:54]

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