Add network parameters fields in FrameStatsMessage

Add network parameters fields in FrameStatsMessage

1. Added rtt_estimate_ms fields for the current network RTT estimate.
2. Added bandwidth_estimate_kbps field for BW estimate used by he host.
3. send_pending_time_ms is set by the host now.

The new fields will allow the receiver to estimate total latency
without per-frame reports from WebRTC APIs in JS.

Committed: https://crrev.com/1e8723caadd302ec806a27f071589b7912b10fe1
Cr-Commit-Position: refs/heads/master@{#426148}

[Sergey Ulanov, 2016-10-17 23:54:50]

Description was changed from

==========
Updated add network paramters fields in FrameStatsMessage

1. Added rtt_estimate_ms fields for the current network RTT estimate.
2. Added bandwidth_estimate_kbps field for BW estimate used by he host.
3. send_pending_time_ms is set by the host now.

The new fields will allow the receiver to estimate total latency
without per-frame reports from WebRTC APIs in JS.
==========

to

==========
Updated add network parameters fields in FrameStatsMessage

1. Added rtt_estimate_ms fields for the current network RTT estimate.
2. Added bandwidth_estimate_kbps field for BW estimate used by he host.
3. send_pending_time_ms is set by the host now.

The new fields will allow the receiver to estimate total latency
without per-frame reports from WebRTC APIs in JS.
==========

[Sergey Ulanov, 2016-10-17 23:56:19]

sergeyu@chromium.org changed reviewers:
+ kelvinp@chromium.org

[Sergey Ulanov, 2016-10-17 23:56:24]

Patchset #1 (id:1) has been deleted

[Sergey Ulanov, 2016-10-18 17:01:39]

Description was changed from

==========
Updated add network parameters fields in FrameStatsMessage

1. Added rtt_estimate_ms fields for the current network RTT estimate.
2. Added bandwidth_estimate_kbps field for BW estimate used by he host.
3. send_pending_time_ms is set by the host now.

The new fields will allow the receiver to estimate total latency
without per-frame reports from WebRTC APIs in JS.
==========

to

==========
Add network parameters fields in FrameStatsMessage

1. Added rtt_estimate_ms fields for the current network RTT estimate.
2. Added bandwidth_estimate_kbps field for BW estimate used by he host.
3. send_pending_time_ms is set by the host now.

The new fields will allow the receiver to estimate total latency
without per-frame reports from WebRTC APIs in JS.
==========

[kelvinp, 2016-10-18 23:34:49]

lgtm

[Sergey Ulanov, 2016-10-18 23:48:08]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2016-10-18 23:48:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-19 00:25:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-19 00:25:35]

Failed to apply patch for remoting/protocol/webrtc_frame_scheduler_simple.h:
While running git apply --index -3 -p1;
  error: patch failed: remoting/protocol/webrtc_frame_scheduler_simple.h:53
  Falling back to three-way merge...
  Applied patch to 'remoting/protocol/webrtc_frame_scheduler_simple.h' with
conflicts.
  U remoting/protocol/webrtc_frame_scheduler_simple.h

Patch:       remoting/protocol/webrtc_frame_scheduler_simple.h
Index: remoting/protocol/webrtc_frame_scheduler_simple.h
diff --git a/remoting/protocol/webrtc_frame_scheduler_simple.h
b/remoting/protocol/webrtc_frame_scheduler_simple.h
index
6a38b39dd1bab9f6f5778c5778ccd3b70b04d566..ef0861e615e902a95698869cda8d231372ff6ec4
100644
--- a/remoting/protocol/webrtc_frame_scheduler_simple.h
+++ b/remoting/protocol/webrtc_frame_scheduler_simple.h
@@ -37,9 +37,9 @@ class WebrtcFrameSchedulerSimple : public
VideoChannelStateObserver,
   bool GetEncoderFrameParams(
       const webrtc::DesktopFrame& frame,
       WebrtcVideoEncoder::FrameParams* params_out) override;
-  void OnFrameEncoded(
-      const WebrtcVideoEncoder::EncodedFrame& encoded_frame,
-      const webrtc::EncodedImageCallback::Result& send_result) override;
+  void OnFrameEncoded(const WebrtcVideoEncoder::EncodedFrame& encoded_frame,
+                      const webrtc::EncodedImageCallback::Result& send_result,
+                      HostFrameStats* frame_stats) override;
 
  private:
   void ScheduleNextFrame(base::TimeTicks now);
@@ -53,6 +53,8 @@ class WebrtcFrameSchedulerSimple : public
VideoChannelStateObserver,
 
   LeakyBucket pacing_bucket_;
 
+  base::TimeDelta rtt_estimate_;
+
   // Set to true when encoding unchanged frames for top-off.
   bool top_off_is_active_ = false;

[Sergey Ulanov, 2016-10-19 04:05:54]

Patchset #2 (id:40001) has been deleted

[Sergey Ulanov, 2016-10-19 05:22:00]

The CQ bit was checked by sergeyu@chromium.org

[Sergey Ulanov, 2016-10-19 05:22:01]

The patchset sent to the CQ was uploaded after l-g-t-m from kelvinp@chromium.org
Link to the patchset: https://codereview.chromium.org/2425873002/#ps60001
(title: ".")

[commit-bot: I haz the power, 2016-10-19 05:23:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-19 11:08:59]

Description was changed from

==========
Add network parameters fields in FrameStatsMessage

1. Added rtt_estimate_ms fields for the current network RTT estimate.
2. Added bandwidth_estimate_kbps field for BW estimate used by he host.
3. send_pending_time_ms is set by the host now.

The new fields will allow the receiver to estimate total latency
without per-frame reports from WebRTC APIs in JS.
==========

to

==========
Add network parameters fields in FrameStatsMessage

1. Added rtt_estimate_ms fields for the current network RTT estimate.
2. Added bandwidth_estimate_kbps field for BW estimate used by he host.
3. send_pending_time_ms is set by the host now.

The new fields will allow the receiver to estimate total latency
without per-frame reports from WebRTC APIs in JS.
==========

[commit-bot: I haz the power, 2016-10-19 11:09:00]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-10-21 13:06:58]

Description was changed from

==========
Add network parameters fields in FrameStatsMessage

1. Added rtt_estimate_ms fields for the current network RTT estimate.
2. Added bandwidth_estimate_kbps field for BW estimate used by he host.
3. send_pending_time_ms is set by the host now.

The new fields will allow the receiver to estimate total latency
without per-frame reports from WebRTC APIs in JS.
==========

to

==========
Add network parameters fields in FrameStatsMessage

1. Added rtt_estimate_ms fields for the current network RTT estimate.
2. Added bandwidth_estimate_kbps field for BW estimate used by he host.
3. send_pending_time_ms is set by the host now.

The new fields will allow the receiver to estimate total latency
without per-frame reports from WebRTC APIs in JS.

Committed: https://crrev.com/1e8723caadd302ec806a27f071589b7912b10fe1
Cr-Commit-Position: refs/heads/master@{#426148}
==========

[commit-bot: I haz the power, 2016-10-21 13:06:59]

Patchset 2 (id:??) landed as
https://crrev.com/1e8723caadd302ec806a27f071589b7912b10fe1
Cr-Commit-Position: refs/heads/master@{#426148}