https://codereview.appspot.com/8913045/diff/1/src/com/google/caja/ses/StringMap.js
File src/com/google/caja/ses/StringMap.js (right):

https://codereview.appspot.com/8913045/diff/1/src/com/google/caja/ses/StringMap.js#newcode46
src/com/google/caja/ses/StringMap.js:46: // Object.create(null) is
broken; fall back to ES3-style implementation
The code also covers the case where ses.es5ProblemReports doesn't exist.
The comment leaves this out of the explanation.

I don't feel strongly about this. If you feel strongly to the contrary,
leave this alone. LGTM anyway.

https://codereview.appspot.com/8913045/

--

--- 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.


Reply via email to