Add int params to URLDataSource::StartDataRequest().

content/public/browser/url_data_source.h

Index: content/public/browser/url_data_source.h
diff --git a/content/public/browser/url_data_source.h b/content/public/browser/url_data_source.h
index 93d38c0c2ae71c1675c7951a39ad456acb0a950b..acd47fdb60acd0a742e671aae62c738c6a61fe94 100644
--- a/content/public/browser/url_data_source.h
+++ b/content/public/browser/url_data_source.h
@@ -43,12 +43,19 @@ class CONTENT_EXPORT URLDataSource {
   // it's available.
   typedef base::Callback<void(base::RefCountedMemory*)> GotDataCallback;
 
+  // Request fields passed to StartDataRequest in addition to the path.
+  struct ExtraRequestInfo {
+    bool is_incognito;
+    int render_process_id;
+    int render_view_id;
+  };
+
   // Called by URLDataSource to request data at |path|. The string parameter is
   // the path of the request. The child class should run |callback| when the
   // data is available or if the request could not be satisfied. This can be
   // called either in this callback or asynchronously with the response.
   virtual void StartDataRequest(const std::string& path,
-                                bool is_incognito,
+                                const ExtraRequestInfo& info,

[jam, 2013/04/16 23:15:30]

Per the conventions for the Content API
(http://www.chromium.org/developers/content-module/content-api), each
struct/class/enum should be in a separate file. we also try to minimize the
number of files, so I would avoid the struct. this is made easier since
"is_incognito" is now redundant if you pass in the render_process_id and
render_view_is: one can figure out incognito by doing
RenderProcessHost::FromID(render_process_id)->GetBrowserContext()->IsOffTheRecord()

it looks like only 3 of the URLDataSource implementations use this boolean, so
it's pretty easy to convert them to the above.

[Jered, 2013/04/16 23:48:15]

One of the callees who wants to check is_incognito is on the IO thread, but
RenderProcessHost::FromID is only allowed on the UI thread. That caller is just
doing a DCHECK, and I think I can convince the owner to remove it (it's Instant
code); but what would you prefer?

                                 const GotDataCallback& callback) = 0;
 
   // Return the mimetype that should be sent with this response, or empty