Conversion of any, NodeFilter and XPathNSResolver arguments is trivial

Conversion of any, NodeFilter and XPathNSResolver arguments is trivial

The 'any' type is converted to by constructing a ScriptValue object, which
simply wraps the v8::Value.

The 'NodeFilter' and 'XPathNSResolver' types are converted to by wrapping
the v8::Value in custom objects, with "soft" type checks (null is returned
on failure.)

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

[Jens Widell, 2014-09-24 14:39:45]

PTAL

[Jens Widell, 2014-09-24 14:40:08]

jl@opera.com changed reviewers:
+ haraken@chromium.org

[Jens Widell, 2014-09-24 14:40:09]

PTAL

[haraken, 2014-09-24 14:43:26]

LGTM

(Though we should get rid of the special-casing for NodeFilter and
XPathNSResolver.)

[Jens Widell, 2014-09-24 14:51:58]

On 2014/09/24 14:43:26, haraken wrote:
> (Though we should get rid of the special-casing for NodeFilter and
> XPathNSResolver.)

Yes, I agree.

BTW, once I've covered all specific conversions, I'll clean up by removing the
v8::TryCatch, and then I think I'll end up removing this "is trivial" logic too;
it'll necessarily be the default (with strings and "uses exceptionState" being
the non-default options.)

[Jens Widell, 2014-09-24 14:52:23]

The CQ bit was checked by jl@opera.com

[commit-bot: I haz the power, 2014-09-24 14:53:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/600833003/1

[haraken, 2014-09-24 14:56:11]

On 2014/09/24 14:51:58, Jens Widell wrote:
> On 2014/09/24 14:43:26, haraken wrote:
> > (Though we should get rid of the special-casing for NodeFilter and
> > XPathNSResolver.)
> 
> Yes, I agree.
> 
> BTW, once I've covered all specific conversions, I'll clean up by removing the
> v8::TryCatch, and then I think I'll end up removing this "is trivial" logic
too;
> it'll necessarily be the default (with strings and "uses exceptionState" being
> the non-default options.)

That's cool!

[commit-bot: I haz the power, 2014-09-24 16:29:44]

Committed patchset #1 (id:1) as 182612