[Cronet] Make metrics reporting happen after terminal callbacks.

components/cronet/android/cronet_url_request_adapter.h

Index: components/cronet/android/cronet_url_request_adapter.h
diff --git a/components/cronet/android/cronet_url_request_adapter.h b/components/cronet/android/cronet_url_request_adapter.h
index a57a305b5b1bca79378d54f307652c0474036970..6d5467156d6bd6b6a1adc7a5cee97a4e7320640d 100644
--- a/components/cronet/android/cronet_url_request_adapter.h
+++ b/components/cronet/android/cronet_url_request_adapter.h
@@ -132,12 +132,12 @@ class CronetURLRequestAdapter : public net::URLRequest::Delegate {
   void ReadDataOnNetworkThread(
       scoped_refptr<IOBufferWithByteBuffer> read_buffer,
       int buffer_size);
-  void DestroyOnNetworkThread(bool send_on_canceled);
+  void DestroyOnNetworkThread(bool jsend_on_canceled);
 
   // Report error and cancel request_adapter.
   void ReportError(net::URLRequest* request, int net_error);
   // Reports metrics collected to the Java layer
-  void MaybeReportMetrics(JNIEnv* env) const;
+  void MaybeReportMetrics(JNIEnv* env);
 
   CronetURLRequestContextAdapter* context_;
 
@@ -154,9 +154,10 @@ class CronetURLRequestAdapter : public net::URLRequest::Delegate {
   scoped_refptr<IOBufferWithByteBuffer> read_buffer_;
   std::unique_ptr<net::URLRequest> url_request_;
 
-  // Whether detailed metrics should be collected and reported to Java for this
-  // request.
+  // Whether detailed metrics should be collected and reported to Java.
   const bool enable_metrics_;
+  // Whether metrics has been reported to Java.

[mef, 2017/04/28 21:16:54]

nit: have

[xunjieli, 2017/05/01 19:17:15]

Done.

+  bool metrics_reported_;
 
   DISALLOW_COPY_AND_ASSIGN(CronetURLRequestAdapter);
 };