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

components/offline_pages/core/snapshot_controller.cc

Index: components/offline_pages/core/snapshot_controller.cc
diff --git a/components/offline_pages/core/snapshot_controller.cc b/components/offline_pages/core/snapshot_controller.cc
index c683a1a9b1056a0e039db12074f9bb48cd36c51b..1cc399fd8a1fe9ded5fcee7b9ee4840aeaea6347 100644
--- a/components/offline_pages/core/snapshot_controller.cc
+++ b/components/offline_pages/core/snapshot_controller.cc
@@ -11,6 +11,8 @@
 #include "components/offline_pages/core/offline_page_feature.h"
 
 namespace {
+const bool kConsiderDocumentAvailableForSnapshot = true;

[fgorski, 2017/04/20 16:39:05]

This is where I'd put all constants for foreground and background until we
decide to vary within the class.

[chili, 2017/04/21 00:12:54]

Done.

+
 // Default delay, in milliseconds, between the main document parsed event and
 // snapshot. Note: this snapshot might not occur if the OnLoad event and
 // OnLoad delay elapses first to trigger a final snapshot.
@@ -33,19 +35,23 @@ SnapshotController::SnapshotController(
     : SnapshotController(task_runner,
                          client,
                          kDefaultDelayAfterDocumentAvailableMs,
-                         kDelayAfterDocumentOnLoadCompletedMs) {}
+                         kDelayAfterDocumentOnLoadCompletedMs,
+                         kConsiderDocumentAvailableForSnapshot) {}
 
 SnapshotController::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 consider_document_available_for_snapshot)
     : task_runner_(task_runner),
       client_(client),
       state_(State::READY),
       delay_after_document_available_ms_(delay_after_document_available_ms),
       delay_after_document_on_load_completed_ms_(
           delay_after_document_on_load_completed_ms),
+      consider_document_available_for_snapshot_(
+          consider_document_available_for_snapshot),
       weak_ptr_factory_(this) {
   if (offline_pages::ShouldUseTestingSnapshotDelay()) {
     delay_after_document_available_ms_ = kDelayForTests;
@@ -75,13 +81,16 @@ void SnapshotController::PendingSnapshotCompleted() {
 }
 
 void SnapshotController::DocumentAvailableInMainFrame() {
-  DCHECK_EQ(PageQuality::POOR, current_page_quality_);
-  // Post a delayed task to snapshot.
-  task_runner_->PostDelayedTask(
-      FROM_HERE, base::Bind(&SnapshotController::MaybeStartSnapshot,
-                            weak_ptr_factory_.GetWeakPtr(),
-                            PageQuality::FAIR_AND_IMPROVING),
-      base::TimeDelta::FromMilliseconds(delay_after_document_available_ms_));
+  if (consider_document_available_for_snapshot_) {
+    DCHECK_EQ(PageQuality::POOR, current_page_quality_);
+    // Post a delayed task to snapshot.
+    task_runner_->PostDelayedTask(
+        FROM_HERE,
+        base::Bind(&SnapshotController::MaybeStartSnapshot,
+                   weak_ptr_factory_.GetWeakPtr(),
+                   PageQuality::FAIR_AND_IMPROVING),
+        base::TimeDelta::FromMilliseconds(delay_after_document_available_ms_));
+  }
 }
 
 void SnapshotController::DocumentOnLoadCompletedInMainFrame() {