[gwt-contrib] Re: Firefox 13 DevMode Plugin (issue1738803)

2012-06-11 Thread conroy
LGTM + nits http://gwt-code-reviews.appspot.com/1738803/diff/1/plugins/xpcom/JSRunner.cpp File plugins/xpcom/JSRunner.cpp (right): http://gwt-code-reviews.appspot.com/1738803/diff/1/plugins/xpcom/JSRunner.cpp#newcode121 plugins/xpcom/JSRunner.cpp:121: Debug::log(Debug::Error) Get JSPrincial

[gwt-contrib] Re: Firefox 13 DevMode Plugin (issue1738803)

2012-06-11 Thread acleung
http://gwt-code-reviews.appspot.com/1738803/diff/1/plugins/xpcom/JSRunner.cpp File plugins/xpcom/JSRunner.cpp (right): http://gwt-code-reviews.appspot.com/1738803/diff/1/plugins/xpcom/JSRunner.cpp#newcode121 plugins/xpcom/JSRunner.cpp:121: Debug::log(Debug::Error) Get JSPrincial at