Revises RequestCoordinator to not try to schedule another request when previous one fails.

components/offline_pages/background/request_coordinator.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/offline_pages/background/request_coordinator.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 119 matching lines @@
                                    weak_ptr_factory_.GetWeakPtr()));
 }
 
 void RequestCoordinator::OfflinerDoneCallback(const SavePageRequest& request,
                                               Offliner::RequestStatus status) {
   DVLOG(2) << "offliner finished, saved: "
            << (status == Offliner::RequestStatus::SAVED) << ", "
            << __FUNCTION__;
   last_offlining_status_ = status;
 
-  // TODO(petewil): If the request succeeded, remove it from the Queue.
+  // If the request succeeded, remove it from the Queue and maybe schedule
+  // another one.
   if (status == Offliner::RequestStatus::SAVED) {
     queue_->RemoveRequest(request.request_id(),
                           base::Bind(&RequestCoordinator::UpdateRequestCallback,
                                      weak_ptr_factory_.GetWeakPtr()));
+
+    // TODO(petewil): Check time budget. Return to the scheduler if we are out.
+
+    // Start another request if we have time.
+    TryNextRequest();
   }
-
-  // TODO(petewil): Check time budget. Return to the scheduler if we are out.
-
-  // Start a request if we have time.
-  TryNextRequest();
 }
 
 }  // namespace offline_pages