fix highbit video

fix highbit video

My recent cl for implementing 12-bit video broke re-drawing of highbit video layers.
This is because the resource has to be marked with the right multiplier & offset every time,
even if we're re-using the texture.  This fixes that.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/a14c49913f865eabc5b2a22ed97dc6ab49ff4d18
Cr-Commit-Position: refs/heads/master@{#423382}

[hubbe, 2016-10-05 23:06:35]

Description was changed from

==========
fix highbit video

My recent cl for implementing 12-bit video broke re-drawing of highbit video
layers.
This is because the resource has to be marked with the right multiplier & offset
every time,
even if we're re-using the texture.  This fixes that.
==========

to

==========
fix highbit video

My recent cl for implementing 12-bit video broke re-drawing of highbit video
layers.
This is because the resource has to be marked with the right multiplier & offset
every time,
even if we're re-using the texture.  This fixes that.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[hubbe, 2016-10-05 23:06:54]

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

[hubbe, 2016-10-05 23:07:05]

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

[commit-bot: I haz the power, 2016-10-05 23:07:40]

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

[danakj, 2016-10-05 23:20:01]

Can unit test?

https://codereview.chromium.org/2395943002/diff/1/cc/resources/video_resource...
File cc/resources/video_resource_updater.cc (right):

https://codereview.chromium.org/2395943002/diff/1/cc/resources/video_resource...
cc/resources/video_resource_updater.cc:578: // then we used the OR path below,
which means that we need to
comment wrong now

[hubbe, 2016-10-05 23:41:21]

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

[commit-bot: I haz the power, 2016-10-05 23:41:37]

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

[hubbe, 2016-10-06 00:15:10]

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

[commit-bot: I haz the power, 2016-10-06 00:15:30]

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

[hubbe, 2016-10-06 00:16:30]

Can haz test.

https://codereview.chromium.org/2395943002/diff/1/cc/resources/video_resource...
File cc/resources/video_resource_updater.cc (right):

https://codereview.chromium.org/2395943002/diff/1/cc/resources/video_resource...
cc/resources/video_resource_updater.cc:578: // then we used the OR path below,
which means that we need to
On 2016/10/05 23:20:01, danakj wrote:
> comment wrong now

Done.

[danakj, 2016-10-06 00:19:07]

https://codereview.chromium.org/2395943002/diff/20001/cc/resources/video_reso...
File cc/resources/video_resource_updater_unittest.cc (right):

https://codereview.chromium.org/2395943002/diff/20001/cc/resources/video_reso...
cc/resources/video_resource_updater_unittest.cc:283: VideoFrameExternalResources
resources2 =
What's this 2nd case testing that's diffrent?

[danakj, 2016-10-06 00:19:33]

Description was changed from

==========
fix highbit video

My recent cl for implementing 12-bit video broke re-drawing of highbit video
layers.
This is because the resource has to be marked with the right multiplier & offset
every time,
even if we're re-using the texture.  This fixes that.
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
fix highbit video

My recent cl for implementing 12-bit video broke re-drawing of highbit video
layers.
This is because the resource has to be marked with the right multiplier & offset
every time,
even if we're re-using the texture.  This fixes that.

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

[hubbe, 2016-10-06 00:21:50]

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

[hubbe, 2016-10-06 00:21:51]

https://codereview.chromium.org/2395943002/diff/20001/cc/resources/video_reso...
File cc/resources/video_resource_updater_unittest.cc (right):

https://codereview.chromium.org/2395943002/diff/20001/cc/resources/video_reso...
cc/resources/video_resource_updater_unittest.cc:283: VideoFrameExternalResources
resources2 =
On 2016/10/06 00:19:06, danakj wrote:
> What's this 2nd case testing that's diffrent?

Comment added.

[danakj, 2016-10-06 00:22:35]

Thanks! LGTM

[hubbe, 2016-10-06 00:25:22]

The CQ bit was checked by hubbe@chromium.org

[commit-bot: I haz the power, 2016-10-06 00:25:57]

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-06 01:34:34]

Description was changed from

==========
fix highbit video

My recent cl for implementing 12-bit video broke re-drawing of highbit video
layers.
This is because the resource has to be marked with the right multiplier & offset
every time,
even if we're re-using the texture.  This fixes that.

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

to

==========
fix highbit video

My recent cl for implementing 12-bit video broke re-drawing of highbit video
layers.
This is because the resource has to be marked with the right multiplier & offset
every time,
even if we're re-using the texture.  This fixes that.

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

[commit-bot: I haz the power, 2016-10-06 01:34:36]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-06 01:38:50]

Description was changed from

==========
fix highbit video

My recent cl for implementing 12-bit video broke re-drawing of highbit video
layers.
This is because the resource has to be marked with the right multiplier & offset
every time,
even if we're re-using the texture.  This fixes that.

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

to

==========
fix highbit video

My recent cl for implementing 12-bit video broke re-drawing of highbit video
layers.
This is because the resource has to be marked with the right multiplier & offset
every time,
even if we're re-using the texture.  This fixes that.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel

Committed: https://crrev.com/a14c49913f865eabc5b2a22ed97dc6ab49ff4d18
Cr-Commit-Position: refs/heads/master@{#423382}
==========

[commit-bot: I haz the power, 2016-10-06 01:38:51]

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