Got javascript: URIs working

Made some minor changes to plugin_dispatchEvent___ since code in
javascript: URI does not have this or event bound differently than top
level script code.

Updated HtmlAttributeRewriter to use the new dispatch method, and to
store the handler someplace where the dispatch method can get at it
since href values are strings, while the setAttr for
onclick handlers takes a function instance as a value.

Modified bridal to workaround Firefox which runs javascript: URLs
in the frame specified by target.


Submitted @4031

[Jasvir, 2010-03-09 01:58:01]

LGTM

http://codereview.appspot.com/239041/diff/1/4
File src/com/google/caja/plugin/domita.js (right):

http://codereview.appspot.com/239041/diff/1/4#newcode4145
src/com/google/caja/plugin/domita.js:4145: || imports[handler];
imports.handler is always readable?

[MikeSamuel, 2010-03-09 03:48:46]

http://codereview.appspot.com/239041/diff/1/4
File src/com/google/caja/plugin/domita.js (right):

http://codereview.appspot.com/239041/diff/1/4#newcode4145
src/com/google/caja/plugin/domita.js:4145: || imports[handler];
On 2010/03/09 01:58:01, jasvir wrote:
> imports.handler is always readable?

It is a name in the protected namespace, e.g. c_1___, so will never be readable.

[MikeSamuel, 2010-03-12 22:27:28]

ping

[MikeSamuel, 2010-03-16 00:40:26]

ping