[SCM] WebKit Debian packaging branch, debian/experimental, updated. debian/1.3.8-1-1049-g2e11a8e

antonm at chromium.org antonm at chromium.org
Fri Jan 21 14:47:22 UTC 2011


The following commit has been merged in the debian/experimental branch:
commit aeb83933aa4d95ad220145258df7e2607296efb3
Author: antonm at chromium.org <antonm at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Thu Dec 30 04:07:01 2010 +0000

    2010-12-29  Anton Muhin  <antonm at chromium.org>
    
            Reviewed by Eric Seidel.
    
            [v8] Minor cleanup: remove unused method (the only method which can mutate context of V8DOMWindowShell)
            https://bugs.webkit.org/show_bug.cgi?id=51704
    
            * bindings/v8/V8DOMWindowShell.cpp:
            * bindings/v8/V8DOMWindowShell.h:
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@74778 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 115159b..ae6f753 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,13 @@
+2010-12-29  Anton Muhin  <antonm at chromium.org>
+
+        Reviewed by Eric Seidel.
+
+        [v8] Minor cleanup: remove unused method (the only method which can mutate context of V8DOMWindowShell)
+        https://bugs.webkit.org/show_bug.cgi?id=51704
+
+        * bindings/v8/V8DOMWindowShell.cpp:
+        * bindings/v8/V8DOMWindowShell.h:
+
 2010-12-29  Kenichi Ishibashi  <bashi at google.com>
 
         Reviewed by Darin Adler.
diff --git a/WebCore/bindings/v8/V8DOMWindowShell.cpp b/WebCore/bindings/v8/V8DOMWindowShell.cpp
index 80bf683..c1d2933 100644
--- a/WebCore/bindings/v8/V8DOMWindowShell.cpp
+++ b/WebCore/bindings/v8/V8DOMWindowShell.cpp
@@ -349,16 +349,6 @@ v8::Persistent<v8::Context> V8DOMWindowShell::createNewContext(v8::Handle<v8::Ob
     return result;
 }
 
-void V8DOMWindowShell::setContext(v8::Handle<v8::Context> context)
-{
-    // if we already have a context, clear it before setting the new one.
-    if (!m_context.IsEmpty()) {
-        m_context.Dispose();
-        m_context.Clear();
-    }
-    m_context = v8::Persistent<v8::Context>::New(context);
-}
-
 bool V8DOMWindowShell::installDOMWindow(v8::Handle<v8::Context> context, DOMWindow* window)
 {
     // Create a new JS window object and use it as the prototype for the  shadow global object.
diff --git a/WebCore/bindings/v8/V8DOMWindowShell.h b/WebCore/bindings/v8/V8DOMWindowShell.h
index 76c27af..84d14b9 100644
--- a/WebCore/bindings/v8/V8DOMWindowShell.h
+++ b/WebCore/bindings/v8/V8DOMWindowShell.h
@@ -66,8 +66,7 @@ public:
     bool isContextInitialized();
 
     v8::Persistent<v8::Context> createNewContext(v8::Handle<v8::Object> global, int extensionGroup);
-    void setContext(v8::Handle<v8::Context>);
-    static bool installDOMWindow(v8::Handle<v8::Context> context, DOMWindow*);
+    static bool installDOMWindow(v8::Handle<v8::Context>, DOMWindow*);
 
     bool initContextIfNeeded();
     void updateDocumentWrapper(v8::Handle<v8::Object> wrapper);

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list