Implement mapAncestorToLocal for LayoutSVG{ModelObject,Block}

Implement mapAncestorToLocal for LayoutSVG{ModelObject,Block}

This CL adds an implementation of mapAncestorToLocal for SVG
LayoutObject types via a helper in SVGLayoutSupport.
This should help cases which use any of the LayoutObject::ancestorTo*
methods (or similar/wrappers.) Examples of users are various form
controls and scrollbars.

BUG=568614
Committed: https://crrev.com/a663c6ee1aef79f2de75c7620b72cea48630b9ca
Cr-Commit-Position: refs/heads/master@{#378716}

[fs, 2016-03-01 14:50:36]

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

[pdr., 2016-03-01 22:58:39]

LGTM, you are on a roll!

https://codereview.chromium.org/1747223002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/layout/svg/SVGLayoutSupport.cpp (right):

https://codereview.chromium.org/1747223002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/layout/svg/SVGLayoutSupport.cpp:168: // |object|
is either a LayoutSVGModelObject or a LayoutSVGBlock here. In
Can you add an assert for this?

[fs, 2016-03-02 08:54:30]

https://codereview.chromium.org/1747223002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/layout/svg/SVGLayoutSupport.cpp (right):

https://codereview.chromium.org/1747223002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/layout/svg/SVGLayoutSupport.cpp:168: // |object|
is either a LayoutSVGModelObject or a LayoutSVGBlock here. In
On 2016/03/01 at 22:58:39, pdr wrote:
> Can you add an assert for this?

It's a bit of a handful to express, but sure, added.

[pdr., 2016-03-02 09:00:11]

On 2016/03/02 at 08:54:30, fs wrote:
>
https://codereview.chromium.org/1747223002/diff/20001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/core/layout/svg/SVGLayoutSupport.cpp (right):
> 
>
https://codereview.chromium.org/1747223002/diff/20001/third_party/WebKit/Sour...
> third_party/WebKit/Source/core/layout/svg/SVGLayoutSupport.cpp:168: //
|object| is either a LayoutSVGModelObject or a LayoutSVGBlock here. In
> On 2016/03/01 at 22:58:39, pdr wrote:
> > Can you add an assert for this?
> 
> It's a bit of a handful to express, but sure, added.

Thanks. LGTM

[fs, 2016-03-02 10:28:44]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-03-02 10:29:05]

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

[commit-bot: I haz the power, 2016-03-02 10:37:09]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-02 10:39:39]

Description was changed from

==========
Implement mapAncestorToLocal for LayoutSVG{ModelObject,Block}

This CL adds an implementation of mapAncestorToLocal for SVG
LayoutObject types via a helper in SVGLayoutSupport.
This should help cases which use any of the LayoutObject::ancestorTo*
methods (or similar/wrappers.) Examples of users are various form
controls and scrollbars.

BUG=568614
==========

to

==========
Implement mapAncestorToLocal for LayoutSVG{ModelObject,Block}

This CL adds an implementation of mapAncestorToLocal for SVG
LayoutObject types via a helper in SVGLayoutSupport.
This should help cases which use any of the LayoutObject::ancestorTo*
methods (or similar/wrappers.) Examples of users are various form
controls and scrollbars.

BUG=568614

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

[commit-bot: I haz the power, 2016-03-02 10:39:40]

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