990484 - Expose a JS_IsIdentifier that takes chars+length

Status: RESOLVED FIXED

(Core :: JavaScript: Standard Library, defect)

Description

[Andrea Marchesini [:baku]]

Read bug 957086 comment 47:

1) Can you file a followup bug to get JSAPI to expose a version of JS_IsIdentifier that takes chars+length and then nix the need for a cx in IDB's CreateObjectStore and CreateIndex code?  That seems all the cx is used for there.

Comment 1

[Boris Zbarsky [:bzbarsky, bz on IRC]]

Basically, out situation is that we have an XPCOM string that we want to check for being a JS identifier.  Right now we have to go and create a JSString from it to do that, which seems a bit silly.

Jason, if you're OK with the basic idea, I'll just do this.

Comment 2

[Jason Orendorff [:jorendorff]]

Yes, of course!

Comment 3

[Boris Zbarsky [:bzbarsky, bz on IRC]]

Attachment: part 1. Add an overload of JS_IsIdentifier that takes chars+length

Comment 4

[Boris Zbarsky [:bzbarsky, bz on IRC]]

Attachment: part 2. Remove a bunch of now-unnecessary JSContext bits in indexedDB code

Comment 5

[Andrea Marchesini [:baku]]

Comment on attachment 8529511 [details] [diff] [review]
part 2.  Remove a bunch of now-unnecessary JSContext bits in indexedDB code

Review of attachment 8529511 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/datastore/DataStoreDB.cpp
@@ +271,5 @@
>      }
>    }
>  
>    {
>      RootedDictionary<IDBIndexParameters> params(cx);

Do we still need to root the dictionary in this case?

::: dom/indexedDB/KeyPath.cpp
@@ +29,5 @@
>  }
>  
>  typedef nsCharSeparatedTokenizerTemplate<IgnoreWhitespace> KeyPathTokenizer;
>  
> +static bool

This doesn't need to be static because it's included in an anonymous namespace.

Comment 6

[Boris Zbarsky [:bzbarsky, bz on IRC]]

> Do we still need to root the dictionary in this case?

Yes, until bug 1105615 is fixed.

> This doesn't need to be static because it's included in an anonymous namespace.

OK, fixed.

https://hg.mozilla.org/integration/mozilla-inbound/rev/f5154e4a4117
https://hg.mozilla.org/integration/mozilla-inbound/rev/c5ce96161a42

Comment 7

[Wes Kocher (:KWierso)]

https://hg.mozilla.org/mozilla-central/rev/f5154e4a4117
https://hg.mozilla.org/mozilla-central/rev/c5ce96161a42