[on hold] Remove dependency on RenderViewHost from LoadState notifications

content/browser/loader/resource_dispatcher_host_impl.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.
 
 // This is the browser side of the resource dispatcher, it receives requests
 // from the child process (i.e. [Renderer, Plugin, Worker]ProcessHost), and
 // dispatches them to URLRequests. It then forwards the messages from the
 // URLRequests back to the correct process for handling.
 //
 // See http://dev.chromium.org/developers/design-documents/multi-process-resource-loading
@@ skipping 12 matching lines @@
 #include "base/gtest_prod_util.h"
 #include "base/macros.h"
 #include "base/observer_list.h"
 #include "base/time/time.h"
 #include "base/timer/timer.h"
 #include "content/browser/download/download_resource_handler.h"
 #include "content/browser/download/save_types.h"
 #include "content/browser/loader/global_routing_id.h"
 #include "content/browser/loader/resource_loader.h"
 #include "content/browser/loader/resource_loader_delegate.h"
+#include "content/browser/loader/resource_request_info_impl.h"
 #include "content/browser/loader/resource_scheduler.h"
 #include "content/common/content_export.h"
 #include "content/common/resource_request_body.h"
 #include "content/public/browser/child_process_data.h"
 #include "content/public/browser/download_item.h"
 #include "content/public/browser/download_url_parameters.h"
 #include "content/public/browser/global_request_id.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/resource_dispatcher_host.h"
 #include "content/public/browser/web_contents_observer.h"
@@ skipping 292 matching lines @@
 
   struct OustandingRequestsStats {
     int memory_cost;
     int num_requests;
   };
 
   friend class ShutdownTask;
   friend class ResourceMessageDelegate;
 
   // Information about status of a ResourceLoader.
-  struct LoadInfo {
+  struct CONTENT_EXPORT LoadInfo {
+    LoadInfo();
+    LoadInfo(const LoadInfo& info);
+    ~LoadInfo();
+
+    bool IsUploading() const;
+
+    // This function returns true if the LoadInfo of |a| is "more interesting"
+    // than the LoadInfo of |b|.  The load that is currently sending the larger
+    // request body is considered more interesting.  If neither request is
+    // sending a body (Neither request has a body, or any request that has a
+    // body is not currently sending the body), the request that is further
+    // along is considered more interesting.
+    //
+    // This takes advantage of the fact that the load states are an enumeration
+    // listed in the order in which they usually occur during the lifetime of a
+    // request, so states with larger numeric values are generally further along
+    // toward completion.
+    //
+    // For example, by this measure "tranferring data" is a more interesting
+    // state than "resolving host" because when transferring data something is
+    // being done that corresponds to changes that the user might observe,
+    // whereas waiting for a host name to resolve implies being stuck.
+    bool IsMoreInteresting(const LoadInfo& other) const;
+
     GURL url;
     net::LoadStateWithParam load_state;
     uint64_t upload_position;
     uint64_t upload_size;
+
+    // Associates the load with a web contents.
+    ResourceRequestInfo::WebContentsGetter web_contents_getter;
   };
 
-  // Map from ProcessID+RouteID pair to the "most interesting" LoadState.
-  typedef std::map<GlobalRoutingID, LoadInfo> LoadInfoMap;
+  // Map from ProcessID+FrameRouteID pair to the "most interesting" LoadState.
+  typedef std::map<GlobalFrameRoutingId, LoadInfo> LoadInfoMap;
 
   // ResourceLoaderDelegate implementation:
   ResourceDispatcherHostLoginDelegate* CreateLoginDelegate(
       ResourceLoader* loader,
       net::AuthChallengeInfo* auth_info) override;
   bool HandleExternalProtocol(ResourceLoader* loader, const GURL& url) override;
   void DidStartRequest(ResourceLoader* loader) override;
   void DidReceiveRedirect(ResourceLoader* loader, const GURL& new_url) override;
   void DidReceiveResponse(ResourceLoader* loader) override;
   void DidFinishLoading(ResourceLoader* loader) override;
@@ skipping 59 matching lines @@
   // It may be enhanced in the future to provide some kind of prioritization
   // mechanism. We should also consider a hashtable or binary tree if it turns
   // out we have a lot of things here.
   using LoaderMap = std::map<GlobalRequestID, std::unique_ptr<ResourceLoader>>;
 
   // Deletes the pending request identified by the iterator passed in.
   // This function will invalidate the iterator passed in. Callers should
   // not rely on this iterator being valid on return.
   void RemovePendingLoader(const LoaderMap::iterator& iter);
 
-  // This function returns true if the LoadInfo of |a| is "more interesting"
-  // than the LoadInfo of |b|.  The load that is currently sending the larger
-  // request body is considered more interesting.  If neither request is
-  // sending a body (Neither request has a body, or any request that has a body
-  // is not currently sending the body), the request that is further along is
-  // considered more interesting.
-  //
-  // This takes advantage of the fact that the load states are an enumeration
-  // listed in the order in which they usually occur during the lifetime of a
-  // request, so states with larger numeric values are generally further along
-  // toward completion.
-  //
-  // For example, by this measure "tranferring data" is a more interesting state
-  // than "resolving host" because when transferring data something is being
-  // done that corresponds to changes that the user might observe, whereas
-  // waiting for a host name to resolve implies being stuck.
-  static bool LoadInfoIsMoreInteresting(const LoadInfo& a, const LoadInfo& b);
-
   // Used to marshal calls to LoadStateChanged from the IO to UI threads.  All
-  // are done as a single callback to avoid spamming the UI thread.
+  // are done as a single callback to avoid spamming the UI thread. Note that
+  // GetLoadInfoForAllRoutes de-dupes based on frame, but not based on tab.
+  // Thus, logic here also constructs a map and finds the most interesting load
+  // state per WebContents.
   static void UpdateLoadInfoOnUIThread(std::unique_ptr<LoadInfoMap> info_map);
 
-  // Gets the most interesting LoadInfo for each GlobalRoutingID.
+  // Gets the most interesting LoadInfo for each GlobalFrameRoutingID.
   std::unique_ptr<LoadInfoMap> GetLoadInfoForAllRoutes();
 
   // Checks all pending requests and updates the load info if necessary.
   void UpdateLoadInfo();
 
   // Resumes or cancels (if |cancel_requests| is true) any blocked requests.
   void ProcessBlockedRequestsForRoute(
       const GlobalFrameRoutingId& global_routing_id,
       bool cancel_requests);
 
@@ skipping 196 matching lines @@
   // Allows tests to use a mock CertStore. If set, the CertStore must
   // outlive this ResourceDispatcherHostImpl.
   CertStore* cert_store_for_testing_;
 
   DISALLOW_COPY_AND_ASSIGN(ResourceDispatcherHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_LOADER_RESOURCE_DISPATCHER_HOST_IMPL_H_