Teach CRFO:CapturePageText to recognize page refreshes with long delays.

chrome/renderer/chrome_render_frame_observer.cc

 // Copyright 2013 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/renderer/chrome_render_frame_observer.h"
 
 #include <stddef.h>
 
 #include <limits>
 #include <string>
@@ skipping 46 matching lines @@
 using content::SSLStatus;
 using content::RenderFrame;
 
 // Maximum number of characters in the document to index.
 // Any text beyond this point will be clipped.
 static const size_t kMaxIndexChars = 65535;
 
 // Constants for UMA statistic collection.
 static const char kTranslateCaptureText[] = "Translate.CaptureText";
 
+// For a page that auto-refreshes, we still show the bubble, if
+// the refresh delay is less than this value (in seconds).
+static const double kLocationChangeIntervalInSeconds = 10;
+
 namespace {
 
 // If the source image is null or occupies less area than
 // |thumbnail_min_area_pixels|, we return the image unmodified.  Otherwise, we
 // scale down the image so that the width and height do not exceed
 // |thumbnail_max_size_pixels|, preserving the original aspect ratio.
 SkBitmap Downscale(const blink::WebImage& image,
                    int thumbnail_min_area_pixels,
                    const gfx::Size& thumbnail_max_size_pixels) {
   if (image.isNull())
@@ skipping 254 matching lines @@
       crash_keys::kViewCount,
       base::SizeTToString(content::RenderView::GetRenderViewCount()));
 }
 
 void ChromeRenderFrameObserver::CapturePageText(TextCaptureType capture_type) {
   WebLocalFrame* frame = render_frame()->GetWebFrame();
   if (!frame)
     return;
 
   // Don't capture pages that have pending redirect or location change.
-  if (frame->isNavigationScheduled())
+  if (frame->isNavigationScheduledWithin(kLocationChangeIntervalInSeconds))
     return;
 
   // Don't index/capture pages that are in view source mode.
   if (frame->isViewSourceModeEnabled())
     return;
 
   // Don't capture text of the error pages.
   WebDataSource* ds = frame->dataSource();
   if (ds && ds->hasUnreachableURL())
     return;
@@ skipping 35 matching lines @@
     case blink::WebMeaningfulLayout::FinishedParsing:
       CapturePageText(PRELIMINARY_CAPTURE);
       break;
     case blink::WebMeaningfulLayout::FinishedLoading:
       CapturePageText(FINAL_CAPTURE);
       break;
     default:
       break;
   }
 }