early-bind idClass in caja.js and Playground

Right now caja.js and the Playground will call the cajoler
without an idClass parameter, which means that rewriting html ids
is deferred until gadget instantiation time.

I don't see much benefit to doing that, since there isn't an api
for multiply-instantiatiating the same gadget on the same host
page, and there isn't any particular demand for doing so.

This change makes caja.js and the Playground pass an idClass to
the cajoler in some cases, which has a few performance benefits:

1. With early-binding of idClass, the cajoler doesn't have to
generate html-emitter statements to set every html id, which
reduces the amount of js that the cajoler processes, which speeds
up the cajoler.

2. Avoiding all those html-emitter statements also speeds
up client-side rendering.

If in the future we do want to support multiply-instantiating
a gadget without calling cajole each time, then it's easy enough
to add an option for that.

[metaweta, 2011-12-13 04:23:46]

LGTM.

http://codereview.appspot.com/5488053/diff/1/src/com/google/caja/demos/playgr...
File src/com/google/caja/demos/playground/client/ui/PlaygroundView.java (right):

http://codereview.appspot.com/5488053/diff/1/src/com/google/caja/demos/playgr...
src/com/google/caja/demos/playground/client/ui/PlaygroundView.java:436: o[key] =
value;
Someday I'll have to write code that says o[key] = do[key].

http://codereview.appspot.com/5488053/diff/1/src/com/google/caja/plugin/guest...
File src/com/google/caja/plugin/guest-manager.js (right):

http://codereview.appspot.com/5488053/diff/1/src/com/google/caja/plugin/guest...
src/com/google/caja/plugin/guest-manager.js:68: idSuffix: divs.idClass,     //
misnomer; idSuffix should have leading '-'
So is a hyphen prefixed to it somewhere else?

http://codereview.appspot.com/5488053/diff/1/src/com/google/caja/plugin/ses-f...
File src/com/google/caja/plugin/ses-frame-group.js (right):

http://codereview.appspot.com/5488053/diff/1/src/com/google/caja/plugin/ses-f...
src/com/google/caja/plugin/ses-frame-group.js:78: '-' + divs.idClass, uriPolicy,
divs.inner);
Ah, here it is.  Maybe extend the 'misnomer' comment in the last file to point
here?

[felix8a, 2011-12-13 18:57:49]

@44679

http://codereview.appspot.com/5488053/diff/1/src/com/google/caja/plugin/ses-f...
File src/com/google/caja/plugin/ses-frame-group.js (right):

http://codereview.appspot.com/5488053/diff/1/src/com/google/caja/plugin/ses-f...
src/com/google/caja/plugin/ses-frame-group.js:78: '-' + divs.idClass, uriPolicy,
divs.inner);
On 2011/12/13 04:23:46, metaweta wrote:
> Ah, here it is.  Maybe extend the 'misnomer' comment in the last file to point
> here?

Done.