Only log BWE updates if the actual estimate changed or if we have non-zero loss reports.

Only log BWE updates if the actual estimate changed or if we have non-zero loss reports.

BUG=webrtc:6295
Committed: https://crrev.com/3821ff8df57e8d9622b13ea0636bf833ab4733a4
Cr-Commit-Position: refs/heads/master@{#14061}

[stefan-webrtc, 2016-09-02 10:19:49]

Add periodic updates.

[stefan-webrtc, 2016-09-02 10:36:12]

stefan@webrtc.org changed reviewers:
+ philipel@webrtc.org

[philipel, 2016-09-02 11:26:07]

https://codereview.webrtc.org/2306963002/diff/20001/webrtc/modules/bitrate_co...
File webrtc/modules/bitrate_controller/send_side_bandwidth_estimation.cc
(right):

https://codereview.webrtc.org/2306963002/diff/20001/webrtc/modules/bitrate_co...
webrtc/modules/bitrate_controller/send_side_bandwidth_estimation.cc:254: if
(capped_bitrate != bitrate_ || last_fraction_loss_ > 0 ||
Maybe add the same type of logic to fraction loss, i.e. only log in the case of
a change in packet loss (line 151).

[stefan-webrtc, 2016-09-02 13:03:10]

.

[stefan-webrtc, 2016-09-02 13:13:18]

https://codereview.webrtc.org/2306963002/diff/20001/webrtc/modules/bitrate_co...
File webrtc/modules/bitrate_controller/send_side_bandwidth_estimation.cc
(right):

https://codereview.webrtc.org/2306963002/diff/20001/webrtc/modules/bitrate_co...
webrtc/modules/bitrate_controller/send_side_bandwidth_estimation.cc:254: if
(capped_bitrate != bitrate_ || last_fraction_loss_ > 0 ||
On 2016/09/02 11:26:07, philipel wrote:
> Maybe add the same type of logic to fraction loss, i.e. only log in the case
of
> a change in packet loss (line 151).

Done.

[philipel, 2016-09-02 13:19:07]

lgtm

[stefan-webrtc, 2016-09-02 13:56:10]

The CQ bit was checked by stefan@webrtc.org

[commit-bot: I haz the power, 2016-09-02 13:56:16]

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

[commit-bot: I haz the power, 2016-09-02 15:56:47]

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

[commit-bot: I haz the power, 2016-09-02 15:56:48]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_clang_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_clang_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_drmemory_light on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_clang_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_clang_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_gyp_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_gyp_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[stefan-webrtc, 2016-09-04 11:20:45]

The CQ bit was checked by stefan@webrtc.org

[commit-bot: I haz the power, 2016-09-04 11:20:49]

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

[commit-bot: I haz the power, 2016-09-04 12:07:29]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-04 12:07:42]

Description was changed from

==========
Only log BWE updates if the actual estimate changed or if we have non-zero loss
reports.

BUG=webrtc:6295
==========

to

==========
Only log BWE updates if the actual estimate changed or if we have non-zero loss
reports.

BUG=webrtc:6295

Committed: https://crrev.com/3821ff8df57e8d9622b13ea0636bf833ab4733a4
Cr-Commit-Position: refs/heads/master@{#14061}
==========

[commit-bot: I haz the power, 2016-09-04 12:07:47]

Patchset 3 (id:??) landed as
https://crrev.com/3821ff8df57e8d9622b13ea0636bf833ab4733a4
Cr-Commit-Position: refs/heads/master@{#14061}