Consistently use LoFi for an entire page

content/common/navigation_params.h

Index: content/common/navigation_params.h
diff --git a/content/common/navigation_params.h b/content/common/navigation_params.h
index 37b1af035401ee8e9ab394831888b68c6519e4e9..edf0615b051288c518c6a89cf5bd8fc52d9ddd7e 100644
--- a/content/common/navigation_params.h
+++ b/content/common/navigation_params.h
@@ -22,6 +22,18 @@ class RefCountedMemory;
 
 namespace content {
 
+// The LoFi state which determines whether to add the Lo-Fi header.
+enum LoFiState {
+  // Let the browser process decide whether or not to request the Lo-Fi version.
+  LOFI_UNSPECIFIED = 0,
+
+  // Request a normal (non-Lo-Fi) version of the resource.
+  LOFI_OFF,
+
+  // Request a Lo-Fi version of the resource.
+  LOFI_ON,
+};
+
 // PlzNavigate
 // Helper function to determine if the navigation to |url| should make a request
 // to the network stack. A request should not be sent for data URLs, JavaScript
@@ -89,6 +101,10 @@ struct CONTENT_EXPORT CommonNavigationParams {
   // History URL for use in Blink's SubstituteData.
   // Is only used with data: URLs.
   GURL history_url_for_data_url;
+
+  // Whether or not to request a LoFi version of the document or let the browser
+  // decide.
+  LoFiState lofi_state;

[clamy, 2015/10/02 16:08:30]

Why are we adding this parameter to this struct? I cannot find a single instance
where it will be set to something else than LOFI_UNSPECIFIED. Also, if you add a
parameter to this struct, you should update its constructor.

[megjablon, 2015/10/02 17:47:39]

It's going to be used in a follow up cl that switches the snackbar that reloads
the page with images to using this mechanism. Added to the constructor.

 };
 
 // Provided by the renderer ----------------------------------------------------