Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(599)

Issue 1682403002: Enable adaptive threshold experiment by default. (Closed)

Created:
4 years, 10 months ago by stefan-webrtc
Modified:
4 years, 10 months ago
Reviewers:
mflodman
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com, zhuangzesen_agora.io
Base URL:
https://chromium.googlesource.com/external/webrtc.git@master
Target Ref:
refs/pending/heads/master
Project:
webrtc
Visibility:
Public.

Description

Enable adaptive threshold experiment by default. BUG=webrtc:4711 R=mflodman@webrtc.org Committed: https://crrev.com/44c65e9eed546a390b4dd73e98140b2ae8ae0b7f Cr-Commit-Position: refs/heads/master@{#11640}

Patch Set 1 #

Patch Set 2 : Remove AdaptiveThresholdExperimentIsEnabled. #

Patch Set 3 : Cleanup. #

Patch Set 4 : Tune tests. #

Patch Set 5 : Tune tests #

Messages

Total messages: 22 (8 generated)
stefan-webrtc
4 years, 10 months ago (2016-02-10 15:42:33 UTC) #2
stefan-webrtc
Remove AdaptiveThresholdExperimentIsEnabled.
4 years, 10 months ago (2016-02-10 15:43:38 UTC) #3
stefan-webrtc
Cleanup.
4 years, 10 months ago (2016-02-10 15:44:10 UTC) #4
mflodman
lgtm
4 years, 10 months ago (2016-02-16 08:55:44 UTC) #5
mflodman
On 2016/02/16 08:55:44, mflodman wrote: > lgtm If the intention is to still be able ...
4 years, 10 months ago (2016-02-16 08:56:06 UTC) #6
stefan-webrtc
On 2016/02/16 08:56:06, mflodman wrote: > On 2016/02/16 08:55:44, mflodman wrote: > > lgtm > ...
4 years, 10 months ago (2016-02-16 09:03:58 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1682403002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1682403002/40001
4 years, 10 months ago (2016-02-16 10:01:46 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: linux_rel on tryserver.webrtc (JOB_FAILED, http://build.chromium.org/p/tryserver.webrtc/builders/linux_rel/builds/12723) win_rel on tryserver.webrtc (JOB_FAILED, ...
4 years, 10 months ago (2016-02-16 10:11:30 UTC) #11
stefan-webrtc
Tune tests.
4 years, 10 months ago (2016-02-16 13:09:13 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1682403002/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1682403002/60001
4 years, 10 months ago (2016-02-16 13:09:34 UTC) #15
commit-bot: I haz the power
Try jobs failed on following builders: android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL) win_baremetal on ...
4 years, 10 months ago (2016-02-16 15:10:04 UTC) #17
stefan-webrtc
Tune tests
4 years, 10 months ago (2016-02-16 17:15:09 UTC) #18
stefan-webrtc
Committed patchset #5 (id:80001) manually as 44c65e9eed546a390b4dd73e98140b2ae8ae0b7f (presubmit successful).
4 years, 10 months ago (2016-02-16 17:22:35 UTC) #20
commit-bot: I haz the power
4 years, 10 months ago (2016-02-16 17:22:37 UTC) #22
Message was sent while issue was closed.
Patchset 5 (id:??) landed as
https://crrev.com/44c65e9eed546a390b4dd73e98140b2ae8ae0b7f
Cr-Commit-Position: refs/heads/master@{#11640}

Powered by Google App Engine
This is Rietveld 408576698