"Load image" context menu item to reload a LoFi image.

content/public/common/context_menu_params.h

Index: content/public/common/context_menu_params.h
diff --git a/content/public/common/context_menu_params.h b/content/public/common/context_menu_params.h
index 9fe170a0184e4b1f8a3ff62f705a4866d565759e..cf1379861484950b7978110cd1604024dd5b3099 100644
--- a/content/public/common/context_menu_params.h
+++ b/content/public/common/context_menu_params.h
@@ -83,6 +83,10 @@ struct CONTENT_EXPORT ContextMenuParams {
   // non-empty contents.
   bool has_image_contents;
 
+  // If |media_type| is MediaTypeImage and |has_image_contents| is true, then
+  // this contains the if the image was a Lo-Fi image.

[Charlie Reis, 2015/05/11 22:32:58]

nit: this contains whether the image

[megjablon, 2015/05/11 22:58:40]

Done.

+  bool image_was_fetched_lo_fi;

[Charlie Reis, 2015/05/11 22:32:58]

Same question: all the uses seem to be outside content, which suggests that this
should not be in the content public interface.

https://www.chromium.org/developers/content-module/content-api
"all classes/structs/enums in the public API must be used by embedders and
content. i.e. if the chrome layer uses a struct but content doesn't know about
it, it doesn't belong in content/public but instead some module that's higher
level."

[megjablon, 2015/05/11 22:58:40]

Acknowledged.

+
   // This is the URL of the top level page that the context menu was invoked
   // on.
   GURL page_url;