[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

commit-queue at webkit.org commit-queue at webkit.org
Wed Dec 22 11:09:36 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit 4082256c9ac01431b71ea0ec64dbbefe5b3f56d1
Author: commit-queue at webkit.org <commit-queue at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Wed Jul 14 01:38:00 2010 +0000

    2010-07-13  Sheriff Bot  <webkit.review.bot at gmail.com>
    
            Unreviewed, rolling out r63162.
            http://trac.webkit.org/changeset/63162
            https://bugs.webkit.org/show_bug.cgi?id=42224
    
            This is no longer needed (Requested by olliej_ on #webkit).
    
            * bindings/v8/ScriptSourceCode.h:
            (WebCore::ScriptSourceCode::ScriptSourceCode):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@63275 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 3170f38..326349b 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,14 @@
+2010-07-13  Sheriff Bot  <webkit.review.bot at gmail.com>
+
+        Unreviewed, rolling out r63162.
+        http://trac.webkit.org/changeset/63162
+        https://bugs.webkit.org/show_bug.cgi?id=42224
+
+        This is no longer needed (Requested by olliej_ on #webkit).
+
+        * bindings/v8/ScriptSourceCode.h:
+        (WebCore::ScriptSourceCode::ScriptSourceCode):
+
 2010-07-13  Eric Seidel  <eric at webkit.org>
 
         Reviewed by Adam Barth.
diff --git a/WebCore/bindings/v8/ScriptSourceCode.h b/WebCore/bindings/v8/ScriptSourceCode.h
index 485daea..dbc9d5e 100644
--- a/WebCore/bindings/v8/ScriptSourceCode.h
+++ b/WebCore/bindings/v8/ScriptSourceCode.h
@@ -46,14 +46,6 @@ public:
         , m_url(url)
         , m_startLine(startLine)
     {
-        // FIXME: Copying the source to strip BOMs isn't strictly necessary
-        // because V8 treats BOMs as whitespace. However, it is here because it
-        // must be in sync with CachedScript::script() which strips the BOMs. We
-        // should investigate the performance implications.
-        if (m_source.length()) { 
-            bool scratch = false; 
-            m_source = String(source.impl()->copyStringWithoutBOMs(false, scratch)); 
-        } 
     }
 
     // We lose the encoding information from CachedScript.

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list