[Offline pages]: Move logic for whether to consider the DocumentAvailableInMainFrame signal to the …

components/offline_pages/core/snapshot_controller.h

Index: components/offline_pages/core/snapshot_controller.h
diff --git a/components/offline_pages/core/snapshot_controller.h b/components/offline_pages/core/snapshot_controller.h
index 36e1680a6f03f515a7c0305eef0669a6602796d6..069c08c8b9ce2126ca5a5a42a6e56065978891a0 100644
--- a/components/offline_pages/core/snapshot_controller.h
+++ b/components/offline_pages/core/snapshot_controller.h
@@ -60,14 +60,23 @@ class SnapshotController {
     virtual ~Client() {}
   };
 
-  SnapshotController(
+  // Creates a SnapshotController with document available delay = 7s

[fgorski, 2017/04/21 16:05:10]

nit: please add comma after 7s.

[chili, 2017/04/24 20:34:14]

Done.

+  // document on load delay = 1s and triggers snapshot on document available.
+  static SnapshotController* CreateForForegroundOfflining(

[fgorski, 2017/04/21 16:05:10]

I think these 2 will be safer with a unique_ptr<> as a return type to clearly
stress the ownership.

[chili, 2017/04/24 20:34:14]

Done.

+      const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
+      SnapshotController::Client* client);
+  // Creates a SnapshotController with document on load delay = 2s
+  // and ignores document available signal.
+  static SnapshotController* CreateForBackgroundOfflining(
       const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
       SnapshotController::Client* client);
+
   SnapshotController(
       const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
       SnapshotController::Client* client,
       int64_t delay_after_document_available_ms,
-      int64_t delay_after_document_on_load_completed_ms);
+      int64_t delay_after_document_on_load_completed_ms,
+      bool document_available_triggers_snapshot);
   virtual ~SnapshotController();
 
   // Resets the 'session', returning controller to initial state.
@@ -103,6 +112,7 @@ class SnapshotController {
   SnapshotController::State state_;
   int64_t delay_after_document_available_ms_;
   int64_t delay_after_document_on_load_completed_ms_;
+  bool document_available_triggers_snapshot_;
 
   // The expected quality of a snapshot taken at the moment this value is
   // queried.