createTreeWalker and createNodeIterator should deal with null NodeFilter

createTreeWalker and createNodeIterator should deal with null NodeFilter in Document.

spec: http://dom.spec.whatwg.org/#interface-document
According to spec, optional third parameter(NodeFilter)
of createTreeWalker and createNodeIterator can be null.

So this CL modified behavior as per specified.

TEST=LayoutTests/fast/dom/TreeWalker/TreeWalker-basic.html
    ,LayoutTests/fast/dom/NodeIterator/NodeIterator-basic.html

In addition, with this CL,
Optional arguments to createTreeWalker should be optional (3 passed, null) test
at http://w3c-test.org/dom/nodes/Document-createTreeWalker.html will be passed.

Behavior in other browsers.
*)FF: PASS
*)IE: FAIL

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

[kangil_, 2014-07-04 13:36:37]

PTAL

[haraken, 2014-07-04 14:35:43]

LGTM

[Jens Widell, 2014-07-04 16:00:57]

Same goes for Document.createNodeIterator(), and it is also fixed with this CL.
Could be mentioned in the description, and perhaps tested too.

We're effectively making all NodeFilter arguments implicitly nullable here, but
that ought to be fine; no reason to think there will ever be any others.

[kangil_, 2014-07-05 01:59:00]

PTAL, thanks!

[haraken, 2014-07-05 02:02:11]

LGTM

https://codereview.chromium.org/366223006/diff/20001/LayoutTests/fast/dom/Tre...
File LayoutTests/fast/dom/TreeWalker/TreeWalker-basic.html (right):

https://codereview.chromium.org/366223006/diff/20001/LayoutTests/fast/dom/Tre...
LayoutTests/fast/dom/TreeWalker/TreeWalker-basic.html:124: var tw =
document.createTreeWalker(document.body, 42, null);

Nit: tw => treeWalker

Blink prefers a fully qualified name.

[kangil_, 2014-07-05 02:10:50]

Thanks and done!

https://codereview.chromium.org/366223006/diff/20001/LayoutTests/fast/dom/Tre...
File LayoutTests/fast/dom/TreeWalker/TreeWalker-basic.html (right):

https://codereview.chromium.org/366223006/diff/20001/LayoutTests/fast/dom/Tre...
LayoutTests/fast/dom/TreeWalker/TreeWalker-basic.html:124: var tw =
document.createTreeWalker(document.body, 42, null);
On 2014/07/05 02:02:11, haraken wrote:
> 
> Nit: tw => treeWalker
> 
> Blink prefers a fully qualified name.

Done.

[kangil_, 2014-07-05 02:11:14]

The CQ bit was checked by kangil.han@samsung.com

[commit-bot: I haz the power, 2014-07-05 02:11:37]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kangil.han@samsung.com/366223006/40001

[commit-bot: I haz the power, 2014-07-05 03:16:24]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/15550)

[commit-bot: I haz the power, 2014-07-05 04:14:55]

Change committed as 177560