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() {
On 2015/07/20 16:34:43, kpreid_google wrote:
Add a doc comment explaining the line2CWFrame function.
Done.
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*(?:\(.*\))?@(.*?)$/;
On 2015/07/20 16:34:43, kpreid_google wrote:
remove extra whitespace (alignment is not helpful here).
Done.
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;
On 2015/07/20 16:34:43, kpreid_google wrote:
Do we really want to do this ourselves?
Gone.
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
On 2015/07/20 16:34:43, kpreid_google wrote:
Don't document the format primarily here. Document it at the point
where it is
returned to outside users.
Done. I explained in the doc-comment for the file.
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.
