[SVG2] Svg elements that overflow the viewport should also be hitdetected.

[SVG2] Svg elements that overflow the viewport should also be hittested.

When we have an svg that doesn't clip to its viewport (aka
overflow:visible), and there is some overflowing content, we
need to consider the overflowing elements when hit testing.
Otherwise it becomes impossible to interact with svg
elements outside the viewport rect.

BUG=401443
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=179806

[pdr., 2014-08-07 22:51:48]

LGTM

It took me a little bit to understand that we are hit testing the overflow.
Could you add a little bit more to the change description so this patch will be
easier to understand a year from now?

https://codereview.chromium.org/444223005/diff/1/LayoutTests/svg/custom/click...
File LayoutTests/svg/custom/click-overflowing-element.html (right):

https://codereview.chromium.org/444223005/diff/1/LayoutTests/svg/custom/click...
LayoutTests/svg/custom/click-overflowing-element.html:14: if
(window.eventSender) {
Nit: the indentation of this file is pretty random.

https://codereview.chromium.org/444223005/diff/1/Source/core/rendering/svg/Re...
File Source/core/rendering/svg/RenderSVGRoot.cpp (right):

https://codereview.chromium.org/444223005/diff/1/Source/core/rendering/svg/Re...
Source/core/rendering/svg/RenderSVGRoot.cpp:425: // Only test SVG content if the
point is in our content box.
Can you update this comment too?

[Erik Dahlström (inactive), 2014-08-08 07:37:02]

The CQ bit was checked by ed@opera.com

[commit-bot: I haz the power, 2014-08-08 07:37:32]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ed@opera.com/444223005/20001

[commit-bot: I haz the power, 2014-08-08 09:22:35]

Change committed as 179806