Command buffer: generate appropriate error when querying IMPLEMENTATION_COLOR_READ_FORMAT/TYPE

Command buffer: generate appropriate error when querying IMPLEMENTATION_COLOR_READ_FORMAT/TYPE.

When fbo incomplete, it should generate INVALID_FRAMEBUFFER_OPERATION, instead of INVALID_OPERATION.

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

[yunchao, 2016-12-09 07:43:33]

Description was changed from

==========
Command buffer: generate appropriate error when querying
IMPLEMENTATION_COLOR_READ_FORMAT/TYPE

BUG=672711
==========

to

==========
Command buffer: generate appropriate error when querying
IMPLEMENTATION_COLOR_READ_FORMAT/TYPE

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

[yunchao, 2016-12-09 08:47:57]

Description was changed from

==========
Command buffer: generate appropriate error when querying
IMPLEMENTATION_COLOR_READ_FORMAT/TYPE

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

==========
Command buffer: generate appropriate error when querying
IMPLEMENTATION_COLOR_READ_FORMAT/TYPE.

When fbo incomplete, it should generate INVALID_FRAMEBUFFER_OPERATION, instead
of INVALID_OPERATION.

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

[yunchao, 2016-12-09 08:47:57]

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

[yunchao, 2016-12-09 08:51:32]

I came across crbug/672711, it seems to me that a small change can fix it. Here
it is. PTAL.

After the roll-in patch at https://codereview.chromium.org/2565543004/ landed,
the bots should be all green.

[yunchao, 2016-12-10 00:51:54]

The CQ bit was checked by yunchao.he@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-12-10 00:53:00]

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-12-10 03:54:12]

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

[commit-bot: I haz the power, 2016-12-10 03:54:13]

Dry run: This issue passed the CQ dry run.

[Ken Russell (switch to Gerrit), 2016-12-13 00:14:55]

Sorry Yunchao but Mo found spec text that made it clear that INVALID_OPERATION
is the right error. Let's close this.

[yunchao, 2016-12-13 01:42:32]

On 2016/12/13 00:14:55, Ken Russell wrote:
> Sorry Yunchao but Mo found spec text that made it clear that INVALID_OPERATION
> is the right error. Let's close this.

That's fine. Thanks!