[OfflinePages] Call NotifyInterrupted for pending requests

components/offline_pages/background/request_coordinator.cc

Index: components/offline_pages/background/request_coordinator.cc
diff --git a/components/offline_pages/background/request_coordinator.cc b/components/offline_pages/background/request_coordinator.cc
index 8dcc4d94def87f3cd13e2d2d56eefe69908e5153..b5f40489ddf4a4d3c5ed59da0b00a32e93c9445e 100644
--- a/components/offline_pages/background/request_coordinator.cc
+++ b/components/offline_pages/background/request_coordinator.cc
@@ -431,6 +431,7 @@ void RequestCoordinator::AddRequestResultCallback(
     AddRequestResult result,
     const SavePageRequest& request) {
   NotifyAdded(request);
+  NotifyChanged(request);

[Pete Williamson, 2016/11/28 18:12:03]

Did you test this in the running app as opposed to just unit tests?  I'm
concerned that the code may not expect a NotifyChange when a request first gets
added in addition to a NotifyAdded, and want to make sure it doesn't cause any
problems.

[dougarnett, 2016/11/28 18:36:54]

Yeah, actually added this due to behavior seen in app testing.

[fgorski, 2016/11/28 20:53:49]

Why both added and changed? I don't understand it.

[dougarnett, 2016/11/28 21:24:55]

Dropped this and fixed request state handling in OnAdded

   // Inform the scheduler that we have an outstanding task.
   scheduler_->Schedule(GetTriggerConditions(kUserRequest));