[bindings] Remove custom binding usage from XMLHttpRequest.open()

Source/core/xmlhttprequest/XMLHttpRequest.h

Index: Source/core/xmlhttprequest/XMLHttpRequest.h
diff --git a/Source/core/xmlhttprequest/XMLHttpRequest.h b/Source/core/xmlhttprequest/XMLHttpRequest.h
index 7747e3e89aca776ca84fa32f1ab58fce27146f76..26761e449cc7a3f737e047d15b1fe641fbd8a383 100644
--- a/Source/core/xmlhttprequest/XMLHttpRequest.h
+++ b/Source/core/xmlhttprequest/XMLHttpRequest.h
@@ -112,10 +112,11 @@ public:
     State readyState() const;
     bool withCredentials() const { return m_includeCredentials; }
     void setWithCredentials(bool, ExceptionState&);
-    void open(const AtomicString& method, const KURL&, ExceptionState&);
+    void open(const AtomicString& method, const AtomicString& url, ExceptionState&);

[yhirano, 2015/03/24 09:10:46]

Why do you use AtomicString for URLs?
AtomicString has a static table, so using it for URLs doesn't look good.

+    void open(const AtomicString& method, const AtomicString& url, bool async, ExceptionState&);
+    void open(const AtomicString& method, const AtomicString& url, bool async, const String& user, ExceptionState&);
+    void open(const AtomicString& method, const AtomicString& url, bool async, const String& user, const String& password, ExceptionState&);
     void open(const AtomicString& method, const KURL&, bool async, ExceptionState&);
-    void open(const AtomicString& method, const KURL&, bool async, const String& user, ExceptionState&);
-    void open(const AtomicString& method, const KURL&, bool async, const String& user, const String& password, ExceptionState&);
     void send(ExceptionState&);
     void send(const ArrayBufferOrArrayBufferViewOrBlobOrDocumentOrStringOrFormData&, ExceptionState&);
     void abort();