hi10p+LUT bugfix

hi10p+LUT bugfix

BUG=622133
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/26c4111e54e3f8c0636480ee7b309f588be13b94
Cr-Commit-Position: refs/heads/master@{#411903}

[hubbe, 2016-08-12 23:08:20]

Description was changed from

==========
hi10p+LUT bugfix

BUG=622133
==========

to

==========
hi10p+LUT bugfix

BUG=622133
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[hubbe, 2016-08-12 23:08:29]

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

[commit-bot: I haz the power, 2016-08-12 23:09:22]

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

[hubbe, 2016-08-12 23:09:51]

hubbe@chromium.org changed reviewers:
+ danakj@chromium.org

[danakj, 2016-08-12 23:25:25]

Is there no pixel tests for this shader?

[hubbe, 2016-08-12 23:45:05]

On 2016/08/12 23:25:25, danakj wrote:
> Is there no pixel tests for this shader?

The LUT code is behind a flag, and there are no tests that explicitly enable the
flag yet.

[danakj, 2016-08-12 23:51:14]

On 2016/08/12 23:45:05, hubbe wrote:
> On 2016/08/12 23:25:25, danakj wrote:
> > Is there no pixel tests for this shader?
> 
> The LUT code is behind a flag, and there are no tests that explicitly enable
the
> flag yet.

Ah, time to add one yet?

[hubbe, 2016-08-13 00:19:12]

On 2016/08/12 23:51:14, danakj wrote:
> On 2016/08/12 23:45:05, hubbe wrote:
> > On 2016/08/12 23:25:25, danakj wrote:
> > > Is there no pixel tests for this shader?
> > 
> > The LUT code is behind a flag, and there are no tests that explicitly enable
> the
> > flag yet.
> 
> Ah, time to add one yet?

Yes, plenty of time, but not today. :)

[danakj, 2016-08-13 00:19:49]

Well, rubberstamp LGTM then

[commit-bot: I haz the power, 2016-08-13 00:25:35]

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

[commit-bot: I haz the power, 2016-08-13 00:25:36]

Dry run: This issue passed the CQ dry run.

[hubbe, 2016-08-13 22:57:58]

The CQ bit was checked by hubbe@chromium.org

[commit-bot: I haz the power, 2016-08-13 22:58:03]

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-08-13 23:01:43]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-13 23:03:28]

Description was changed from

==========
hi10p+LUT bugfix

BUG=622133
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
hi10p+LUT bugfix

BUG=622133
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/26c4111e54e3f8c0636480ee7b309f588be13b94
Cr-Commit-Position: refs/heads/master@{#411903}
==========

[commit-bot: I haz the power, 2016-08-13 23:03:29]

Patchset 1 (id:??) landed as
https://crrev.com/26c4111e54e3f8c0636480ee7b309f588be13b94
Cr-Commit-Position: refs/heads/master@{#411903}