Make sure the root node of selector queries is a ContainerNode

Make sure the root node of selector queries is a ContainerNode

Make sure the root node of selector queries is a ContainerNode by moving the
querySelector*() methods from the Node class to the ContainerNode class. These
methods are exposed to JS on Document and Element, both of which are
container nodes.

Doing this gets rid of an unnecessary container node check in SelectorQuery
and also enables using the slightly faster firstWithin(ContainerNode) over
firstWithin(Node) traversal method.

R=eseidel, esprehn, adamk

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

[Inactive, 2014-01-22 15:10:03]

I rebased the patch. Any opinion on this patch?

This is similar to my other patch for LiveNodeListBase:
https://codereview.chromium.org/132923003/

[Inactive, 2014-01-24 02:50:03]

+adamk

[esprehn, 2014-01-24 03:04:10]

rsgtm

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[Inactive, 2014-01-24 14:04:02]

On 2014/01/24 03:04:10, esprehn wrote:
> rsgtm
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:blink-reviews+unsubscribe@chromium.org.

Esprehn, you misspelled your rslgtm :)

Adamk, could you please take a look at the changes in web/?

[esprehn, 2014-01-24 16:25:01]

rslgtm

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[adamk, 2014-01-24 20:01:53]

lgtm for Source/web/

[commit-bot: I haz the power, 2014-01-24 20:02:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/138743008/160001

[commit-bot: I haz the power, 2014-01-24 22:20:53]

Change committed as 165783