LGTM ======================================================================== http://mondrian.corp.google.com/file/8747274///depot/googleclient/gears/opensource/gears/installer/iemobile/cab_updater.cc?a=1 File //depot/googleclient/gears/opensource/gears/installer/iemobile/cab_updater.cc (snapshot 1) ------------------------------------ Line 192: // Marshall the callback to the browser thread so that we can show the dialog. line too long ======================================================================== http://mondrian.corp.google.com/file/8747274///depot/googleclient/gears/opensource/gears/installer/iemobile/periodic_checker.cc?a=1 File //depot/googleclient/gears/opensource/gears/installer/iemobile/periodic_checker.cc (snapshot 1) ------------------------------------ Line 362: url_.replace(0, 5, HttpConstants::kHttpScheme); Using HTTP is a security risk, but I guess this isn't any worse than before since we use HTTP to talk to Omaha anyway... ========================================================================
-- To respond, reply to this email or visit http://mondrian.corp.google.com/8747274
