Implement timing metrics for UrlRequest

components/cronet/android/cronet_url_request_adapter.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_ADAPTER_H_
 #define COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_ADAPTER_H_
 
 #include <jni.h>
 
 #include <memory>
@@ skipping 42 matching lines @@
   // use cache, |jdisable_cache| has no effect. |jdisable_connection_migration|
   // causes connection migration to be disabled for this request if true. If
   // global connection migration flag is not enabled,
   // |jdisable_connection_migration| has no effect.
   CronetURLRequestAdapter(CronetURLRequestContextAdapter* context,
                           JNIEnv* env,
                           jobject jurl_request,
                           const GURL& url,
                           net::RequestPriority priority,
                           jboolean jdisable_cache,
-                          jboolean jdisable_connection_migration);
+                          jboolean jdisable_connection_migration,
+                          jboolean jenable_metrics);
   ~CronetURLRequestAdapter() override;
 
   // Methods called prior to Start are never called on network thread.
 
   // Sets the request method GET, POST etc.
   jboolean SetHttpMethod(JNIEnv* env,
                          const base::android::JavaParamRef<jobject>& jcaller,
                          const base::android::JavaParamRef<jstring>& jmethod);
 
   // Adds a header to the request before it starts.
@@ skipping 58 matching lines @@
       JNIEnv* env);
   void FollowDeferredRedirectOnNetworkThread();
   void ReadDataOnNetworkThread(
       scoped_refptr<IOBufferWithByteBuffer> read_buffer,
       int buffer_size);
   void DestroyOnNetworkThread(bool send_on_canceled);
 
   // Checks status of the request_adapter, return false if |is_success()| is
   // true, otherwise report error and cancel request_adapter.
   bool MaybeReportError(net::URLRequest* request) const;
+  // Reports metrics collected to the Java layer
+  void MaybeReportMetrics(net::URLRequest* request) const;
+  int64_t ConvertTime(base::TimeTicks ticks,

[xunjieli, 2016/09/20 14:32:46]

#include base/time/time.h

[mgersh, 2016/09/20 20:48:25]

Done.

+                      base::Time start_time,
+                      base::TimeTicks start_ticks) const;
 
   CronetURLRequestContextAdapter* context_;
 
   // Java object that owns this CronetURLRequestContextAdapter.
   base::android::ScopedJavaGlobalRef<jobject> owner_;
 
   const GURL initial_url_;
   const net::RequestPriority initial_priority_;
   std::string initial_method_;
   int load_flags_;
   net::HttpRequestHeaders initial_request_headers_;
   std::unique_ptr<net::UploadDataStream> upload_;
 
   scoped_refptr<IOBufferWithByteBuffer> read_buffer_;
   std::unique_ptr<net::URLRequest> url_request_;
 
+  bool enable_metrics_;

[xunjieli, 2016/09/20 14:32:46]

nit: Add a documentation. It's clear what this means now, but the meaning might
be obscured when this file grows in size.

[mgersh, 2016/09/20 20:48:24]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(CronetURLRequestAdapter);
 };
 
 }  // namespace cronet
 
 #endif  // COMPONENTS_CRONET_ANDROID_CRONET_URL_REQUEST_ADAPTER_H_