Stop using ScopedAllowIO in content::ResourceDispatcherHostImpl

net/url_request/url_request.h

Index: net/url_request/url_request.h
diff --git a/net/url_request/url_request.h b/net/url_request/url_request.h
index 59b5a28ee024fcd4579eafc71a8c427bdf069018..e83158ffd98504e5d2431e3192eb42f406cb7fff 100644
--- a/net/url_request/url_request.h
+++ b/net/url_request/url_request.h
@@ -235,6 +235,9 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
   //
   class NET_EXPORT Delegate {
    public:
+    // Called when the request body (upload data) is initialized.
+    virtual void OnRequestBodyInitialized(URLRequest* request, uint64 size);

[darin (slow to review), 2012/08/14 18:24:27]

Does this really need to be an event?  Why not just let the caller poll for the
upload size?  Have a GetUploadSize function that returns -1 or fail if the
upload size is unknown?  I'd also be OK with GetUploadProgress having an out
parameter to report the total size:

  // Returns true if progress has been made uploading data and reports
  // the upload progress and total upload size in bytes.  Otherwise,
  // returns false if there is no upload data or if we haven't begun
  // uploading data yet.
  bool GetUploadProgress(uint64* progress, uint64* total) const;

In other words, since we don't need an edge trigger for when the upload begins,
we could very well do away with the event and just simply provide a polling
interface.

+
     // Called upon a server-initiated redirect.  The delegate may call the
     // request's Cancel method to prevent the redirect from being followed.
     // Since there may be multiple chained redirects, there may also be more
@@ -638,6 +641,9 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
   void set_stack_trace(const base::debug::StackTrace& stack_trace);
   const base::debug::StackTrace* stack_trace() const;
 
+  // Called by HttpNetworkTransaction when the request body is initialized.
+  void NotifyRequestBodyInitialized(uint64 size);
+
  protected:
   // Allow the URLRequestJob class to control the is_pending() flag.
   void set_is_pending(bool value) { is_pending_ = value; }