Reload Lo-Fi images inline instead of reloading the whole page

content/public/browser/web_contents.h

Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index a96ed1cfc41f87740f56b2e1c2fd6cf3fb618433..4f0f29d4bb1a7e1563fabbc551ffc43efe4c9761 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -446,6 +446,10 @@ class WebContents : public PageNavigator,
   virtual void NotifyContextMenuClosed(
       const CustomContextMenuContext& context) = 0;
 
+  // Reloads all the Lo-Fi images in this WebContents. Ignores the cache and
+  // reloads from the network.

[Charlie Reis, 2016/01/29 23:01:12]

Is this supposed to disable LoFi state as well?  Is there any LoFi state in the
browser, or does that only exist in RenderFrameImpl?

[megjablon, 2016/01/29 23:22:23]

Yes it disables LoFi. If the user requests that LoFi images are loaded, they
should not get more LoFi images after this request. Does
ReloadAndDisableLoFiImages work better? The only LoFi state is RenderFrameImpl.

[Charlie Reis, 2016/01/29 23:52:00]

Ok.  I just wasn't clear whether there was additional state.

No need to change the method name, but please do mention that this disables LoFi
in the comment.

+  virtual void ReloadLoFiImages() = 0;

[Charlie Reis, 2016/01/29 23:01:12]

This doesn't really belong in the Editing commands section.

There's no LoFi mentions in this class yet and it's kind of similar to
NavigationController::ReloadDisableLoFi, so that would be one option.

I suppose it's not really navigation related, though, so maybe it's ok to put it
below ReloadFocusedFrame in the Commands section above.  (Please reorder in the
impl files as well.)

[megjablon, 2016/01/29 23:22:23]

Done.

+
   // Executes custom context menu action that was provided from Blink.
   virtual void ExecuteCustomContextMenuCommand(
       int action, const CustomContextMenuContext& context) = 0;