@r5508
https://codereview.appspot.com/11778043/diff/1/src/com/google/caja/ses/startSES.js
File src/com/google/caja/ses/startSES.js (right):
https://codereview.appspot.com/11778043/diff/1/src/com/google/caja/ses/startSES.js#newcode1600
src/com/google/caja/ses/startSES.js:1600: // Note this is safe not in
that it is safe for the prop to be an
On 2013/07/24 22:39:14, MarkM wrote:
Copy paste error? Inverted comment?
Copy-paste and also I neglected to revise the comment since an earlier
version (we _are_ cleaning).
https://codereview.appspot.com/11778043/diff/1/src/com/google/caja/ses/whitelist.js
File src/com/google/caja/ses/whitelist.js (right):
https://codereview.appspot.com/11778043/diff/1/src/com/google/caja/ses/whitelist.js#newcode51
src/com/google/caja/ses/whitelist.js:51: * <p>If the property is an
accessor property, it is not
On 2013/07/24 22:39:14, MarkM wrote:
Do you really want a paragraph break within an <li> section?
Removed.
That is valid, but on second thought not a good idea in this way as it
results in inconsistent formatting; any given list should contain either
all bare text or all paragraphs, not a mix.
https://codereview.appspot.com/11778043/
--
---
You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
