Changeset 62416 in webkit for trunk/JavaScriptCore/wtf/text/StringImpl.h

Timestamp:

Jul 2, 2010, 4:49:07 PM (15 years ago)

Author:

[email protected]

Message:

2010-07-02 Sheriff Bot <​[email protected]>

Unreviewed, rolling out r62410.
​http://trac.webkit.org/changeset/62410
​https://bugs.webkit.org/show_bug.cgi?id=41549

accursed last minute changes (Requested by olliej on #webkit).

• parser/Lexer.cpp: (JSC::Lexer::setCode): (JSC::Lexer::copyCodeWithoutBOMs): (JSC::Lexer::sourceCode):
• parser/SourceProvider.h: (JSC::): (JSC::SourceProvider::SourceProvider): (JSC::SourceProvider::hasBOMs): (JSC::UStringSourceProvider::create): (JSC::UStringSourceProvider::getRange): (JSC::UStringSourceProvider::UStringSourceProvider):
• wtf/text/StringImpl.h:

2010-07-02 Sheriff Bot <​[email protected]>

Unreviewed, rolling out r62410.
​http://trac.webkit.org/changeset/62410
​https://bugs.webkit.org/show_bug.cgi?id=41549

accursed last minute changes (Requested by olliej on #webkit).

• bindings/js/ScriptSourceProvider.h: (WebCore::ScriptSourceProvider::ScriptSourceProvider):
• bindings/js/StringSourceProvider.h: (WebCore::StringSourceProvider::StringSourceProvider):
• loader/CachedScript.cpp: (WebCore::CachedScript::CachedScript): (WebCore::CachedScript::script):
• loader/CachedScript.h:

File:

• trunk/JavaScriptCore/wtf/text/StringImpl.h (modified) (1 diff)

trunk/JavaScriptCore/wtf/text/StringImpl.h

@@ -258 +258 @@
     }
 
-    PassRefPtr<StringImpl> copyStringWithoutBOMs(bool definitelyHasBOMs, bool& hasBOMs)
-    {
-        static const UChar byteOrderMark = 0xFEFF;
-        size_t i = 0;
-        if (!definitelyHasBOMs) {
-            hasBOMs = false;
-            // ECMA-262 calls for stripping all Cf characters, but we only strip BOM characters.
-            // See <https://bugs.webkit.org/show_bug.cgi?id=4931> for details.
-            for (; i < m_length; i++) {
-                if (UNLIKELY(m_data[i] == byteOrderMark)) {
-                    hasBOMs = true;
-                    break;
-                }
-            }
-            if (!hasBOMs)
-                return this;
-        }
-        Vector<UChar> result;
-        result.reserveInitialCapacity(m_length);
-        for (; i < m_length; i++)
-            result.append(m_data[i]);
-        for (; i < m_length; i++) {
-            UChar c = m_data[i];
-            if (c != byteOrderMark)
-                result.append(c);
-        }
-        return StringImpl::adopt(result);
-    }
-
     // Returns a StringImpl suitable for use on another thread.
     PassRefPtr<StringImpl> crossThreadString();