Title: [125478] trunk/Source/WebCore
Revision
125478
Author
[email protected]
Date
2012-08-13 17:42:36 -0700 (Mon, 13 Aug 2012)

Log Message

[V8] Remove unused methods and headers from V8Utilities
https://bugs.webkit.org/show_bug.cgi?id=93805

Reviewed by Adam Barth.

This patch removes unused methods, forward declarations and
headers from V8Utilities.{h,cpp}.

No tests. No change in behavior.

* bindings/v8/V8Utilities.cpp:
* bindings/v8/V8Utilities.h:
(WebCore):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (125477 => 125478)


--- trunk/Source/WebCore/ChangeLog	2012-08-14 00:40:51 UTC (rev 125477)
+++ trunk/Source/WebCore/ChangeLog	2012-08-14 00:42:36 UTC (rev 125478)
@@ -1,5 +1,21 @@
 2012-08-13  Kentaro Hara  <[email protected]>
 
+        [V8] Remove unused methods and headers from V8Utilities
+        https://bugs.webkit.org/show_bug.cgi?id=93805
+
+        Reviewed by Adam Barth.
+
+        This patch removes unused methods, forward declarations and
+        headers from V8Utilities.{h,cpp}.
+
+        No tests. No change in behavior.
+
+        * bindings/v8/V8Utilities.cpp:
+        * bindings/v8/V8Utilities.h:
+        (WebCore):
+
+2012-08-13  Kentaro Hara  <[email protected]>
+
         [V8] Move V8PerIsolateData::visitExternalStrings() to V8PerIsolateData.cpp
         https://bugs.webkit.org/show_bug.cgi?id=93803
 

Modified: trunk/Source/WebCore/bindings/v8/V8Utilities.cpp (125477 => 125478)


--- trunk/Source/WebCore/bindings/v8/V8Utilities.cpp	2012-08-14 00:40:51 UTC (rev 125477)
+++ trunk/Source/WebCore/bindings/v8/V8Utilities.cpp	2012-08-14 00:42:36 UTC (rev 125478)
@@ -35,7 +35,6 @@
 #include "Document.h"
 #include "ExceptionCode.h"
 #include "Frame.h"
-#include "GenericBinding.h"
 #include "MessagePort.h"
 #include "ScriptExecutionContext.h"
 #include "ScriptState.h"
@@ -44,10 +43,8 @@
 #include "V8MessagePort.h"
 #include "V8Proxy.h"
 #include "WorkerContext.h"
-#include "WorkerContextExecutionProxy.h"
 #include <v8.h>
 #include <wtf/ArrayBuffer.h>
-#include <wtf/Assertions.h>
 
 namespace WebCore {
 
@@ -168,16 +165,6 @@
         createHiddenDependency(object, newValue, cacheIndex);
 }
 
-Frame* callingOrEnteredFrame()
-{
-    return activeFrame(BindingState::instance());
-}
-
-KURL completeURL(const String& relativeURL)
-{
-    return completeURL(BindingState::instance(), relativeURL);
-}
-
 ScriptExecutionContext* getScriptExecutionContext()
 {
 #if ENABLE(WORKERS)

Modified: trunk/Source/WebCore/bindings/v8/V8Utilities.h (125477 => 125478)


--- trunk/Source/WebCore/bindings/v8/V8Utilities.h	2012-08-14 00:40:51 UTC (rev 125477)
+++ trunk/Source/WebCore/bindings/v8/V8Utilities.h	2012-08-14 00:42:36 UTC (rev 125478)
@@ -34,8 +34,6 @@
 #include <wtf/Forward.h>
 #include <v8.h>
 
-#include "OwnHandle.h"
-
 namespace WTF {
 class ArrayBuffer;
 }
@@ -43,11 +41,8 @@
 namespace WebCore {
 
     class EventListener;
-    class Frame;
-    class KURL;
     class MessagePort;
     class ScriptExecutionContext;
-    class ScriptState;
 
     // Use an array to hold dependents. It works like a ref-counted scheme. A value can be added more than once to the DOM object.
     void createHiddenDependency(v8::Handle<v8::Object>, v8::Local<v8::Value>, int cacheIndex);
@@ -56,8 +51,6 @@
     // Combo create/remove, for generated event-handler-setter bindings:
     void transferHiddenDependency(v8::Handle<v8::Object>, EventListener* oldValue, v8::Local<v8::Value> newValue, int cacheIndex);
 
-    KURL completeURL(const String& relativeURL);
-
     ScriptExecutionContext* getScriptExecutionContext();
 
     void setTypeMismatchException(v8::Isolate*);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to