======================================================================== http://mondrian.corp.google.com/file/10229053///depot/googleclient/gears/opensource/gears/base/npapi/browser_utils.cc?a=2 File //depot/googleclient/gears/opensource/gears/base/npapi/browser_utils.cc (snapshot 2) ------------------------------------ Line 35: #elif ANDROID On 5:21 pm, andreip wrote: > OS_ANDROID
Done. ------------------------------------ Line 172: bool ret = false; On 5:23 pm, andreip wrote: > true Done. ------------------------------------ Line 182: return false; On 5:23 pm, andreip wrote: > true (and in the rest of the error cases) Done. ======================================================================== http://mondrian.corp.google.com/file/10229053///depot/googleclient/gears/opensource/gears/localserver/managed_resource_store_module.cc?a=2 File //depot/googleclient/gears/opensource/gears/localserver/managed_resource_store_module.cc (snapshot 2) ------------------------------------ Line 203: if (!BrowserUtils::IsOnline(context->js_context())) { On 5:27 pm, andreip wrote: > Not sure about this one. I do agree that the call can be expensive (especially > when we're in a worker). Maybe add a comment about this? We could ask michaeln > for a second opinion as well? Done. ======================================================================== http://mondrian.corp.google.com/file/10229053///depot/googleclient/gears/opensource/gears/localserver/npapi/async_task_np.cc?a=2 File //depot/googleclient/gears/opensource/gears/localserver/npapi/async_task_np.cc (snapshot 2) ------------------------------------ Line 30: #include "gears/base/android/browsing_context_android.h" On 5:28 pm, andreip wrote: > wrong order Done. ------------------------------------ Line 368: #ifdef ANDROID On 5:29 pm, andreip wrote: > OS_ANDROID Done. ======================================================================== -- To respond, reply to this email or visit http://mondrian.corp.google.com/10229053
