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

https://codereview.appspot.com/10075043/diff/86001/src/com/google/caja/ses/StringMap.js#newcode20
src/com/google/caja/ses/StringMap.js:20: * @overrides ses
On 2013/07/13 04:18:47, MarkM wrote:
Doing as suggested (patch 14) broke runtests, apparently detecting the
kind of
inconsistency you raised. With patch 15, which chains tests
analogously to the
original, runtests completes successfully. Not sure why yet.

That's a bit vague. Which of the objects in the chain is missing? (The
error should have a relevant property name.) Or more generally, which
tests fail? What does the Chrome console show for those which do?

https://codereview.appspot.com/10075043/

--

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