Changeset 221402 in webkit for trunk/Source/JavaScriptCore/runtime/JSString.h

Timestamp:

Aug 30, 2017, 3:48:08 PM (8 years ago)

Author:

Matt Lewis

Message:

Unreviewed, rolling out r221384.

This patch caused multiple 32-bit JSC test failures.

Reverted changeset:

"Strings need to be in some kind of gigacage"
​https://bugs.webkit.org/show_bug.cgi?id=174924
​http://trac.webkit.org/changeset/221384

File:

• trunk/Source/JavaScriptCore/runtime/JSString.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/runtime/JSString.h

@@ -140 +140 @@
     static JSString* create(VM& vm, Ref<StringImpl>&& value)
     {
-        value->assertCaged();
         unsigned length = value->length();
         size_t cost = value->cost();
@@ -149 +148 @@
     static JSString* createHasOtherOwner(VM& vm, Ref<StringImpl>&& value)
     {
-        value->assertCaged();
         size_t length = value->length();
         JSString* newString = new (NotNull, allocateCell<JSString>(vm.heap)) JSString(vm, WTFMove(value));