Sync the HTMLElement interface with the spec

Source/core/html/HTMLElement.idl

Index: Source/core/html/HTMLElement.idl
diff --git a/Source/core/html/HTMLElement.idl b/Source/core/html/HTMLElement.idl
index 85598835381da3affc0d2cc9ed53b4e64e96a3a2..79d2aa09664d2ae61efb730f6b29dab22fa5ec09 100644
--- a/Source/core/html/HTMLElement.idl
+++ b/Source/core/html/HTMLElement.idl
@@ -18,32 +18,39 @@
  * Boston, MA 02110-1301, USA.
  */
 
+// https://html.spec.whatwg.org/#htmlelement
+
 interface HTMLElement : Element {
+    // metadata attributes
     [Reflect] attribute DOMString title;
     [Reflect] attribute DOMString lang;
     attribute boolean translate;
     attribute DOMString dir;
+    // FIXME: dataset is on HTMLElement

[Jens Widell, 2015/03/06 19:04:22]

Don't understand. This is HTMLElement. Where is dataset? 

[philipj_slow, 2015/03/07 15:20:24]

Oops, just copypasta, it's on Element.

 
-    [CustomElementCallbacks] attribute long              tabIndex;
-    [CustomElementCallbacks] attribute boolean           draggable;
-    [Reflect, TreatNullAs=NullString, MeasureAs=PrefixedHTMLElementDropzone] attribute DOMString webkitdropzone;
+    // user interaction
     [Reflect] attribute boolean hidden;
+    void click();
+    [CustomElementCallbacks] attribute long tabIndex;
+    // FIXME: focus() and blur() are on Element.
     [Reflect] attribute DOMString accessKey;
+    [CustomElementCallbacks] attribute boolean draggable;
+    [RuntimeEnabled=ContextMenu] attribute HTMLMenuElement? contextMenu;
+    attribute boolean spellcheck;
 
-    // Extensions
-    [TreatNullAs=NullString, CustomElementCallbacks, RaisesException=Setter, MeasureAs=HTMLElementInnerText] attribute DOMString innerText;
-    [TreatNullAs=NullString, CustomElementCallbacks, RaisesException=Setter, MeasureAs=HTMLElementOuterText] attribute DOMString outerText;
-
-    [RuntimeEnabled=IMEAPI] readonly attribute InputMethodContext inputMethodContext;
-
+    // HTMLElement implements ElementContentEditable
+    // https://html.spec.whatwg.org/#contenteditable
     [CustomElementCallbacks, RaisesException=Setter] attribute DOMString contentEditable;
     readonly attribute boolean isContentEditable;
-    [RuntimeEnabled=ContextMenu] attribute HTMLMenuElement? contextMenu;
 
-             attribute boolean spellcheck;
+    // Input Method Editor API
+    // https://dvcs.w3.org/hg/ime-api/raw-file/default/Overview.html#the-inputmethodcontext-property
+    [RuntimeEnabled=IMEAPI] readonly attribute InputMethodContext? inputMethodContext;
 
-    void click();
+    // Non-standard APIs
+    [TreatNullAs=NullString, CustomElementCallbacks, RaisesException=Setter, MeasureAs=HTMLElementInnerText] attribute DOMString innerText;
+    [TreatNullAs=NullString, CustomElementCallbacks, RaisesException=Setter, MeasureAs=HTMLElementOuterText] attribute DOMString outerText;
+    [Reflect, TreatNullAs=NullString, MeasureAs=PrefixedHTMLElementDropzone] attribute DOMString webkitdropzone;

[Jens Widell, 2015/03/06 19:04:22]

Should perhaps be nullable instead of TreatNullAs=NullString? 

[philipj_slow, 2015/03/07 15:20:24]

That actually changes the generated code from v8SetReturnValueString to
v8SetReturnValueStringOrNull, which makes the attribute null instead of "" when
the content attribute it not present. Probably best to leave this alone and do
it right with the unprefixed attribute.

 };
 
 HTMLElement implements GlobalEventHandlers;
-