Created attachment 578480 [details] [diff] [review]
Spec uses ToString, we should too -- it's nicer to read, to boot. I also went through all js_ValueToString uses and converted them to ToString if their argument might be a string and ToStringSlow if it definitely wasn't.
Comment on attachment 578480 [details] [diff] [review]
>- JSString *str = js_ValueToString(cx, Int32Value(JSID_TO_INT(id)));
>+ JSString *str = js_IntToString(cx, JSID_TO_INT(id));
Hah. What's js_ doing in front of IntToString...