cc cleanup: simplify mask_uv_scale and MaskUVRect().

cc cleanup: simplify mask_uv_scale and MaskUVRect().

RenderPassDrawQuad::MaskUVRect() computes the UV rect by scaling
the quad->rect() by mask_uv_scale, then dividing by quad->rect size.
However, RenderSurfaceImpl::AppendQuads() computes the mask UV scale
as quad->rect() size / mask size. So let's just take the quad->rect() size
out of mask_uv_scale in the first place, so we don't have to divide
it back out in MaskUVRect().

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[Stephen White, 2016-01-27 19:25:10]

Description was changed from

==========
cc cleanup: simplify mask_uv_scale and MaskUVRect().

RenderPassDrawQuad::MaskUVRect() computes the UV rect by scaling
the quad->rect() by mask_uv_scale, then dividing by quad->rect size.
However, RenderSurfaceImpl::AppendQuads() computes the mask UV scale
as quad->rect() size / mask size. So let's just take the quad->rect() size
out of mask_uv_scale in the first place, so we don't have to divide
it back out in MaskUVRect().

BUG=
==========

to

==========
cc cleanup: simplify mask_uv_scale and MaskUVRect().

RenderPassDrawQuad::MaskUVRect() computes the UV rect by scaling
the quad->rect() by mask_uv_scale, then dividing by quad->rect size.
However, RenderSurfaceImpl::AppendQuads() computes the mask UV scale
as quad->rect() size / mask size. So let's just take the quad->rect() size
out of mask_uv_scale in the first place, so we don't have to divide
it back out in MaskUVRect().

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Stephen White, 2016-01-27 19:26:08]

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

[commit-bot: I haz the power, 2016-01-27 19:26:40]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1647573002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1647573002/1

[Stephen White, 2016-01-27 19:26:55]

Description was changed from

==========
cc cleanup: simplify mask_uv_scale and MaskUVRect().

RenderPassDrawQuad::MaskUVRect() computes the UV rect by scaling
the quad->rect() by mask_uv_scale, then dividing by quad->rect size.
However, RenderSurfaceImpl::AppendQuads() computes the mask UV scale
as quad->rect() size / mask size. So let's just take the quad->rect() size
out of mask_uv_scale in the first place, so we don't have to divide
it back out in MaskUVRect().

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc cleanup: simplify mask_uv_scale and MaskUVRect().

RenderPassDrawQuad::MaskUVRect() computes the UV rect by scaling
the quad->rect() by mask_uv_scale, then dividing by quad->rect size.
However, RenderSurfaceImpl::AppendQuads() computes the mask UV scale
as quad->rect() size / mask size. So let's just take the quad->rect() size
out of mask_uv_scale in the first place, so we don't have to divide
it back out in MaskUVRect().

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[Stephen White, 2016-01-27 19:26:55]

senorblanco@chromium.org changed reviewers:
+ enne@chromium.org

[Stephen White, 2016-01-27 19:27:04]

enne@: PTAL. Thanks!

[commit-bot: I haz the power, 2016-01-27 20:43:26]

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

[commit-bot: I haz the power, 2016-01-27 20:43:26]

Dry run: This issue passed the CQ dry run.

[enne (OOO), 2016-01-27 21:35:39]

lgtm

[Stephen White, 2016-01-27 21:39:41]

The CQ bit was checked by senorblanco@chromium.org

[commit-bot: I haz the power, 2016-01-27 21:40:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1647573002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1647573002/1

[commit-bot: I haz the power, 2016-01-27 21:48:07]

Description was changed from

==========
cc cleanup: simplify mask_uv_scale and MaskUVRect().

RenderPassDrawQuad::MaskUVRect() computes the UV rect by scaling
the quad->rect() by mask_uv_scale, then dividing by quad->rect size.
However, RenderSurfaceImpl::AppendQuads() computes the mask UV scale
as quad->rect() size / mask size. So let's just take the quad->rect() size
out of mask_uv_scale in the first place, so we don't have to divide
it back out in MaskUVRect().

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc cleanup: simplify mask_uv_scale and MaskUVRect().

RenderPassDrawQuad::MaskUVRect() computes the UV rect by scaling
the quad->rect() by mask_uv_scale, then dividing by quad->rect size.
However, RenderSurfaceImpl::AppendQuads() computes the mask UV scale
as quad->rect() size / mask size. So let's just take the quad->rect() size
out of mask_uv_scale in the first place, so we don't have to divide
it back out in MaskUVRect().

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-01-27 21:48:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-27 21:49:22]

Description was changed from

==========
cc cleanup: simplify mask_uv_scale and MaskUVRect().

RenderPassDrawQuad::MaskUVRect() computes the UV rect by scaling
the quad->rect() by mask_uv_scale, then dividing by quad->rect size.
However, RenderSurfaceImpl::AppendQuads() computes the mask UV scale
as quad->rect() size / mask size. So let's just take the quad->rect() size
out of mask_uv_scale in the first place, so we don't have to divide
it back out in MaskUVRect().

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc cleanup: simplify mask_uv_scale and MaskUVRect().

RenderPassDrawQuad::MaskUVRect() computes the UV rect by scaling
the quad->rect() by mask_uv_scale, then dividing by quad->rect size.
However, RenderSurfaceImpl::AppendQuads() computes the mask UV scale
as quad->rect() size / mask size. So let's just take the quad->rect() size
out of mask_uv_scale in the first place, so we don't have to divide
it back out in MaskUVRect().

BUG=
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

[commit-bot: I haz the power, 2016-01-27 21:49:23]

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