http://codereview.appspot.com/120062/diff/1/2
File tests/com/google/caja/plugin/domita_test.html (right):

http://codereview.appspot.com/120062/diff/1/2#newcode309
Line 309: cl = cl.replace(/clickme|waiting/, '');
Just to satisfy my silly pedantry, can you make this regex
  /\b(clickme|waiting)\b\s*/g

http://codereview.appspot.com/120062/diff/1/3
File tests/com/google/caja/plugin/domita_test_untrusted.html (right):

http://codereview.appspot.com/120062/diff/1/3#newcode19
Line 19: .clickme { border: 2px solid fuchsia; }
Wow.  Our test pages are going to look like a bunch of skittles.

http://codereview.appspot.com/120062

Reply via email to