[EasyUnlock] Reduce Bluetooth throttler time from 7 seconds to 2 seconds.

[EasyUnlock] Reduce Bluetooth throttler time from 7 seconds to 1 seconds.

Ideally, we should simply remove this throttling logic altogether, as it was
used to work around some Bluetooth classic bugs. These bugs should no longer be
relevant for BLE.

However, to be safe, we'll simply reduce the throttling time first and see if
this causes insability.

BUG=724719
Review-Url: https://codereview.chromium.org/2896733002
Cr-Commit-Position: refs/heads/master@{#473648}
Committed: https://chromium.googlesource.com/chromium/src/+/e9d3b3997673274a86176c159bf453012b747196

[Tim Song, 2017-05-21 23:17:16]

tengs@chromium.org changed reviewers:
+ hansberry@chromium.org, sacomoto@chromium.org

[sacomoto, 2017-05-22 07:44:46]

lgtm

https://codereview.chromium.org/2896733002/diff/1/components/cryptauth/blueto...
File components/cryptauth/bluetooth_throttler_impl.cc (right):

https://codereview.chromium.org/2896733002/diff/1/components/cryptauth/blueto...
components/cryptauth/bluetooth_throttler_impl.cc:19: const int kCooldownTimeSecs
= 2;
nit: set it to 1.

[Tim Song, 2017-05-22 18:10:22]

Description was changed from

==========
[EasyUnlock] Reduce Bluetooth throttler time from 7 seconds to 2 seconds.

Ideally, we should simply remove this throttling logic altogether, as it was
used to work around some Bluetooth classic bugs. These bugs should no longer be
relevant for BLE.

However, to be safe, we'll simply reduce the throttling time first and see if
this causes insability.

BUG=724719
==========

to

==========
[EasyUnlock] Reduce Bluetooth throttler time from 7 seconds to 1 seconds.

Ideally, we should simply remove this throttling logic altogether, as it was
used to work around some Bluetooth classic bugs. These bugs should no longer be
relevant for BLE.

However, to be safe, we'll simply reduce the throttling time first and see if
this causes insability.

BUG=724719
==========

[Tim Song, 2017-05-22 18:10:27]

https://codereview.chromium.org/2896733002/diff/1/components/cryptauth/blueto...
File components/cryptauth/bluetooth_throttler_impl.cc (right):

https://codereview.chromium.org/2896733002/diff/1/components/cryptauth/blueto...
components/cryptauth/bluetooth_throttler_impl.cc:19: const int kCooldownTimeSecs
= 2;
On 2017/05/22 07:44:46, sacomoto wrote:
> nit: set it to 1.

Done.

[Tim Song, 2017-05-22 18:10:30]

The CQ bit was checked by tengs@chromium.org

[Tim Song, 2017-05-22 18:10:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
sacomoto@chromium.org
Link to the patchset: https://codereview.chromium.org/2896733002/#ps20001
(title: "[EasyUnlock] Reduce Bluetooth throttler time from 7 seconds to 2
seconds.")

[commit-bot: I haz the power, 2017-05-22 18:11:34]

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-05-22 19:11:29]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495476630887030,
"parent_rev": "bf7e0532e34476062226eab6ef1affe78abc8a8d", "commit_rev":
"e9d3b3997673274a86176c159bf453012b747196"}

[commit-bot: I haz the power, 2017-05-22 19:11:37]

Description was changed from

==========
[EasyUnlock] Reduce Bluetooth throttler time from 7 seconds to 1 seconds.

Ideally, we should simply remove this throttling logic altogether, as it was
used to work around some Bluetooth classic bugs. These bugs should no longer be
relevant for BLE.

However, to be safe, we'll simply reduce the throttling time first and see if
this causes insability.

BUG=724719
==========

to

==========
[EasyUnlock] Reduce Bluetooth throttler time from 7 seconds to 1 seconds.

Ideally, we should simply remove this throttling logic altogether, as it was
used to work around some Bluetooth classic bugs. These bugs should no longer be
relevant for BLE.

However, to be safe, we'll simply reduce the throttling time first and see if
this causes insability.

BUG=724719

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

[commit-bot: I haz the power, 2017-05-22 19:11:39]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/e9d3b3997673274a86176c159bf4...

[Ryan Hansberry, 2017-05-22 19:50:39]

lgtm