https://codereview.appspot.com/256790043/diff/1/src/com/google/caja/ses/debug.js
File src/com/google/caja/ses/debug.js (right):
https://codereview.appspot.com/256790043/diff/1/src/com/google/caja/ses/debug.js#newcode129
src/com/google/caja/ses/debug.js:129: var line2CWFrame = (function() {
Add a doc comment explaining the line2CWFrame function.
https://codereview.appspot.com/256790043/diff/1/src/com/google/caja/ses/debug.js#newcode146
src/com/google/caja/ses/debug.js:146: var FFFramePattern =
/^\s*([^:@(]*?)\s*(?:\(.*\))?@(.*?)$/;
remove extra whitespace (alignment is not helpful here).
https://codereview.appspot.com/256790043/diff/1/src/com/google/caja/ses/debug.js#newcode291
src/com/google/caja/ses/debug.js:291: UnsafeError.stackTraceLimit = 100;
Do we really want to do this ourselves?
https://codereview.appspot.com/256790043/diff/1/src/com/google/caja/ses/debug.js#newcode341
src/com/google/caja/ses/debug.js:341: * <p>The difference between
Causeway format and extended
Don't document the format primarily here. Document it at the point where
it is returned to outside users.
https://codereview.appspot.com/256790043/
--
---
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/d/optout.
