Remove ViewHostMsg_DidFirstPaintAfterLoad

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 906788e4b419dfc4a37a2666b8650178c88fde79..726ba3e3cb6a2d58639cd212dbfb3b40ee7cf8cc 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -3693,15 +3693,8 @@ void RenderFrameImpl::didCommitProvisionalLoad(
       return;
   }
 
-  // For navigations that change the document, the browser process needs to be
-  // notified of the first paint of that page, so it can cancel the timer that
-  // waits for it.
-  if (is_main_frame_ && !navigation_state->WasWithinSameDocument()) {
+  if (is_main_frame_ && !navigation_state->WasWithinSameDocument())

[Charlie Reis, 2017/03/24 16:52:01]

Let's leave some comment here so that it's clear why this is needed.  Something
like:

Navigations that change the document represent a new content source.  Keep track
of that on the widget to help the browser process detect when stale compositor
frames are being shown after a commit.

[Saman Sami, 2017/03/24 18:04:50]

That's a good comment. I'll add it verbatim.

     GetRenderWidget()->IncrementContentSourceId();
-    render_view_->QueueMessage(
-        new ViewHostMsg_DidFirstPaintAfterLoad(render_view_->routing_id_),
-        MESSAGE_DELIVERY_POLICY_WITH_VISUAL_STATE);
-  }
 
   // When we perform a new navigation, we need to update the last committed
   // session history entry with state for the page we are leaving. Do this