Replace sizeof(kGainCycleLength) with kGainCycleLength. This fortunately does not have impact on t…

Replace sizeof(kGainCycleLength) with kGainCycleLength.  This fortunately does not have impact on the server side (where BBR is currently enabled via an experiment), since those two values both happen to be 8.

On Windows and Android, this will result in expanding the possible set
of starting gain cycling offsets from 3 to 7.

This was reported by Felix Weinrank.

Merge internal change: 152933950

R=rch@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2825443002
Cr-Commit-Position: refs/heads/master@{#464933}
Committed: https://chromium.googlesource.com/chromium/src/+/cffb01887fc1b0719aea82138aef58a1054405a8

[Ryan Hamilton, 2017-04-17 13:29:59]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2017-04-17 13:29:59]

lgtm

[commit-bot: I haz the power, 2017-04-17 13:30:07]

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

[Ryan Hamilton, 2017-04-17 13:30:30]

The CQ bit was unchecked by rch@chromium.org

[Ryan Hamilton, 2017-04-17 14:03:25]

If it is a merge, please mention the original CL in the description before
landing.

[Victor Vasiliev, 2017-04-17 16:10:25]

Description was changed from

==========
Replace sizeof(kGainCycleLength) with kGainCycleLength.  This fortunately does
not have impact on the server side (where BBR is currently enabled via an
experiment), since those two values both happen to be 8.

On Windows and Android, this will result in expanding the possible set
of starting gain cycling offsets from 3 to 7.

This was reported by Felix Weinrank.

R=rch@chromium.org
BUG=
==========

to

==========
Replace sizeof(kGainCycleLength) with kGainCycleLength.  This fortunately does
not have impact on the server side (where BBR is currently enabled via an
experiment), since those two values both happen to be 8.

On Windows and Android, this will result in expanding the possible set
of starting gain cycling offsets from 3 to 7.

This was reported by Felix Weinrank.

Merge internal change: 152933950

R=rch@chromium.org
BUG=
==========

[Victor Vasiliev, 2017-04-17 16:16:00]

On 2017/04/17 14:03:25, Ryan Hamilton wrote:
> If it is a merge, please mention the original CL in the description before
> landing.

Added.

[Victor Vasiliev, 2017-04-17 16:16:10]

The CQ bit was checked by vasilvv@chromium.org

[commit-bot: I haz the power, 2017-04-17 16:16:27]

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, 2017-04-17 17:01:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492445770564870, "parent_rev":
"0fe1451d08ef167436488e7dbacbd8d840a5c858", "commit_rev":
"cffb01887fc1b0719aea82138aef58a1054405a8"}

[commit-bot: I haz the power, 2017-04-17 17:01:26]

Description was changed from

==========
Replace sizeof(kGainCycleLength) with kGainCycleLength.  This fortunately does
not have impact on the server side (where BBR is currently enabled via an
experiment), since those two values both happen to be 8.

On Windows and Android, this will result in expanding the possible set
of starting gain cycling offsets from 3 to 7.

This was reported by Felix Weinrank.

Merge internal change: 152933950

R=rch@chromium.org
BUG=
==========

to

==========
Replace sizeof(kGainCycleLength) with kGainCycleLength.  This fortunately does
not have impact on the server side (where BBR is currently enabled via an
experiment), since those two values both happen to be 8.

On Windows and Android, this will result in expanding the possible set
of starting gain cycling offsets from 3 to 7.

This was reported by Felix Weinrank.

Merge internal change: 152933950

R=rch@chromium.org
BUG=

Review-Url: https://codereview.chromium.org/2825443002
Cr-Commit-Position: refs/heads/master@{#464933}
Committed:
https://chromium.googlesource.com/chromium/src/+/cffb01887fc1b0719aea82138aef...
==========

[commit-bot: I haz the power, 2017-04-17 17:01:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/cffb01887fc1b0719aea82138aef...