2011-06-30 Noel Gordon <noel.gordon@gmail.com>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 1 Jul 2011 06:06:59 +0000 (06:06 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 1 Jul 2011 06:06:59 +0000 (06:06 +0000)
        Reviewed by Hajime Morita.

        [Chromium] Remove WebBindings::makeNode.
        https://bugs.webkit.org/show_bug.cgi?id=63613

        Dead code elimination. No change in behaviour, so no new tests.

        * public/WebBindings.h:
        * src/WebBindings.cpp:

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

Source/WebKit/chromium/ChangeLog
Source/WebKit/chromium/public/WebBindings.h
Source/WebKit/chromium/src/WebBindings.cpp

index 920cf0a..90ad5b0 100644 (file)
@@ -1,3 +1,15 @@
+2011-06-30  Noel Gordon  <noel.gordon@gmail.com>
+
+        Reviewed by Hajime Morita.
+
+        [Chromium] Remove WebBindings::makeNode.
+        https://bugs.webkit.org/show_bug.cgi?id=63613
+
+        Dead code elimination. No change in behaviour, so no new tests.
+
+        * public/WebBindings.h:
+        * src/WebBindings.cpp:
+
 2011-06-30  Kent Tamura  <tkent@chromium.org>
 
         Reviewed by Darin Fisher.
index 149bff3..5aa4c8d 100644 (file)
@@ -128,6 +128,8 @@ public:
     // undefined.  If iString is false, the opposite is true.
     WEBKIT_API static void extractIdentifierData(const NPIdentifier&, const NPUTF8*& string, int32_t& number, bool& isString);
 
+    // DumpRenderTree support -------------------------------------------------
+
     // Return true (success) if the given npobj is a range object.
     // If so, return that range as a WebRange object.
     WEBKIT_API static bool getRange(NPObject* range, WebRange*);
@@ -138,7 +140,6 @@ public:
 
     WEBKIT_API static NPObject* makeIntArray(const WebVector<int>&);
     WEBKIT_API static NPObject* makeStringArray(const WebVector<WebString>&);
-    WEBKIT_API static NPObject* makeNode(const WebNode&);
 
     // Exceptions -------------------------------------------------------------
 
index caf8eee..4744a64 100644 (file)
@@ -41,7 +41,6 @@
 #include "V8DOMWrapper.h"
 #include "V8Element.h"
 #include "V8NPUtils.h"
-#include "V8Node.h"
 #include "V8Proxy.h"
 #include "V8Range.h"
 #elif USE(JSC)
@@ -257,16 +256,6 @@ static NPObject* makeStringArrayImpl(const WebVector<WebString>& data)
     return npCreateV8ScriptObject(0, result, window);
 }
 
-static NPObject* makeNodeImpl(WebNode data)
-{
-    v8::HandleScope handleScope;
-    if (data.isNull())
-        return 0;
-    v8::Handle<v8::Object> result = V8Node::wrap(data.unwrap<Node>());
-    WebCore::DOMWindow* window = WebCore::V8Proxy::retrieveWindow(WebCore::V8Proxy::currentContext());
-    return npCreateV8ScriptObject(0, result, window);
-}
-
 #endif
 
 bool WebBindings::getRange(NPObject* range, WebRange* webRange)
@@ -309,16 +298,6 @@ NPObject* WebBindings::makeStringArray(const WebVector<WebString>& data)
 #endif
 }
 
-NPObject* WebBindings::makeNode(const WebNode& data)
-{
-#if USE(V8)
-    return makeNodeImpl(data);
-#else
-    // Not supported on other ports (JSC, etc.).
-    return 0;
-#endif
-}
-
 void WebBindings::pushExceptionHandler(ExceptionHandler handler, void* data)
 {
     WebCore::pushExceptionHandler(handler, data);