From 3c53ecca56d08c9b44445b1fae18fd35250a9ad1 Mon Sep 17 00:00:00 2001
From: David Mandelin <dmandelin@mozilla.com>
Date: Thu, 28 Oct 2010 17:44:24 -0700
Subject: [PATCH] Backed out changeset c133d3c084c0: now we want to measure if
 not doing sync stuff on background thread reduces crashes.

---
 js/src/jsstr.cpp | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/js/src/jsstr.cpp b/js/src/jsstr.cpp
index b3beae68ffd..925bf319fe6 100644
--- a/js/src/jsstr.cpp
+++ b/js/src/jsstr.cpp
@@ -104,18 +104,6 @@ js_GetStringChars(JSContext *cx, JSString *str)
 void
 JSString::flatten()
 {
-    // Diagnostic: serialize all calls to this function to see
-    // if concurrent calls are crashing us.
-    JS_LOCK_RUNTIME(asCell()->compartment()->rt);
-    // The main body of this function can be executed only if
-    // the string is a rope. With multiple threads, it's possible
-    // we waited while another one ran, and the string has
-    // already been flattened for us.
-    if (!isRope()) {
-        JS_UNLOCK_RUNTIME(asCell()->compartment()->rt);
-        return;
-    }
-
     JSString *topNode;
     jschar *chars;
     size_t capacity;
@@ -193,8 +181,6 @@ JSString::flatten()
     /* Set null terminator. */
     chars[pos] = 0;
     topNode->initFlatMutable(chars, pos, capacity);
-
-    JS_UNLOCK_RUNTIME(asCell()->compartment()->rt);
 }
 
 #ifdef JS_TRACER