http://codereview.appspot.com/96177/diff/7064/6069
File src/com/google/caja/cajita-module.js (right):

http://codereview.appspot.com/96177/diff/7064/6069#newcode92
Line 92: handle: ___.markFuncFreeze(function theHandler(module) {
As we discussed, this function should exit without doing anything if it
detects "savedHead !== head".

http://codereview.appspot.com/96177/diff/7064/6069#newcode107
Line 107: function timeout(p) {
As we discussed, this does not need a param -- just use "savedHead".
Also add a comment clarifying the case where "savedHead !== head".

http://codereview.appspot.com/96177/diff/7064/6069#newcode118
Line 118: var savedHead = head;
As we discussed, move savedHead up ....

http://codereview.appspot.com/96177/diff/7064/6069#newcode122
Line 122: timeout(savedHead);
As we discussed, script.onerror and script.onreadystatechange should
*both* do "script.onerror = script.onreadystatechange = null" to clear
things out.

http://codereview.appspot.com/96177

Reply via email to