[SVG] Compensate for container size snapping

[SVG] Compensate for container size snapping

BoxPainter rounds its box rect coordinates to ensure optimal alignment
and minimize AA bleed.

SVGRootPainter attempts to do the same, but it only adjusts the offset.

Since int-snapping may also change the size, the SVG content transform
needs to take this residual scale into account (such that it maps onto
the full int-snapped area).

BUG=599095
R=schenney@chromium.org,fs@opera.com,pdr@chromium.org

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

[f(malita), 2016-03-31 19:24:50]

Description was changed from

==========
[SVG] Compensate for container size snapping

WIP

BUG=599095
==========

to

==========
[SVG] Compensate for container size snapping

BoxPainter rounds coordinates to ensure optimal alignment and minimize
AA bleed.

SVGRootPainter attempts to do the same, but it only adjusts the offset.

Since int-snapping may also change the size, the SVG content transform
needs to take this residual scale into account (such that it maps onto
the full int-snapped area).

BUG=599095
R=schenney@chromium.org,fs@opera.com,pdr@chromium.org
==========

[f(malita), 2016-03-31 19:24:50]

fmalita@chromium.org changed reviewers:
+ fs@opera.com, pdr@chromium.org, schenney@chromium.org

[f(malita), 2016-03-31 19:27:18]

Description was changed from

==========
[SVG] Compensate for container size snapping

BoxPainter rounds coordinates to ensure optimal alignment and minimize
AA bleed.

SVGRootPainter attempts to do the same, but it only adjusts the offset.

Since int-snapping may also change the size, the SVG content transform
needs to take this residual scale into account (such that it maps onto
the full int-snapped area).

BUG=599095
R=schenney@chromium.org,fs@opera.com,pdr@chromium.org
==========

to

==========
[SVG] Compensate for container size snapping

BoxPainter rounds its box rect coordinates to ensure optimal alignment
and minimize AA bleed.

SVGRootPainter attempts to do the same, but it only adjusts the offset.

Since int-snapping may also change the size, the SVG content transform
needs to take this residual scale into account (such that it maps onto
the full int-snapped area).

BUG=599095
R=schenney@chromium.org,fs@opera.com,pdr@chromium.org
==========

[f(malita), 2016-03-31 20:09:02]

10 minor rebaselines:
https://storage.googleapis.com/chromium-layout-test-archives/mac_chromium_rel...

[Stephen Chennney, 2016-03-31 20:26:12]

LGTM. It's worth also checking that the page from the original bug report looks
correct now. If it doesn't there's probably another bug to file.

[f(malita), 2016-03-31 20:42:51]

On 2016/03/31 20:26:12, Stephen Chenney wrote:
> LGTM. It's worth also checking that the page from the original bug report
looks
> correct now. If it doesn't there's probably another bug to file.

This change fixes the bug-highlighted artifacts (lower right side), but I'm
still seeing some bg bleed in the upper right corner at 25% zoom only.  Not
clear whether that's still SVG related or something else, will try to minimize.

[f(malita), 2016-03-31 20:59:45]

On 2016/03/31 20:42:51, f(malita) wrote:
> On 2016/03/31 20:26:12, Stephen Chenney wrote:
> > LGTM. It's worth also checking that the page from the original bug report
> looks
> > correct now. If it doesn't there's probably another bug to file.
> 
> This change fixes the bug-highlighted artifacts (lower right side), but I'm
> still seeing some bg bleed in the upper right corner at 25% zoom only.  Not
> clear whether that's still SVG related or something else, will try to
minimize.

Verified that the remaining bleed doesn't involve SVG: the upper section is just
an image, so we probably a different snapping issue with bitmaps.

[f(malita), 2016-04-01 12:40:32]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2016-04-01 12:40:51]

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

[commit-bot: I haz the power, 2016-04-01 14:26:18]

Description was changed from

==========
[SVG] Compensate for container size snapping

BoxPainter rounds its box rect coordinates to ensure optimal alignment
and minimize AA bleed.

SVGRootPainter attempts to do the same, but it only adjusts the offset.

Since int-snapping may also change the size, the SVG content transform
needs to take this residual scale into account (such that it maps onto
the full int-snapped area).

BUG=599095
R=schenney@chromium.org,fs@opera.com,pdr@chromium.org
==========

to

==========
[SVG] Compensate for container size snapping

BoxPainter rounds its box rect coordinates to ensure optimal alignment
and minimize AA bleed.

SVGRootPainter attempts to do the same, but it only adjusts the offset.

Since int-snapping may also change the size, the SVG content transform
needs to take this residual scale into account (such that it maps onto
the full int-snapped area).

BUG=599095
R=schenney@chromium.org,fs@opera.com,pdr@chromium.org
==========

[commit-bot: I haz the power, 2016-04-01 14:26:18]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-01 14:27:33]

Description was changed from

==========
[SVG] Compensate for container size snapping

BoxPainter rounds its box rect coordinates to ensure optimal alignment
and minimize AA bleed.

SVGRootPainter attempts to do the same, but it only adjusts the offset.

Since int-snapping may also change the size, the SVG content transform
needs to take this residual scale into account (such that it maps onto
the full int-snapped area).

BUG=599095
R=schenney@chromium.org,fs@opera.com,pdr@chromium.org
==========

to

==========
[SVG] Compensate for container size snapping

BoxPainter rounds its box rect coordinates to ensure optimal alignment
and minimize AA bleed.

SVGRootPainter attempts to do the same, but it only adjusts the offset.

Since int-snapping may also change the size, the SVG content transform
needs to take this residual scale into account (such that it maps onto
the full int-snapped area).

BUG=599095
R=schenney@chromium.org,fs@opera.com,pdr@chromium.org

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

[commit-bot: I haz the power, 2016-04-01 14:27:34]

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