Consistently use LoFi for an entire page

content/public/browser/resource_dispatcher_host_delegate.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_RESOURCE_DISPATCHER_HOST_DELEGATE_H_
 #define CONTENT_PUBLIC_BROWSER_RESOURCE_DISPATCHER_HOST_DELEGATE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 106 matching lines @@
 
   // Informs the delegate that a request has been redirected.
   virtual void OnRequestRedirected(const GURL& redirect_url,
                                    net::URLRequest* request,
                                    ResourceContext* resource_context,
                                    ResourceResponse* response);
 
   // Notification that a request has completed.
   virtual void RequestComplete(net::URLRequest* url_request);
 
+  // Asks the embedder if Lo-Fi mode should be enabled for the given request. It
+  // is only called for main frame requests with an unspecified Lo-Fi value.

[nasko, 2015/10/15 16:54:43]

nit: "main frame navigation requests", since we don't call this for subresource
requests, right?

[mmenke, 2015/10/15 17:04:28]

Effectively we don't - subresources inherit values from main frame....Main frame
requests may start out with UNSPECIFIED, but once this assigns a value, then
it's passed back to the renderer, and used for all subresource.  That having
been said...There could certainly be some path where subresources are requested
before it's been set for the main frame, so I agree that more weasily language
is probably warranted.

[megjablon, 2015/10/19 22:13:49]

Changed this to just "called for requests." It will be called for anything that
goes through ResourceDispatcherHostImpl::BeginRequest with LOFI_UNSPECIFIED.
Paths where the main frame hasn't set the state yet or there isn't a main frame
request will also call this.

+  virtual bool ShouldEnableLoFiMode(net::URLRequest* url_request,
+                                    content::ResourceContext* resource_context);
+
  protected:
   ResourceDispatcherHostDelegate();
   virtual ~ResourceDispatcherHostDelegate();
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RESOURCE_DISPATCHER_HOST_DELEGATE_H_