Made Blink tooltip ShadowDOM-aware

Made Blink tooltip ShadowDOM-aware

Before this change, tooltip lookup on EventHandler::mouseMoved() filtered
in-shadow nodes out. That results broken tooltip display of these in-shadow elements.
Such filtering was originally introduced to hide UA shadows, but the implementation
was a bit too aggressive that it filtered all shadow out.

This chagne add HitTestResult::setToShadowHostIfUserAgentShadowRoot().
This is a relaxed version of such filtering, which only hides nodes in UA shadow.
Using this, tooltip is correctly located even if it is inside shadows.

BUG=234100
TEST=tooltips-in-shdaow.html

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

[Hajime Morrita, 2013-05-13 10:49:31]

Hayato-san, could you take an informal review?
Then Dimitri might be able to stamp while we're asleep.

[dglazkov, 2013-05-13 16:11:42]

https://codereview.chromium.org/14677012/diff/1/LayoutTests/fast/dom/shadow/t...
File LayoutTests/fast/dom/shadow/tooltips-in-shadow.html (right):

https://codereview.chromium.org/14677012/diff/1/LayoutTests/fast/dom/shadow/t...
LayoutTests/fast/dom/shadow/tooltips-in-shadow.html:27:
container.appendChild(D('div', {'id': 'host3'}, S(D('div', {'id': 'scope3a'})),
S(D('a', {'id': 'scope3b', 'title': label3 },
document.createTextNode("Hello3")))));
It would be cool to start using <template> + a custom element for ShadowRoot in
these cases:

<template>
  <div id="host3">
     <shadow-root>
        <div id="scope3a">
...

On the other hand, mixing in 2 more recent technologies into a test harness
could be a bad idea, too :)

https://codereview.chromium.org/14677012/diff/1/Source/core/rendering/HitTest...
File Source/core/rendering/HitTestResult.cpp (right):

https://codereview.chromium.org/14677012/diff/1/Source/core/rendering/HitTest...
Source/core/rendering/HitTestResult.cpp:116: void
HitTestResult::setToNodesInDocumentTreeScope()
It's not necessarily document tree scope, right? You're just looking one up?

[Hajime Morrita, 2013-05-14 00:13:52]

Thanks for reviewing, Dimitri!

https://codereview.chromium.org/14677012/diff/1/LayoutTests/fast/dom/shadow/t...
File LayoutTests/fast/dom/shadow/tooltips-in-shadow.html (right):

https://codereview.chromium.org/14677012/diff/1/LayoutTests/fast/dom/shadow/t...
LayoutTests/fast/dom/shadow/tooltips-in-shadow.html:27:
container.appendChild(D('div', {'id': 'host3'}, S(D('div', {'id': 'scope3a'})),
S(D('a', {'id': 'scope3b', 'title': label3 },
document.createTextNode("Hello3")))));
Um... next time :-)
It would be more useful when we build larger trees.

https://codereview.chromium.org/14677012/diff/1/Source/core/rendering/HitTest...
File Source/core/rendering/HitTestResult.cpp (right):

https://codereview.chromium.org/14677012/diff/1/Source/core/rendering/HitTest...
Source/core/rendering/HitTestResult.cpp:119: node =
node->document()->ancestorInThisScope(node);
This asks document()->ancestorInThisScope(), which has while() in it.

[dglazkov, 2013-05-14 00:44:13]

LGTM.

[Hajime Morrita, 2013-05-14 00:52:07]

Thx!

[commit-bot: I haz the power, 2013-05-14 00:52:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/morrita@chromium.org/14677012/1

[commit-bot: I haz the power, 2013-05-14 02:01:52]

Change committed as 150278