Back out bug 356083 to fix bustage

js/src/jsobj.c

@@ -832,35 +832,14 @@ js_obj_toSource(JSContext *cx, JSObject *obj, uintN argc, jsval *argv,
     val = argv + 2;
 
     for (i = 0, length = ida->length; i < length; i++) {
-        JSBool idIsLexicalIdentifier, needOldStyleGetterSetter;
-
         /* Get strings for id and value and GC-root them via argv. */
         id = ida->vector[i];
 
 #if JS_HAS_GETTER_SETTER
+
         ok = OBJ_LOOKUP_PROPERTY(cx, obj, id, &obj2, &prop);
         if (!ok)
             goto error;
-#endif
-
-        /*
-         * Convert id to a jsval and then to a string.  Decide early whether we
-         * prefer get/set or old getter/setter syntax.
-         */
-        atom = JSID_IS_ATOM(id) ? JSID_TO_ATOM(id) : NULL;
-        id = ID_TO_VALUE(id);
-        idstr = js_ValueToString(cx, id);
-        if (!idstr) {
-            ok = JS_FALSE;
-            OBJ_DROP_PROPERTY(cx, obj2, prop);
-            goto error;
-        }
-        *rval = STRING_TO_JSVAL(idstr);         /* local root */
-        idIsLexicalIdentifier = js_IsIdentifier(idstr);
-        needOldStyleGetterSetter = !idIsLexicalIdentifier;
-
-#if JS_HAS_GETTER_SETTER
-
         valcnt = 0;
         if (prop) {
             ok = OBJ_GET_ATTRIBUTES(cx, obj2, id, prop, &attrs);
@@ -876,9 +855,8 @@ js_obj_toSource(JSContext *cx, JSObject *obj, uintN argc, jsval *argv,
                     gsop[valcnt] =
                         ATOM_TO_STRING(cx->runtime->atomState.getterAtom);
 #else
-                    gsop[valcnt] = needOldStyleGetterSetter
+                    gsop[valcnt] =
-                        ? ATOM_TO_STRING(cx->runtime->atomState.getterAtom)
+                        ATOM_TO_STRING(cx->runtime->atomState.getAtom);
-                        : ATOM_TO_STRING(cx->runtime->atomState.getAtom);
 #endif
                     valcnt++;
                 }
@@ -888,9 +866,8 @@ js_obj_toSource(JSContext *cx, JSObject *obj, uintN argc, jsval *argv,
                     gsop[valcnt] =
                         ATOM_TO_STRING(cx->runtime->atomState.setterAtom);
 #else
-                    gsop[valcnt] = needOldStyleGetterSetter
+                    gsop[valcnt] =
-                        ? ATOM_TO_STRING(cx->runtime->atomState.setterAtom)
+                        ATOM_TO_STRING(cx->runtime->atomState.setAtom);
-                        : ATOM_TO_STRING(cx->runtime->atomState.setAtom);
 #endif
                     valcnt++;
                 }
@@ -904,12 +881,6 @@ js_obj_toSource(JSContext *cx, JSObject *obj, uintN argc, jsval *argv,
 
 #else  /* !JS_HAS_GETTER_SETTER */
 
-        /*
-         * We simplify the source code at the price of minor dead code bloat in
-         * the ECMA version (for testing only, see jsconfig.h).  The null
-         * default values in gsop[j] suffice to disable non-ECMA getter and
-         * setter code.
-         */
         valcnt = 1;
         gsop[0] = NULL;
         ok = OBJ_GET_PROPERTY(cx, obj, id, &val[0]);
@@ -919,12 +890,22 @@ js_obj_toSource(JSContext *cx, JSObject *obj, uintN argc, jsval *argv,
         if (!ok)
             goto error;
 
+        /* Convert id to a jsval and then to a string. */
+        atom = JSID_IS_ATOM(id) ? JSID_TO_ATOM(id) : NULL;
+        id = ID_TO_VALUE(id);
+        idstr = js_ValueToString(cx, id);
+        if (!idstr) {
+            ok = JS_FALSE;
+            goto error;
+        }
+        *rval = STRING_TO_JSVAL(idstr);         /* local root */
+
         /*
          * If id is a string that's not an identifier, then it needs to be
          * quoted.  Also, negative integer ids must be quoted.
          */
         if (atom
-            ? !idIsLexicalIdentifier
+            ? !js_IsIdentifier(idstr)
             : (JSID_IS_OBJECT(id) || JSID_TO_INT(id) < 0)) {
             idstr = js_QuoteString(cx, idstr, (jschar)'\'');
             if (!idstr) {
@@ -952,8 +933,7 @@ js_obj_toSource(JSContext *cx, JSObject *obj, uintN argc, jsval *argv,
              * Remove '(function ' from the beginning of valstr and ')' from the
              * end so that we can put "get" in front of the function definition.
              */
-            if (gsop[j] && VALUE_IS_FUNCTION(cx, val[j]) &&
+            if (gsop[j] && VALUE_IS_FUNCTION(cx, val[j])) {
-                !needOldStyleGetterSetter) {
                 size_t n = strlen(js_function_str) + 2;
                 JS_ASSERT(vlength > n);
                 vchars += n;
@@ -1035,30 +1015,15 @@ js_obj_toSource(JSContext *cx, JSObject *obj, uintN argc, jsval *argv,
             }
             chars[nchars++] = ':';
 #else
-            if (needOldStyleGetterSetter) {
+            if (gsop[j]) {
-                js_strncpy(&chars[nchars], idstrchars, idstrlength);
+                gsoplength = JSSTRING_LENGTH(gsop[j]);
-                nchars += idstrlength;
+                js_strncpy(&chars[nchars], JSSTRING_CHARS(gsop[j]), gsoplength);
-                if (gsop[j]) {
+                nchars += gsoplength;
-                    chars[nchars++] = ' ';
+                chars[nchars++] = ' ';
-                    gsoplength = JSSTRING_LENGTH(gsop[j]);
-                    js_strncpy(&chars[nchars], JSSTRING_CHARS(gsop[j]),
-                               gsoplength);
-                    nchars += gsoplength;
-                }
-                chars[nchars++] = ':';
-            } else {  /* New style "decompilation" */
-                if (gsop[j]) {
-                    gsoplength = JSSTRING_LENGTH(gsop[j]);
-                    js_strncpy(&chars[nchars], JSSTRING_CHARS(gsop[j]),
-                               gsoplength);
-                    nchars += gsoplength;
-                    chars[nchars++] = ' ';
-                }
-                js_strncpy(&chars[nchars], idstrchars, idstrlength);
-                nchars += idstrlength;
-                /* Extraneous space after id here will be extracted later */
-                chars[nchars++] = gsop[j] ? ' ' : ':';
             }
+            js_strncpy(&chars[nchars], idstrchars, idstrlength);
+            nchars += idstrlength;
+            chars[nchars++] = gsop[j] ? ' ' : ':';
 #endif
             if (vsharplength) {
                 js_strncpy(&chars[nchars], vsharp, vsharplength);

js/src/jsopcode.c

@@ -3860,9 +3860,7 @@ Decompile(SprintStack *ss, jsbytecode *pc, intN nb)
                               rval);
 #else
                 if (lastop == JSOP_GETTER || lastop == JSOP_SETTER) {
-                    if (!ATOM_IS_IDENTIFIER(atom) ||
+                    if (strncmp(rval, js_function_str, 8) || rval[8] != ' ') {
-                        strncmp(rval, js_function_str, 8) ||
-                        rval[8] != ' ') {
                         todo = Sprint(&ss->sprinter, "%s%s%s%s%s:%s", lval,
                                       (lval[1] != '\0') ? ", " : "", xval,
                                       (lastop == JSOP_GETTER ||