Change rtc::TimeNanos and rtc::TimeMicros return value from uint64_t to int64_t.

webrtc/api/stats/rtcstatsreport.h

Index: webrtc/api/stats/rtcstatsreport.h
diff --git a/webrtc/api/stats/rtcstatsreport.h b/webrtc/api/stats/rtcstatsreport.h
index 05bc529550085f8bd45c912eb07b1b56766fa646..32514e765a21e2791a989896512b138565cf82c5 100644
--- a/webrtc/api/stats/rtcstatsreport.h
+++ b/webrtc/api/stats/rtcstatsreport.h
@@ -52,12 +52,12 @@ class RTCStatsReport : public rtc::RefCountInterface {
 
   // TODO(hbos): Remove "= 0" once Chromium unittest has been updated to call
   // with a parameter. crbug.com/627816
-  static rtc::scoped_refptr<RTCStatsReport> Create(uint64_t timestamp_us = 0);
+  static rtc::scoped_refptr<RTCStatsReport> Create(int64_t timestamp_us = 0);
 
-  explicit RTCStatsReport(uint64_t timestamp_us);
+  explicit RTCStatsReport(int64_t timestamp_us);
   RTCStatsReport(const RTCStatsReport& other) = delete;
 
-  uint64_t timestamp_us() const { return timestamp_us_; }
+  int64_t timestamp_us() const { return timestamp_us_; }

[kwiberg-webrtc, 2016/11/21 12:12:34]

These are also changes in the public API, right? See the other comment.

[nisse-webrtc, 2016/11/21 12:39:12]

If at all possible, I'd like to change the return type *without* renaming the
method (which as far as I understand is the only way to transition gradually).

   bool AddStats(std::unique_ptr<const RTCStats> stats);
   const RTCStats* Get(const std::string& id) const;
   size_t size() const { return stats_.size(); }
@@ -90,7 +90,7 @@ class RTCStatsReport : public rtc::RefCountInterface {
  private:
   ~RTCStatsReport() override;
 
-  uint64_t timestamp_us_;
+  int64_t timestamp_us_;
   StatsMap stats_;
 };