Make Document.createTouch arguments non-optional

third_party/WebKit/Source/core/dom/Document.idl

Index: third_party/WebKit/Source/core/dom/Document.idl
diff --git a/third_party/WebKit/Source/core/dom/Document.idl b/third_party/WebKit/Source/core/dom/Document.idl
index 8b3ae3e71b25e99932fb891d427cfb90e5922f57..11b0b9ef3c2e3e5995182b4da221220d399b1bc7 100644
--- a/third_party/WebKit/Source/core/dom/Document.idl
+++ b/third_party/WebKit/Source/core/dom/Document.idl
@@ -156,15 +156,15 @@ interface Document : Node {
 
     // Touch Events
     // https://w3c.github.io/touch-events/#extensions-to-the-document-interface
-    // FIXME: The arguments should not be optional.
+    // TODO(lunalu): The spec uses the WindowProxy type for view.
     [RuntimeEnabled=TouchEventFeatureDetection, Measure, LegacyInterfaceTypeChecking, Custom=CallPrologue]
-    Touch createTouch([Default=Undefined] optional Window window,
-                      [Default=Undefined] optional EventTarget target,
-                      [Default=Undefined] optional long identifier,
-                      [Default=Undefined] optional unrestricted double pageX,
-                      [Default=Undefined] optional unrestricted double pageY,
-                      [Default=Undefined] optional unrestricted double screenX,
-                      [Default=Undefined] optional unrestricted double screenY,
+    Touch createTouch(Window view,
+                      EventTarget target,
+                      long identifier,
+                      double pageX,
+                      double pageY,
+                      double screenX,
+                      double screenY,
                       [Default=Undefined] optional unrestricted double radiusX,

[foolip, 2017/03/21 08:01:13]

Can you change these to "optional unrestricted double radiusX = 0" and similar
to get rid of the remaining [Default=Undefined] here?

[lunalu1, 2017/03/21 19:19:27]

OK. The spec didn't mention these arguments. So maybe we should confirm with the
someone on the team if this is safe to do? 

[foolip, 2017/03/22 05:16:24]

The generated code will be slightly different, but it'll amount to the same
thing, so it's a safe change to make.

                       [Default=Undefined] optional unrestricted double radiusY,
                       [Default=Undefined] optional unrestricted float rotationAngle,