Enable send-side BWE by default.

Enable send-side BWE by default.

BUG=webrtc:4173
R=mflodman@webrtc.org

Committed: https://crrev.com/06a5e1aa3982177e875ddc10c82b1fa4f14393ce
Cr-Commit-Position: refs/heads/master@{#14041}

[stefan-webrtc, 2016-09-01 13:40:48]

The CQ bit was checked by stefan@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-01 13:41:02]

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

[stefan-webrtc, 2016-09-01 13:50:22]

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

[commit-bot: I haz the power, 2016-09-01 15:41:34]

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

[commit-bot: I haz the power, 2016-09-01 15:41:35]

Dry run: Try jobs failed on following builders:
  android_arm64_rel 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)

[mflodman, 2016-09-02 08:33:23]

One comment, then LGTM.

https://codereview.webrtc.org/2300003002/diff/20001/webrtc/sdk/objc/Framework...
File webrtc/sdk/objc/Framework/Classes/RTCFieldTrials.mm (right):

https://codereview.webrtc.org/2300003002/diff/20001/webrtc/sdk/objc/Framework...
webrtc/sdk/objc/Framework/Classes/RTCFieldTrials.mm:19: static NSString * const
kRTCEnableSendSideBweString =
Remove this too.

[stefan-webrtc, 2016-09-02 08:38:49]

The CQ bit was checked by stefan@webrtc.org

[stefan-webrtc, 2016-09-02 08:38:52]

The CQ bit was unchecked by stefan@webrtc.org

[stefan-webrtc, 2016-09-02 08:39:41]

nit addressed.

[stefan-webrtc, 2016-09-02 08:39:58]

https://codereview.webrtc.org/2300003002/diff/20001/webrtc/sdk/objc/Framework...
File webrtc/sdk/objc/Framework/Classes/RTCFieldTrials.mm (right):

https://codereview.webrtc.org/2300003002/diff/20001/webrtc/sdk/objc/Framework...
webrtc/sdk/objc/Framework/Classes/RTCFieldTrials.mm:19: static NSString * const
kRTCEnableSendSideBweString =
On 2016/09/02 08:33:23, mflodman wrote:
> Remove this too.

Done.

[stefan-webrtc, 2016-09-02 08:40:12]

The CQ bit was checked by stefan@webrtc.org

[stefan-webrtc, 2016-09-02 08:40:13]

The patchset sent to the CQ was uploaded after l-g-t-m from mflodman@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2300003002/#ps40001 (title:
"nit addressed.")

[commit-bot: I haz the power, 2016-09-02 08:40:21]

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

[stefan-webrtc, 2016-09-02 10:37:06]

Description was changed from

==========
Enable send-side BWE by default.

BUG=webrtc:4173
==========

to

==========
Enable send-side BWE by default.

BUG=webrtc:4173
R=mflodman@webrtc.org

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

[commit-bot: I haz the power, 2016-09-02 10:37:06]

Description was changed from

==========
Enable send-side BWE by default.

BUG=webrtc:4173
==========

to

==========
Enable send-side BWE by default.

BUG=webrtc:4173
R=mflodman@webrtc.org

Committed: https://crrev.com/06a5e1aa3982177e875ddc10c82b1fa4f14393ce
Cr-Commit-Position: refs/heads/master@{#14041}
==========

[commit-bot: I haz the power, 2016-09-02 10:37:06]

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

[stefan-webrtc, 2016-09-02 10:37:07]

Committed patchset #3 (id:40001) manually as
06a5e1aa3982177e875ddc10c82b1fa4f14393ce (presubmit successful).