[Offline Pages] Snapshotting on timeout of last retry.

chrome/browser/android/offline_pages/background_loader_offliner.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 "chrome/browser/android/offline_pages/background_loader_offliner.h"
 
 #include "base/sys_info.h"
 #include "chrome/browser/android/offline_pages/offline_page_mhtml_archiver.h"
 #include "components/offline_pages/core/background/save_page_request.h"
 #include "components/offline_pages/core/offline_page_model.h"
@@ skipping 55 matching lines @@
     return;
 
   if (save_state_ != NONE) {
     save_state_ = DELETE_AFTER_SAVE;
     return;
   }
 
   ResetState();
 }
 
+bool BackgroundLoaderOffliner::HandleTimeout(const SavePageRequest& request) {
+  return false;

[Pete Williamson, 2017/01/25 00:06:16]

This should have a TODO: decide if we want to also take a snapshot on the last
timeout for the background loader offliner.

+}
+
 void BackgroundLoaderOffliner::DidStopLoading() {
   if (!pending_request_.get()) {
     DVLOG(1) << "DidStopLoading called even though no pending request.";
     return;
   }
 
   save_state_ = SAVING;
   SavePageRequest request(*pending_request_.get());
   content::WebContents* web_contents(
       content::WebContentsObserver::web_contents());
@@ skipping 82 matching lines @@
       application_state ==
           base::android::APPLICATION_STATE_HAS_RUNNING_ACTIVITIES) {
     DVLOG(1) << "App became active, canceling current offlining request";
     SavePageRequest* request = pending_request_.get();
     Cancel();
     completion_callback_.Run(*request, RequestStatus::FOREGROUND_CANCELED);
   }
 }
 
 }  // namespace offline_pages