The node and allowPartialContainment arguments for Selection API should not be optional

Source/core/editing/Selection.idl

Index: Source/core/editing/Selection.idl
diff --git a/Source/core/editing/Selection.idl b/Source/core/editing/Selection.idl
index a7ad07b414cbc02147f84e07a9b0386b600e5cb4..715e54303add2e08753795562bfdd49cc3d7300b 100644
--- a/Source/core/editing/Selection.idl
+++ b/Source/core/editing/Selection.idl
@@ -60,8 +60,7 @@
                                                                                  [Default=Undefined] optional long extentOffset);
     [RaisesException, TypeChecking=Interface] void selectAllChildren(Node node);
     [CustomElementCallbacks] void deleteFromDocument();
-    // TODO(philipj): The arguments should not be optional.
-    [MeasureAs=SelectionContainsNode] boolean containsNode([Default=Undefined] optional Node node, [Default=Undefined] optional boolean allowPartialContainment);
+    [MeasureAs=SelectionContainsNode, TypeChecking=Interface] boolean containsNode(Node node, [Default=Undefined] optional boolean allowPartialContainment);

[philipj_slow, 2015/08/31 14:48:21]

Please remove [Default=Undefined] use "optional boolean allowPartialContainment
= false" instead.

[tanay.c, 2015/09/01 10:58:08]

Done.

     // TODO(philipj): The spec does not use [NotEnumerable]. See also:
     // https://codereview.chromium.org/345983004/
     // https://www.w3.org/Bugs/Public/show_bug.cgi?id=26179