Adds GetLoadTimingInfo to net::BidirectionalStreamImpl

net/http/bidirectional_stream_impl.h

 // Copyright 2015 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 NET_HTTP_BIDIRECTIONAL_STREAM_IMPL_H_
 #define NET_HTTP_BIDIRECTIONAL_STREAM_IMPL_H_
 
 #include <stdint.h>
 
 #include <memory>
 #include <vector>
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
+#include "net/base/load_timing_info.h"
 #include "net/base/net_export.h"
 #include "net/socket/next_proto.h"
 
 namespace base {
 class Timer;
 }  // namespace base
 
 namespace net {
 
 class BoundNetLog;
@@ skipping 118 matching lines @@
   // push_promise frames associated with this stream, including the size of
   // frame headers, after SSL decryption and not including proxy overhead.
   virtual int64_t GetTotalReceivedBytes() const = 0;
 
   // Total number of bytes sent over the network of SPDY frames associated with
   // this stream, including the size of frame headers, before SSL encryption and
   // not including proxy overhead. Note that some SPDY frames such as pings are
   // not associated with any stream, and are not included in this value.
   virtual int64_t GetTotalSentBytes() const = 0;
 
+  // Populates the connection establishment part of |load_timing_info|, and
+  // socket reuse info.

[mef, 2016/09/20 20:37:49]

Could you document return value? What does return true vs false mean?

[xunjieli, 2016/09/20 20:43:53]

Done.

+  virtual bool GetLoadTimingInfo(LoadTimingInfo* load_timing_info) const = 0;
+
  private:
   DISALLOW_COPY_AND_ASSIGN(BidirectionalStreamImpl);
 };
 
 }  // namespace net
 
 #endif  // NET_HTTP_BIDIRECTIONAL_STREAM_IMPL_H_