======================================================================== http://mondrian.corp.google.com/file/10662153///depot/googleclient/gears/opensource/gears/geolocation/geolocation.cc?a=4 File //depot/googleclient/gears/opensource/gears/geolocation/geolocation.cc (snapshot 4) ------------------------------------ Line 620: AddProvidersToRequest(urls, info.get()); But later we check for a 0 timeout and remove them. How about checking for a 0 timeout up-front and not bothering starting any providers if 0 is the timeout? ------------------------------------ Line 647: info->timeout != 0 && Why do we check for a non-zero timeout here? ========================================================================
-- To respond, reply to this email or visit http://mondrian.corp.google.com/10662153
