Added and implemented HttpStream::GetTotalSentBytes for basic streams.

net/http/http_basic_stream.h

Index: net/http/http_basic_stream.h
diff --git a/net/http/http_basic_stream.h b/net/http/http_basic_stream.h
index 194f7ef6c206380a55d1b4ab002222d9b98eb637..10a2563d0a768c58b14c40d6d39035de875c19ef 100644
--- a/net/http/http_basic_stream.h
+++ b/net/http/http_basic_stream.h
@@ -9,6 +9,8 @@
 #ifndef NET_HTTP_HTTP_BASIC_STREAM_H_
 #define NET_HTTP_HTTP_BASIC_STREAM_H_
 
+#include <stdint.h>
+
 #include <string>
 
 #include "base/basictypes.h"
@@ -66,6 +68,8 @@ class HttpBasicStream : public HttpStream {
 
   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;