Fixed the reporting of paint times in window.chrome.loadTimes()

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index a8698ade35d805fe1461db0395b778e081105b88..4529b499eb3e3f2d2282cad2d3d9a36f4dd06398 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -3328,8 +3328,10 @@ void RenderViewImpl::DidFlushPaint() {
   WebFrame* main_frame = webview()->mainFrame();
   for (WebFrame* frame = main_frame; frame;

[Charlie Reis, 2014/10/21 23:40:20]

Yikes.  The premise behind this whole loop seems broken to me.  Sorry to have
missed it.

I think this is intended to be the same short-term hack to avoid an OOPIF crash
as in GetScrollOffset below.  Can you copy Nasko's TODO from line 3360 here as
well, so that we update both places in a follow-up CL?

[Pat Meenan, 2014/10/22 12:40:03]

Done.

        frame = frame->traverseNext(false)) {
-    if (frame->isWebLocalFrame())
+    if (frame->isWebLocalFrame() && !frame->parent()) {
       main_frame = frame;
+      break;
+    }
   }
 
   // If we have a provisional frame we are between the start and commit stages