Fix loading placeholders for Lo-Fi

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 23a03d9b75fac5e2fd7bb91fa7001bfe8296fbcc..b0af69ed31a6fc3f5c2f9b10121abec6cb09cad4 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -2559,6 +2559,9 @@ void WebContentsImpl::ReloadFocusedFrame(bool bypass_cache) {
 }
 
 void WebContentsImpl::ReloadLoFiImages() {
+  // Lo-Fi images must first be disabled synchronously so new frames are not

[nasko, 2016/07/12 19:03:57]

You cannot call into renderers synchronously from the browser process. The
ordering of IPCs will basically guarantee that you won't reload a frame before
it is set to disabled state. For new frames, it is a race condition that I don't
know if you can avoid.

+  // created with Lo-Fi enabled.
+  SendToAllFrames(new FrameMsg_DisableLoFiImages(MSG_ROUTING_NONE));
   SendToAllFrames(new FrameMsg_ReloadLoFiImages(MSG_ROUTING_NONE));
 }