Add a WebContents getter callback in ResourceRequestInfo

content/public/browser/resource_request_info.h

Index: content/public/browser/resource_request_info.h
diff --git a/content/public/browser/resource_request_info.h b/content/public/browser/resource_request_info.h
index b6edad7236c8eb2b76693c22c1bb0ef4fbb8c5ef..91f601a29375901368ca25e59d3279b5cb3c2ffb 100644
--- a/content/public/browser/resource_request_info.h
+++ b/content/public/browser/resource_request_info.h
@@ -6,7 +6,9 @@
 #define CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_INFO_H_
 
 #include "base/basictypes.h"
+#include "base/callback.h"

[davidben, 2015/11/19 17:46:13]

callback_forward.h

[clamy, 2015/11/20 13:50:43]

Done.

 #include "content/common/content_export.h"
+#include "content/public/browser/web_contents.h"
 #include "content/public/common/resource_type.h"
 #include "third_party/WebKit/public/platform/WebPageVisibilityState.h"
 #include "third_party/WebKit/public/platform/WebReferrerPolicy.h"
@@ -51,6 +53,12 @@ class ResourceRequestInfo {
       int* render_process_id,
       int* render_frame_id);
 
+  // Returns a callback that returns a pointer to the WebContents this request
+  // is asscoaited with, or nullptr if it does not exit. The callback should

[davidben, 2015/11/19 17:46:13]

asscoaited -> associated
exit -> exist

Probably also add:

 // Note: not all resource requests will be owned by a WebContents. For example,
requests made by a ServiceWorker.

(Aside: Really, this ought to be "is owned by", I think, but it might be too
early for that? Downloads still return WebContentses right now and I'd like to
disconnect that, once the ResourceLoader becomes more cleaned up.)

[clamy, 2015/11/20 13:50:43]

Done.

+  // only run on the UI thread.
+  using WebContentsGetterOnUI = base::Callback<WebContents*(void)>;

[davidben, 2015/11/19 17:46:13]

base::Callback<WebContents*(void)> isn't much longer. I think I might advocate
just using that, especially since it shouldn't appear in all that many type
signatures. I don't have strong preferences though.

[clamy, 2015/11/20 13:50:43]

Done.

+  virtual WebContentsGetterOnUI GetWebContentsForRequest() = 0;
+
   // Returns the associated ResourceContext.
   virtual ResourceContext* GetContext() const = 0;