Looks nice. One thing: ======================================================================== http://mondrian.corp.google.com/file/8151949///depot/googleclient/gears/opensource/gears/base/common/js_runner_ie.cc?a=1 File //depot/googleclient/gears/opensource/gears/base/common/js_runner_ie.cc (snapshot 1) ------------------------------------ Line 569: // This solution is less than ideal - the fundamental problem is probably Maybe we should only run this code when the leak checker is enabled? You could wrap both pieces of code with the same #define guard. ========================================================================
-- To respond, reply to this email or visit http://mondrian.corp.google.com/8151949
