select.add() should support optgroup adding and before index

Source/core/html/HTMLSelectElement.h

Index: Source/core/html/HTMLSelectElement.h
diff --git a/Source/core/html/HTMLSelectElement.h b/Source/core/html/HTMLSelectElement.h
index 8268cc15a8c3508e851f1278dbdbc0c98d14753c..e5dd1ed32aa204f6a3d5bb5801b15d86046d44db 100644
--- a/Source/core/html/HTMLSelectElement.h
+++ b/Source/core/html/HTMLSelectElement.h
@@ -63,6 +63,7 @@ public:
     bool usesMenuList() const;
 
     void add(HTMLElement*, HTMLElement* beforeElement, ExceptionState&);
+    void addBeforeOptionAtIndex(HTMLElement*, unsigned beforeIndex, ExceptionState&);

[tkent, 2014/03/27 01:18:53]

Is ImplementedAs necessary?  Doesn't an overload add(HTMLElement*, int,
ExceptionState&) work?

[keishi, 2014/03/27 06:06:53]

I tried that and the compiler said it doesn't know which to call for:
select.add(newOption, 0, exceptionState);
Since it does different things (one is append, the other is prepend) I thought
better to add ImplementAs.

 
     using Node::remove;
     void remove(int index);