Added and implemented HttpStream::GetTotalSentBytes for basic streams.

net/http/http_basic_stream.h

 // Copyright (c) 2012 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.
 //
 // HttpBasicStream is a simple implementation of HttpStream.  It assumes it is
 // not sharing a sharing with any other HttpStreams, therefore it just reads and
 // writes directly to the Http Stream.
 
 #ifndef NET_HTTP_HTTP_BASIC_STREAM_H_
 #define NET_HTTP_HTTP_BASIC_STREAM_H_
 
+#include <stdint.h>
+
 #include <string>
 
 #include "base/basictypes.h"
 #include "net/http/http_basic_state.h"
 #include "net/http/http_stream.h"
 
 namespace net {
 
 class BoundNetLog;
 class ClientSocketHandle;
@@ skipping 37 matching lines @@
   bool CanFindEndOfResponse() const override;
 
   bool IsConnectionReused() const override;
 
   void SetConnectionReused() override;
 
   bool IsConnectionReusable() const override;
 
   int64 GetTotalReceivedBytes() const override;
 
+  int64_t GetTotalSentBytes() const override;

[Bence, 2015/09/01 13:45:29]

GetTotalReceivedBytes and GetTotalSentBytes should have the same return type. 
Ideally, either GetTotalReceivedBytes() should be changed to int64_t before this
CL lands, or GetTotalSentBytes() should be int64 for now and the two changed
together in a follow-up CL.  But I understand that that's a lot of extra work,
so I would be okay with this CL as it is as long as you make sure to change
GetTotalReceivedBytes() to int64_t soon after this lands.  (In any case, please
don't change it in this CL.)

Same for HttpStreamParses members and accessors.

[mmenke, 2015/09/01 17:39:43]

I completely missed that, thanks for catching it.  I agree that we should be
consistent here.  I suggest just sticking with int64 for now (Or if you want to
change the other one in a preliminary CL, I'm fine with that, too.

[sclittle, 2015/09/01 20:03:00]

Sure. Do you mind if I keep GetTotalSentBytes as int64_t in this CL, and change
over GetTotalReceivedBytes from int64 to int64_t in a separate CL that I'll put
together right now and send out in parallel?

[mmenke, 2015/09/01 20:09:07]

Works for me

+
   bool GetLoadTimingInfo(LoadTimingInfo* load_timing_info) const override;
 
   void GetSSLInfo(SSLInfo* ssl_info) override;
 
   void GetSSLCertRequestInfo(SSLCertRequestInfo* cert_request_info) override;
 
   bool IsSpdyHttpStream() const override;
 
   void Drain(HttpNetworkSession* session) override;
 
   void SetPriority(RequestPriority priority) override;
 
  private:
   HttpStreamParser* parser() const { return state_.parser(); }
 
   HttpBasicState state_;
 
   DISALLOW_COPY_AND_ASSIGN(HttpBasicStream);
 };
 
 }  // namespace net
 
 #endif  // NET_HTTP_HTTP_BASIC_STREAM_H_