Implement IDOMElement::setAttribute
authoraroben <aroben@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Jul 2007 17:16:37 +0000 (17:16 +0000)
committeraroben <aroben@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Jul 2007 17:16:37 +0000 (17:16 +0000)
        This method was mistakenly called "setResult" in DOMCore.idl, so I
        fixed that as well.

        Needed for <rdar://problem/5314906>.

        Reviewed by Ada.

        * DOMCoreClasses.cpp:
        (DOMElement::setAttribute): Implemented/renamed.
        * DOMCoreClasses.h: Renamed setResult => setAttribute.
        * DOMHTMLClasses.h: Ditto.
        * Interfaces/DOMCore.idl: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@24536 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/win/ChangeLog
WebKit/win/DOMCoreClasses.cpp
WebKit/win/DOMCoreClasses.h
WebKit/win/DOMHTMLClasses.h
WebKit/win/Interfaces/DOMCore.idl

index 13e905098f6a0c22bb75f2ffb3d2c5dcfd069108..0ad774bc2df6e7836f3a0c4f00cbbd7df6364894 100644 (file)
@@ -1,3 +1,20 @@
+2007-07-22  Adam Roben  <aroben@apple.com>
+
+        Implement IDOMElement::setAttribute
+
+        This method was mistakenly called "setResult" in DOMCore.idl, so I
+        fixed that as well.
+
+        Needed for <rdar://problem/5314906>.
+
+        Reviewed by Ada.
+
+        * DOMCoreClasses.cpp:
+        (DOMElement::setAttribute): Implemented/renamed.
+        * DOMCoreClasses.h: Renamed setResult => setAttribute.
+        * DOMHTMLClasses.h: Ditto.
+        * Interfaces/DOMCore.idl: Ditto.
+
 2007-07-21  Ada Chan  <adachan@apple.com>
 
         Reviewed by Adam.
index ab9587bb4d920881dd364b898fa1d8841a708d7e..1815171c2b24fcc5c5b8e6daf8dd37ab32f600f0 100644 (file)
@@ -821,12 +821,18 @@ HRESULT STDMETHODCALLTYPE DOMElement::getAttribute(
     return S_OK;
 }
     
-HRESULT STDMETHODCALLTYPE DOMElement::setResult
-        /* [in] */ BSTR /*name*/,
-        /* [in] */ BSTR /*value*/)
+HRESULT STDMETHODCALLTYPE DOMElement::setAttribute
+        /* [in] */ BSTR name,
+        /* [in] */ BSTR value)
 {
-    ASSERT_NOT_REACHED();
-    return E_NOTIMPL;
+    if (!m_element)
+        return E_FAIL;
+
+    WebCore::String nameString(name, SysStringLen(name));
+    WebCore::String valueString(value, SysStringLen(value));
+    WebCore::ExceptionCode ec = 0;
+    m_element->setAttribute(nameString, valueString, ec);
+    return ec ? E_FAIL : S_OK;
 }
     
 HRESULT STDMETHODCALLTYPE DOMElement::removeAttribute( 
index d1a1527b0aec0623ecd20027b11357212720354b..70982e00506dacb8f79ea7653472e21907726428 100644 (file)
@@ -695,7 +695,7 @@ public:
         /* [in] */ BSTR name,
         /* [retval][out] */ BSTR *result);
     
-    virtual HRESULT STDMETHODCALLTYPE setResult
+    virtual HRESULT STDMETHODCALLTYPE setAttribute
         /* [in] */ BSTR name,
         /* [in] */ BSTR value);
     
index 8a684c122ebf608b3bdd39f548e9ab4901a4c995..b36e3f62826253190763855f4ee17d653a83a5bc 100644 (file)
@@ -577,9 +577,9 @@ public:
         /* [in] */ BSTR name,
         /* [retval][out] */ BSTR *result) { return DOMElement::getAttribute(name, result); }
     
-    virtual HRESULT STDMETHODCALLTYPE setResult
+    virtual HRESULT STDMETHODCALLTYPE setAttribute
         /* [in] */ BSTR name,
-        /* [in] */ BSTR value) { return DOMElement::setResult(name, value); }
+        /* [in] */ BSTR value) { return DOMElement::setAttribute(name, value); }
     
     virtual HRESULT STDMETHODCALLTYPE removeAttribute( 
         /* [in] */ BSTR name) { return DOMElement::removeAttribute(name); }
