Fix loading placeholders for Lo-Fi

content/browser/web_contents/web_contents_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/browser/web_contents/web_contents_impl.h"
 
 #include <stddef.h>
 
 #include <cmath>
 #include <utility>
@@ skipping 2541 matching lines @@
 void WebContentsImpl::ReloadFocusedFrame(bool bypass_cache) {
   RenderFrameHost* focused_frame = GetFocusedFrame();
   if (!focused_frame)
     return;
 
   focused_frame->Send(new FrameMsg_Reload(
       focused_frame->GetRoutingID(), 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));
 }
 
 void WebContentsImpl::Undo() {
   RenderFrameHost* focused_frame = GetFocusedFrame();
   if (!focused_frame)
     return;
 
   focused_frame->Send(new InputMsg_Undo(focused_frame->GetRoutingID()));
   RecordAction(base::UserMetricsAction("Undo"));
@@ skipping 2589 matching lines @@
   for (RenderViewHost* render_view_host : render_view_host_set)
     render_view_host->OnWebkitPreferencesChanged();
 }
 
 void WebContentsImpl::SetJavaScriptDialogManagerForTesting(
     JavaScriptDialogManager* dialog_manager) {
   dialog_manager_ = dialog_manager;
 }
 
 }  // namespace content