LGTM ======================================================================== http://mondrian.corp.google.com/file/10347932///depot/googleclient/gears/opensource/gears/geolocation/wifi_data_provider_osx.cc?a=13 File //depot/googleclient/gears/opensource/gears/geolocation/wifi_data_provider_osx.cc (snapshot 13) ------------------------------------ Line 243: IOAllowPowerChange(root_port, reinterpret_cast<long>(message_argument)); This is a bit scary. Does this mean we could inadvertently prevent the system from going to sleep? ------------------------------------ Line 252: } I think you need to have a default. ========================================================================
-- To respond, reply to this email or visit http://mondrian.corp.google.com/10347932
