Changeset 145500 in webkit

Timestamp:

Mar 12, 2013 1:53:41 AM (11 years ago)

Author:

Csaba Osztrogonác

Message:

REGRESSION(r145482): It broke 33 jsc tests and zillion layout tests on all platform
​https://bugs.webkit.org/show_bug.cgi?id=112112

Reviewed by Oliver Hunt.

Rolling out ​https://trac.webkit.org/changeset/145482 to unbreak the bots.

• runtime/JSStringJoiner.cpp:

(JSC::JSStringJoiner::build):

• runtime/JSStringJoiner.h:

(JSStringJoiner):
(JSC::JSStringJoiner::JSStringJoiner):
(JSC::JSStringJoiner::append):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• runtime/JSStringJoiner.cpp (modified) (2 diffs)
• runtime/JSStringJoiner.h (modified) (3 diffs)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-03-12  Csaba Osztrogonác  <ossy@webkit.org>
+
+        REGRESSION(r145482): It broke 33 jsc tests and zillion layout tests on all platform
+        https://bugs.webkit.org/show_bug.cgi?id=112112
+
+        Reviewed by Oliver Hunt.
+
+        Rolling out https://trac.webkit.org/changeset/145482 to unbreak the bots.
+
+        * runtime/JSStringJoiner.cpp:
+        (JSC::JSStringJoiner::build):
+        * runtime/JSStringJoiner.h:
+        (JSStringJoiner):
+        (JSC::JSStringJoiner::JSStringJoiner):
+        (JSC::JSStringJoiner::append):
+
 2013-03-12  Filip Pizlo  <fpizlo@apple.com>
 

trunk/Source/JavaScriptCore/runtime/JSStringJoiner.cpp

@@ -103 +103 @@
         return jsEmptyString(exec);
 
-    Checked<size_t, RecordOverflow> separatorLength = m_separator.length();
+    size_t separatorLength = m_separator.length();
     // FIXME: add special cases of joinStrings() for (separatorLength == 0) and (separatorLength == 1).
     ASSERT(m_strings.size() > 0);
-    Checked<size_t, RecordOverflow> totalSeparactorsLength = separatorLength * (m_strings.size() - 1);
-    Checked<size_t, RecordOverflow> outputStringSize = totalSeparactorsLength + m_accumulatedStringsLength;
+    size_t totalSeparactorsLength = separatorLength * (m_strings.size() - 1);
+    size_t outputStringSize = totalSeparactorsLength + m_cumulatedStringsLength;
 
-    size_t finalSize;
-    if (outputStringSize.safeGet(finalSize))
-        return throwOutOfMemoryError(exec);
-        
     if (!outputStringSize)
         return jsEmptyString(exec);
@@ -118 +114 @@
     RefPtr<StringImpl> outputStringImpl;
     if (m_is8Bits)
-        outputStringImpl = joinStrings<LChar>(m_strings, m_separator, finalSize);
+        outputStringImpl = joinStrings<LChar>(m_strings, m_separator, outputStringSize);
     else
-        outputStringImpl = joinStrings<UChar>(m_strings, m_separator, finalSize);
+        outputStringImpl = joinStrings<UChar>(m_strings, m_separator, outputStringSize);
 
     if (!outputStringImpl)

trunk/Source/JavaScriptCore/runtime/JSStringJoiner.h

@@ -47 +47 @@
     Vector<String> m_strings;
 
-    Checked<unsigned, RecordOverflow> m_accumulatedStringsLength;
+    unsigned m_cumulatedStringsLength;
     bool m_isValid;
     bool m_is8Bits;
@@ -54 +54 @@
 inline JSStringJoiner::JSStringJoiner(const String& separator, size_t stringCount)
     : m_separator(separator)
+    , m_cumulatedStringsLength(0)
     , m_isValid(true)
     , m_is8Bits(m_separator.is8Bit())
@@ -66 +67 @@
         return;
 
-    m_strings.append(str);
+    m_strings.uncheckedAppend(str);
     if (!str.isNull()) {
-        m_accumulatedStringsLength += str.length();
+        m_cumulatedStringsLength += str.length();
         m_is8Bits = m_is8Bits && str.is8Bit();
     }