commit e448a3fb54eec298d9113b2bd85e71e2fe69bb77
author Marina Ciocea <marinaciocea@webrtc.org> Mon Mar 04 14:52:21 2019
committer Commit Bot <commit-bot@chromium.org> Mon Mar 11 13:21:36 2019
tree e0ff3c30cdc67cdfa277a49c07e9b1cc52d8eb56
parent ad89528051f9478cf8738c0dfd264d4a716b4698

Update DataChannel bufferedamount implementation.

Call DataChannelObserver::OnBufferedAmountChange on each successful send.
Previously, the observer would get notified of buffered amount changes only when
queued send data is consumed. Data gets queued only if it cannot be sent right
away. According to the WebRTC standard[1], bufferedamount should be increased
before each sent and decreased after each successful sent. Update implementation
to be standard compliant.

Design doc: http://doc/1lorHBn-GMn5U0T0RQANxrsW0pXhw8XGZM-xZyVUOW90

[1] https://w3c.github.io/webrtc-pc/#dom-datachannel-bufferedamount

Bug: chromium:878682
Change-Id: Ife009d30c4a18dced9a54cf600a445bb1f02561d
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/123237
Commit-Queue: Marina Ciocea <marinaciocea@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27057}

api/data_channel_interface.h

diff --git a/api/data_channel_interface.h b/api/data_channel_interface.h
index 91a9804..1bd874e 100644
--- a/api/data_channel_interface.h
+++ b/api/data_channel_interface.h
@@ -87,7 +87,7 @@
   //  A data buffer was successfully received.
   virtual void OnMessage(const DataBuffer& buffer) = 0;
   // The data channel's buffered_amount has changed.
-  virtual void OnBufferedAmountChange(uint64_t previous_amount) {}
+  virtual void OnBufferedAmountChange(uint64_t sent_data_size) {}
 
  protected:
   virtual ~DataChannelObserver() = default;