Only create |remote_rate| when needed in RemoteBitrateEstimatorSingleStream.

Only create |remote_rate| when needed in RemoteBitrateEstimatorSingleStream.

R=stefan@webrtc.org
BUG=None

Committed: https://crrev.com/ec1a670167c8eb02bea11fb944b6bedf4ddca63a
Cr-Commit-Position: refs/heads/master@{#15264}

[brandtr, 2016-11-28 13:17:18]

https://codereview.webrtc.org/2532113002/diff/1/webrtc/modules/remote_bitrate...
File
webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimator_single_stream.cc
(right):

https://codereview.webrtc.org/2532113002/diff/1/webrtc/modules/remote_bitrate...
webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimator_single_stream.cc:136:
GetRemoteRate()->TimeToReduceFurther(now_ms, *incoming_bitrate_bps))) {
This change (and the one at the end) do not seem to be strictly necessary. Let
me know if you want me to remove them. I guess they do add a little bit of
overhead.

[stefan-webrtc, 2016-11-28 13:18:37]

lgtm

[brandtr, 2016-11-28 13:19:08]

The CQ bit was checked by brandtr@webrtc.org

[commit-bot: I haz the power, 2016-11-28 13:19:13]

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-11-28 13:31:43]

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

[commit-bot: I haz the power, 2016-11-28 13:31:44]

Try jobs failed on following builders:
  linux_msan on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/linux_msan/builds/15274)

[brandtr, 2016-11-28 13:34:41]

The CQ bit was checked by brandtr@webrtc.org

[commit-bot: I haz the power, 2016-11-28 13:34:52]

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

[brandtr, 2016-11-28 13:48:36]

Description was changed from

==========
Only create |remote_rate| when needed in RemoteBitrateEstimatorSingleStream.

R=stefan@webrtc.org
BUG=None
==========

to

==========
Only create |remote_rate| when needed in RemoteBitrateEstimatorSingleStream.

R=stefan@webrtc.org
BUG=None

Committed:
https://chromium.googlesource.com/external/webrtc/+/ec1a670167c8eb02bea11fb94...
==========

[brandtr, 2016-11-28 13:48:37]

Committed patchset #1 (id:1) manually as
ec1a670167c8eb02bea11fb944b6bedf4ddca63a (presubmit successful).

[commit-bot: I haz the power, 2016-11-28 13:48:39]

Description was changed from

==========
Only create |remote_rate| when needed in RemoteBitrateEstimatorSingleStream.

R=stefan@webrtc.org
BUG=None

Committed:
https://chromium.googlesource.com/external/webrtc/+/ec1a670167c8eb02bea11fb94...
==========

to

==========
Only create |remote_rate| when needed in RemoteBitrateEstimatorSingleStream.

R=stefan@webrtc.org
BUG=None

Committed: https://crrev.com/ec1a670167c8eb02bea11fb944b6bedf4ddca63a
Cr-Commit-Position: refs/heads/master@{#15264}
==========