[Offline Pages] Pick correct request when resuming.

components/offline_pages/core/background/request_coordinator.h

Index: components/offline_pages/core/background/request_coordinator.h
diff --git a/components/offline_pages/core/background/request_coordinator.h b/components/offline_pages/core/background/request_coordinator.h
index 770cae3a8f1cd7595a44091e120efbc01741d914..330aafb7fd7e8162b6b527c177e16e9c56739726 100644
--- a/components/offline_pages/core/background/request_coordinator.h
+++ b/components/offline_pages/core/background/request_coordinator.h
@@ -452,6 +452,9 @@ class RequestCoordinator : public KeyedService,
   base::OneShotTimer watchdog_timer_;
   // Used for potential immediate processing when we get network connection.
   std::unique_ptr<ConnectionNotifier> connection_notifier_;
+  // Used to track whether we have a prioritized request. This deque can only be
+  // edited by request_picker_task. Owning it here for persistency.

[Pete Williamson, 2017/03/08 02:04:21]

IS this comment correct?  I thought that RC was filling it, and Picker was
deleting it, so technically it seems like both are editing it.

Furthermore, it seems like RC also removes items when we resume.  Let's document
this carefully somewhere.

+  std::deque<int64_t> prioritized_requests_;
   // Allows us to pass a weak pointer to callbacks.
   base::WeakPtrFactory<RequestCoordinator> weak_ptr_factory_;