No problem. The linter caught both of these problems btw. We should review and check that in.
2009/8/27 <[email protected]>: > LGTM > > > http://codereview.appspot.com/110102/diff/1/2 > File src/com/google/caja/plugin/domita.js (right): > > http://codereview.appspot.com/110102/diff/1/2#newcode2706 > Line 2706: var nameLc = String(name).toLowerCase(); > I remember what happened: I was originally changing the value of name, > not nameLc, so it was already a local variable. Sorry. > > http://codereview.appspot.com/110102 >
