Sync the DOMImplementation interface with the spec

Source/core/dom/DOMImplementation.idl

Index: Source/core/dom/DOMImplementation.idl
diff --git a/Source/core/dom/DOMImplementation.idl b/Source/core/dom/DOMImplementation.idl
index 327543a03d4e729cc9ff63ff164f170db8fcb66a..96dfc38ad3082eabb3c43395ebfe543693f1a76e 100644
--- a/Source/core/dom/DOMImplementation.idl
+++ b/Source/core/dom/DOMImplementation.idl
@@ -18,27 +18,19 @@
  * Boston, MA 02110-1301, USA.
  */
 
+// https://dom.spec.whatwg.org/#interface-domimplementation
+
 [
     SetWrapperReferenceFrom=document,
     TypeChecking=Interface,
     WillBeGarbageCollected,
 ] interface DOMImplementation {
+    [RaisesException] DocumentType createDocumentType(DOMString qualifiedName, DOMString publicId, DOMString systemId);
+    [RaisesException] XMLDocument createDocument(DOMString? namespaceURI, [TreatNullAs=EmptyString] DOMString qualifiedName, optional DocumentType? doctype = null);

[haraken, 2015/02/25 17:25:41]

I'm wondering if we need '= null'. I guess it's just redundant. This is a
question not to your CL but to the spec though.

Jens?

[Jens Widell, 2015/02/25 17:32:56]

"= null" says "treat missing the same as if null was passed". The spec prose
then only needs to explain two cases: "null" and "a DocumentType object".

Without "= null", the spec prose would also need to explain the "missing" case.
(Probably by saying "if missing or null, do X".)

In Blink, we'd need to explicitly implement the "missing" case to behave the
same as null as well, without "= null". Now the implementation just sees a
pointer, always with a valid value, possibly null.

Everybody wins. :-)

+    // FIXME: createHTMLDocument should return a Document. crbug.com/238368
+    HTMLDocument createHTMLDocument(optional DOMString title);
 
-    // DOM Level 1
-
+    // FIXME: hasFeature should take no arguments and always return true. crbug.com/429536
     [ImplementedAs=hasFeatureForBindings, MeasureAs=DOMImplementationHasFeature]
     boolean hasFeature(DOMString feature, DOMString? version);
-
-    // DOM Level 2
-
-    [RaisesException] DocumentType createDocumentType(DOMString qualifiedName,
-                                                      DOMString publicId,
-                                                      DOMString systemId);
-    [RaisesException] XMLDocument createDocument(DOMString? namespaceURI,
-                                                 [TreatNullAs=EmptyString] DOMString qualifiedName,
-                                                 optional DocumentType? doctype = null);
-
-    // HTMLDOMImplementation interface from DOM Level 2 HTML
-
-    HTMLDocument createHTMLDocument(optional DOMString title = null);
 };