Title: [128382] trunk/Source/WebCore
Revision
128382
Author
[email protected]
Date
2012-09-12 16:44:33 -0700 (Wed, 12 Sep 2012)

Log Message

[V8] V8DOMWrapper::perContextData has no callers and can be removed
https://bugs.webkit.org/show_bug.cgi?id=96574

Reviewed by Kentaro Hara.

* bindings/v8/V8DOMWrapper.cpp:
* bindings/v8/V8DOMWrapper.h:
(WebCore::V8DOMWrapper::getCachedWrapper):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (128381 => 128382)


--- trunk/Source/WebCore/ChangeLog	2012-09-12 23:36:35 UTC (rev 128381)
+++ trunk/Source/WebCore/ChangeLog	2012-09-12 23:44:33 UTC (rev 128382)
@@ -1,3 +1,14 @@
+2012-09-12  Adam Barth  <[email protected]>
+
+        [V8] V8DOMWrapper::perContextData has no callers and can be removed
+        https://bugs.webkit.org/show_bug.cgi?id=96574
+
+        Reviewed by Kentaro Hara.
+
+        * bindings/v8/V8DOMWrapper.cpp:
+        * bindings/v8/V8DOMWrapper.h:
+        (WebCore::V8DOMWrapper::getCachedWrapper):
+
 2012-09-12  Alexandru Chiculita  <[email protected]>
 
         [CSS Shaders] Implement transform parameter animations for CSS Custom Filters

Modified: trunk/Source/WebCore/bindings/v8/V8DOMWrapper.cpp (128381 => 128382)


--- trunk/Source/WebCore/bindings/v8/V8DOMWrapper.cpp	2012-09-12 23:36:35 UTC (rev 128381)
+++ trunk/Source/WebCore/bindings/v8/V8DOMWrapper.cpp	2012-09-12 23:44:33 UTC (rev 128382)
@@ -86,15 +86,6 @@
     return wrapperHandle;
 }
 
-#if ENABLE(WORKERS)
-V8PerContextData* V8DOMWrapper::perContextData(WorkerContext*)
-{
-    WorkerScriptController* controller = WorkerScriptController::controllerForContext();
-    WorkerContextExecutionProxy* proxy = controller ? controller->proxy() : 0;
-    return proxy ? proxy->perContextData() : 0;
-}
-#endif
-
 void V8DOMWrapper::setNamedHiddenReference(v8::Handle<v8::Object> parent, const char* name, v8::Handle<v8::Value> child)
 {
     ASSERT(name);

Modified: trunk/Source/WebCore/bindings/v8/V8DOMWrapper.h (128381 => 128382)


--- trunk/Source/WebCore/bindings/v8/V8DOMWrapper.h	2012-09-12 23:36:35 UTC (rev 128381)
+++ trunk/Source/WebCore/bindings/v8/V8DOMWrapper.h	2012-09-12 23:44:33 UTC (rev 128382)
@@ -129,10 +129,6 @@
             DOMNodeMapping& domNodeMap = node->isActiveNode() ? store->activeDomNodeMap() : store->domNodeMap();
             return domNodeMap.get(node);
         }
-    private:
-#if ENABLE(WORKERS)
-        static V8PerContextData* perContextData(WorkerContext*);
-#endif
     };
 
     template<typename T>

Modified: trunk/Source/WebCore/bindings/v8/WorkerContextExecutionProxy.h (128381 => 128382)


--- trunk/Source/WebCore/bindings/v8/WorkerContextExecutionProxy.h	2012-09-12 23:36:35 UTC (rev 128381)
+++ trunk/Source/WebCore/bindings/v8/WorkerContextExecutionProxy.h	2012-09-12 23:44:33 UTC (rev 128382)
@@ -77,8 +77,6 @@
         // Returns a local handle of the context.
         v8::Local<v8::Context> context() { return v8::Local<v8::Context>::New(m_context); }
 
-        V8PerContextData* perContextData() { return m_perContextData.get(); }
-
     private:
         void initIsolate();
         bool initializeIfNeeded();
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to