@@ -836,9 +836,9 @@ public:
         /* [in] */ BSTR name,
         /* [retval][out] */ BSTR *result) { return DOMHTMLElement::getAttribute(name, result); }
     
-    virtual HRESULT STDMETHODCALLTYPE setResult
+    virtual HRESULT STDMETHODCALLTYPE setAttribute
         /* [in] */ BSTR name,
-        /* [in] */ BSTR value) { return DOMHTMLElement::setResult(name, value); }
+        /* [in] */ BSTR value) { return DOMHTMLElement::setAttribute(name, value); }
     
     virtual HRESULT STDMETHODCALLTYPE removeAttribute( 
         /* [in] */ BSTR name) { return DOMHTMLElement::removeAttribute(name); }
@@ -1141,9 +1141,9 @@ public:
         /* [in] */ BSTR name,
         /* [retval][out] */ BSTR *result) { return DOMHTMLElement::getAttribute(name, result); }
     
-    virtual HRESULT STDMETHODCALLTYPE setResult
+    virtual HRESULT STDMETHODCALLTYPE setAttribute
         /* [in] */ BSTR name,
-        /* [in] */ BSTR value) { return DOMHTMLElement::setResult(name, value); }
+        /* [in] */ BSTR value) { return DOMHTMLElement::setAttribute(name, value); }
     
     virtual HRESULT STDMETHODCALLTYPE removeAttribute( 
         /* [in] */ BSTR name) { return DOMHTMLElement::removeAttribute(name); }
@@ -1465,9 +1465,9 @@ public:
         /* [in] */ BSTR name,
         /* [retval][out] */ BSTR *result) { return DOMHTMLElement::getAttribute(name, result); }
     
-    virtual HRESULT STDMETHODCALLTYPE setResult
+    virtual HRESULT STDMETHODCALLTYPE setAttribute
         /* [in] */ BSTR name,
-        /* [in] */ BSTR value) { return DOMHTMLElement::setResult(name, value); }
+        /* [in] */ BSTR value) { return DOMHTMLElement::setAttribute(name, value); }
     
     virtual HRESULT STDMETHODCALLTYPE removeAttribute( 
         /* [in] */ BSTR name) { return DOMHTMLElement::removeAttribute(name); }
@@ -1763,9 +1763,9 @@ public:
         /* [in] */ BSTR name,
         /* [retval][out] */ BSTR *result) { return DOMHTMLElement::getAttribute(name, result); }
     
-    virtual HRESULT STDMETHODCALLTYPE setResult
+    virtual HRESULT STDMETHODCALLTYPE setAttribute
         /* [in] */ BSTR name,
-        /* [in] */ BSTR value) { return DOMHTMLElement::setResult(name, value); }
+        /* [in] */ BSTR value) { return DOMHTMLElement::setAttribute(name, value); }
     
     virtual HRESULT STDMETHODCALLTYPE removeAttribute( 
         /* [in] */ BSTR name) { return DOMHTMLElement::removeAttribute(name); }
@@ -2163,9 +2163,9 @@ public:
         /* [in] */ BSTR name,
         /* [retval][out] */ BSTR *result) { return DOMHTMLElement::getAttribute(name, result); }
     
-    virtual HRESULT STDMETHODCALLTYPE setResult
+    virtual HRESULT STDMETHODCALLTYPE setAttribute
         /* [in] */ BSTR name,
-        /* [in] */ BSTR value) { return DOMHTMLElement::setResult(name, value); }
+        /* [in] */ BSTR value) { return DOMHTMLElement::setAttribute(name, value); }
     
     virtual HRESULT STDMETHODCALLTYPE removeAttribute( 
         /* [in] */ BSTR name) { return DOMHTMLElement::removeAttribute(name); }
index 5ad868636d30cc95f2a45f209a35302a278208c5..a00acd226656adbaa17468e1a1397615f6ce10c3 100644 (file)
@@ -547,7 +547,7 @@ interface IDOMElement : IDOMNode
     /*
         - (void)setAttribute:(NSString *)name :(NSString *)value;
     */
-    HRESULT setResult([in] BSTR name, [in] BSTR value);
+    HRESULT setAttribute([in] BSTR name, [in] BSTR value);
 
     /*
         - (void)removeAttribute:(NSString *)name;