[Offline pages]: Implement background loader to save on last retry, and record last retry success U…

chrome/browser/android/offline_pages/background_loader_offliner.h

 // 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.
 
 #ifndef CHROME_BROWSER_ANDROID_OFFLINE_PAGES_BACKGROUND_LOADER_OFFLINER_H_
 #define CHROME_BROWSER_ANDROID_OFFLINE_PAGES_BACKGROUND_LOADER_OFFLINER_H_
 
 #include <memory>
 
 #include "base/android/application_status_listener.h"
@@ skipping 66 matching lines @@
   void OnApplicationStateChange(
       base::android::ApplicationState application_state);
   void HandleApplicationStateChangeCancel(const SavePageRequest& request,
                                           int64_t offline_id);
 
   std::unique_ptr<background_loader::BackgroundLoaderContents> loader_;
   // Not owned.
   content::BrowserContext* browser_context_;
   // Not owned.
   OfflinePageModel* offline_page_model_;
+  // Not owned.
+  const OfflinerPolicy* policy_;
   // Tracks pending request, if any.
   std::unique_ptr<SavePageRequest> pending_request_;
   // Handles determining when a page should be snapshotted.
   std::unique_ptr<SnapshotController> snapshot_controller_;
   // Callback when pending request completes.
   CompletionCallback completion_callback_;
   // Callback to report progress.
   ProgressCallback progress_callback_;
   // ApplicationStatusListener to monitor if Chrome moves to the foreground.
   std::unique_ptr<base::android::ApplicationStatusListener> app_listener_;
   // Whether we are on a low-end device.
   bool is_low_end_device_;
 
   // Save state.
   SaveState save_state_;
   // Page load state.
   PageLoadState page_load_state_;
   // Seconds to delay before taking snapshot.
   long page_delay_ms_;
   // Network bytes loaded.
   int64_t network_bytes_;
+  // Whether the low bar has been met.

[fgorski, 2017/04/14 05:15:57]

be more specific about the low bar, please... low bar of what?

[chili, 2017/04/15 00:34:58]

Done.

+  bool is_low_bar_met_;
+  // Whether the snapshot is on the last retry.
+  bool did_snapshot_on_last_retry_;
 
   // Callback for cancel.
   CancelCallback cancel_callback_;
 
   base::WeakPtrFactory<BackgroundLoaderOffliner> weak_ptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(BackgroundLoaderOffliner);
 };
 
 }  // namespace offline_pages
 #endif  // CHROME_BROWSER_ANDROID_OFFLINE_PAGES_BACKGROUND_LOADER_OFFLINER_H_