[Offline Pages] Define separate watchdog timeout for concurrent bg loads

chrome/browser/android/offline_pages/background_scheduler_bridge.cc

Index: chrome/browser/android/offline_pages/background_scheduler_bridge.cc
diff --git a/chrome/browser/android/offline_pages/background_scheduler_bridge.cc b/chrome/browser/android/offline_pages/background_scheduler_bridge.cc
index 4feb822d2908db1a31d6bf7b240c128ddc1d5689..56d2bce565cc5ec5feff9305759602c9c97fae59 100644
--- a/chrome/browser/android/offline_pages/background_scheduler_bridge.cc
+++ b/chrome/browser/android/offline_pages/background_scheduler_bridge.cc
@@ -52,7 +52,8 @@ static jboolean StartProcessing(JNIEnv* env,
       static_cast<net::NetworkChangeNotifier::ConnectionType>(
           j_net_connection_type));
   return coordinator->StartProcessing(
-      device_conditions, base::Bind(&ProcessingDoneCallback, j_callback_ref));
+      true /* is_background_scheduled */, device_conditions,

[Pete Williamson, 2016/10/12 22:33:24]

Let's not use a binary argument, they are hard to understand at the point of the
function call.  Perhaps an enum, or make const values indicating the meaning and
assign the values to true or false, using the value instead of true here.

[dougarnett, 2016/10/13 19:46:44]

ok, trying enum internally instead and reverting public api

+      base::Bind(&ProcessingDoneCallback, j_callback_ref));
 }
 
 void BackgroundSchedulerBridge::Schedule(