Cronet modifications to support AGSA.

components/cronet/android/url_request_adapter.cc

Index: components/cronet/android/url_request_adapter.cc
diff --git a/components/cronet/android/url_request_adapter.cc b/components/cronet/android/url_request_adapter.cc
index 8ab88a2f3cd766629e0033c3f231381992528f13..41e8448e865f6418f3f4f3d58b4a94bc36db111b 100644
--- a/components/cronet/android/url_request_adapter.cc
+++ b/components/cronet/android/url_request_adapter.cc
@@ -23,15 +23,16 @@ URLRequestAdapter::URLRequestAdapter(URLRequestContextAdapter* context,
       url_request_(NULL),
       read_buffer_(new net::GrowableIOBuffer()),
       bytes_read_(0),
       total_bytes_read_(0),
       error_code_(0),
       http_status_code_(0),
       canceled_(false),
-      expected_size_(0) {
+      expected_size_(0),
+      chunked_upload_(false) {
   context_ = context;
   delegate_ = delegate;
   url_ = url;
   priority_ = priority;
 }
 
 URLRequestAdapter::~URLRequestAdapter() {
@@ -58,14 +59,32 @@ void URLRequestAdapter::SetUploadContent(const char* bytes, int bytes_len) {
 void URLRequestAdapter::SetUploadChannel(JNIEnv* env, int64 content_length) {
   scoped_ptr<net::UploadElementReader> reader(
       new WrappedChannelElementReader(delegate_, content_length));
   upload_data_stream_.reset(
       net::UploadDataStream::CreateWithReader(reader.Pass(), 0));
 }
 
+void URLRequestAdapter::EnableChunkedUpload() {
+  chunked_upload_ = true;
+}
+
+void URLRequestAdapter::AppendChunk(const char* bytes,
+                                 int bytes_len,
+                                 bool is_last_chunk) {
+  VLOG(1) << "AppendChunk, len: " << bytes_len << ", last: " << is_last_chunk;
+
+  context_->GetNetworkTaskRunner()->PostTask(
+      FROM_HERE,
+      base::Bind(&URLRequestAdapter::OnAppendChunk,
+                 base::Unretained(this),
+                 bytes,
+                 bytes_len,
+                 is_last_chunk));
+}
+
 std::string URLRequestAdapter::GetHeader(const std::string& name) const {
   std::string value;
   if (url_request_ != NULL) {
     url_request_->GetResponseHeaderByName(name, &value);
   }
   return value;
 }
@@ -73,21 +92,38 @@ std::string URLRequestAdapter::GetHeader(const std::string& name) const {
 net::HttpResponseHeaders* URLRequestAdapter::GetResponseHeaders() const {
   if (url_request_ == NULL) {
     return NULL;
   }
   return url_request_->response_headers();
 }
 
+bool URLRequestAdapter::GetFullRequestHeaders(

[mef, 2014/08/15 15:42:52]

Do you need this on the Java side?

[mdumitrescu, 2014/08/15 15:50:20]

I was looking into this. It seems we don't right now so I might remove this (or
add it to the java api).

+    net::HttpRequestHeaders* headers) const {
+  if (url_request_ == NULL) {
+    return false;
+  }
+  return url_request_->GetFullRequestHeaders(headers);
+}
+
 void URLRequestAdapter::Start() {
   context_->GetNetworkTaskRunner()->PostTask(
       FROM_HERE,
       base::Bind(&URLRequestAdapter::OnInitiateConnection,
                  base::Unretained(this)));
 }
 
+void URLRequestAdapter::OnAppendChunk(const char* bytes,
+                                   int bytes_len,
+                                   bool is_last_chunk) {

[mef, 2014/08/15 13:30:22]

nit: align with 'const'.

[mdumitrescu, 2014/08/15 15:23:13]

Done.

+  if (url_request_ != NULL) {
+    url_request_->AppendChunkToUpload(bytes, bytes_len, is_last_chunk);
+    delegate_->OnAppendChunkCompleted(this);
+  }
+}
+
 void URLRequestAdapter::OnInitiateConnection() {
   if (canceled_) {
     return;
   }
 
   VLOG(1) << "Starting chromium request: "
           << url_.possibly_invalid_spec().c_str()
@@ -102,16 +138,19 @@ void URLRequestAdapter::OnInitiateConnection() {
   if (!headers_.HasHeader(net::HttpRequestHeaders::kUserAgent)) {
     std::string user_agent;
     user_agent = context_->GetUserAgent(url_);
     url_request_->SetExtraRequestHeaderByName(
         net::HttpRequestHeaders::kUserAgent, user_agent, true /* override */);
   }
 
-  if (upload_data_stream_)
-    url_request_->set_upload(upload_data_stream_.Pass());

[mef, 2014/08/15 13:30:22]

Pass() is fine here.

[mdumitrescu, 2014/08/15 15:23:13]

Oops. Unintended change.

+  if (upload_data_stream_) {
+    url_request_->set_upload(make_scoped_ptr(upload_data_stream_.release()));
+  } else if (chunked_upload_) {
+    url_request_->EnableChunkedUpload();
+  }
 
   url_request_->SetPriority(priority_);
 
   url_request_->Start();
 }
 
 void URLRequestAdapter::Cancel() {