Generate Cronet Version.java based on VERSION and LASTCHANGE files.

components/cronet/android/url_request_peer.cc

Index: components/cronet/android/url_request_peer.cc
diff --git a/components/cronet/android/url_request_peer.cc b/components/cronet/android/url_request_peer.cc
index 54094bf6773fc760639c32b003d348ae6d816922..e659d3281f60d3303b1a4c620598ccebec9cc237 100644
--- a/components/cronet/android/url_request_peer.cc
+++ b/components/cronet/android/url_request_peer.cc
@@ -12,10 +12,6 @@ namespace cronet {
 
 static const size_t kBufferSizeIncrement = 8192;
 
-// Fragment automatically inserted in the User-Agent header to indicate
-// that the request is coming from this network stack.
-static const char kUserAgentFragment[] = "; ChromiumJNI/";
-
 URLRequestPeer::URLRequestPeer(URLRequestContextPeer* context,
                                URLRequestPeerDelegate* delegate,
                                GURL url,
@@ -116,11 +112,6 @@ void URLRequestPeer::OnInitiateConnection() {
   } else {
     user_agent = context_->GetUserAgent(url_);

[mmenke, 2014/06/20 17:41:16]

We never run this line, since we always pass in the user agent, right?

[mef, 2014/06/20 18:34:04]

Isn't it possible, that there is no User-Agent header yet?

[mmenke, 2014/06/20 19:03:59]

Ahh...you're right.  I was thinking we were adding it to the headers in
UrlRequest.java, not through the context (At some point, may want to switch - a
little more overhead to sending another string per request through Java, but
less code to do stuff here as well...Anyhow, not a huge difference)

   }
-  size_t pos = user_agent.find(')');
-  if (pos != std::string::npos) {
-    user_agent.insert(pos, context_->version());
-    user_agent.insert(pos, kUserAgentFragment);
-  }
   url_request_->SetExtraRequestHeaderByName(
       net::HttpRequestHeaders::kUserAgent, user_agent, true /* override */);

[mmenke, 2014/06/20 17:41:16]

I don't believe this does anything if
headers_.HasHeader(net::HttpRequestHeaders::kUserAgent) is true - we just
overwrite the old value with the same value.

[mef, 2014/06/20 18:34:04]

Done. It used to stick Cronet version into it, but that doesn't make sense now.
Done.