Remove a bunch of provisionalDataSource() usage.

content/renderer/render_view_impl.cc

 // Copyright (c) 2012 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 "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/auto_reset.h"
@@ skipping 2842 matching lines @@
     }
   }
 
   // There's nothing to do if there are no local frames in this RenderView's
   // frame tree. This can happen if DidFlushPaint is called after the
   // RenderView's local main frame is swapped to a remote frame. See
   // http://crbug.com/513552.
   if (main_frame->isWebRemoteFrame())
     return;
 
-  // If we have a provisional frame we are between the start and commit stages
-  // of loading and we don't want to save stats.
-  if (!main_frame->provisionalDataSource()) {
-    WebDataSource* ds = main_frame->dataSource();
-    if (!ds)
-      return;
+  WebDataSource* ds = main_frame->dataSource();
+  if (!ds)
+    return;
 
-    DocumentState* document_state = DocumentState::FromDataSource(ds);
+  DocumentState* document_state = DocumentState::FromDataSource(ds);
 
-    // TODO(jar): The following code should all be inside a method, probably in
-    // NavigatorState.
-    Time now = Time::Now();
-    if (document_state->first_paint_time().is_null()) {
-      document_state->set_first_paint_time(now);
-    }
-    if (document_state->first_paint_after_load_time().is_null() &&
-        !document_state->finish_load_time().is_null()) {
-      document_state->set_first_paint_after_load_time(now);
-    }
+  // TODO(jar): The following code should all be inside a method, probably in
+  // NavigatorState.
+  Time now = Time::Now();
+  if (document_state->first_paint_time().is_null()) {
+    document_state->set_first_paint_time(now);
+  }
+  if (document_state->first_paint_after_load_time().is_null() &&
+      !document_state->finish_load_time().is_null()) {
+    document_state->set_first_paint_after_load_time(now);
   }
 }
 
 gfx::Vector2d RenderViewImpl::GetScrollOffset() {
   WebFrame* main_frame = webview()->mainFrame();
   for (WebFrame* frame = main_frame; frame;
        frame = frame->traverseNext(false)) {
     // TODO(nasko): This is a hack for the case in which the top-level
     // frame is being rendered in another process. It will not
     // behave correctly for out of process iframes.
@@ skipping 800 matching lines @@
     std::vector<gfx::Size> sizes;
     ConvertToFaviconSizes(icon_urls[i].sizes(), &sizes);
     if (!url.isEmpty())
       urls.push_back(
           FaviconURL(url, ToFaviconType(icon_urls[i].iconType()), sizes));
   }
   SendUpdateFaviconURL(urls);
 }
 
 }  // namespace content