[EasyUnlock] Observe proximity changes and clean up TX power strategy.

[EasyUnlock] Observe proximity changes and clean up TX power strategy.

With BLE, we only check the RSSI, so we can now remove all references to the TX
power proximity strategy.

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

[Tim Song, 2017-05-19 20:53:10]

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

[sacomoto, 2017-05-22 07:30:30]

LGTM.

https://codereview.chromium.org/2898513002/diff/1/components/proximity_auth/p...
File components/proximity_auth/proximity_monitor_impl.cc (right):

https://codereview.chromium.org/2898513002/diff/1/components/proximity_auth/p...
components/proximity_auth/proximity_monitor_impl.cc:29: // proximity.
Please add to the comment the pair of devices this value works well (i.e. that
you test with).

[Tim Song, 2017-05-22 17:51:00]

https://codereview.chromium.org/2898513002/diff/1/components/proximity_auth/p...
File components/proximity_auth/proximity_monitor_impl.cc (right):

https://codereview.chromium.org/2898513002/diff/1/components/proximity_auth/p...
components/proximity_auth/proximity_monitor_impl.cc:29: // proximity.
On 2017/05/22 07:30:29, sacomoto wrote:
> Please add to the comment the pair of devices this value works well (i.e. that
> you test with). 

Done.

[Tim Song, 2017-05-22 17:51:02]

The CQ bit was checked by tengs@chromium.org

[commit-bot: I haz the power, 2017-05-22 17:51:36]

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 18:17:02]

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

[commit-bot: I haz the power, 2017-05-22 18:17:03]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Tim Song, 2017-05-22 19:57:13]

The CQ bit was checked by tengs@chromium.org

[Tim Song, 2017-05-22 19:57:13]

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/2898513002/#ps20001
(title: "fix test")

[commit-bot: I haz the power, 2017-05-22 19:57:43]

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 20:36:13]

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

[commit-bot: I haz the power, 2017-05-22 20:36:14]

Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Tim Song, 2017-05-22 22:26:04]

The CQ bit was checked by tengs@chromium.org

[Tim Song, 2017-05-22 22:26:04]

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/2898513002/#ps40001
(title: "fix_unittest")

[commit-bot: I haz the power, 2017-05-22 22:26:33]

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 23:41:11]

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

[commit-bot: I haz the power, 2017-05-22 23:41:14]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Tim Song, 2017-05-23 00:09:47]

The CQ bit was checked by tengs@chromium.org

[commit-bot: I haz the power, 2017-05-23 00:10:38]

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-23 00:55:25]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495498187903530,
"parent_rev": "2187893b82e8052ff073ee1bdb940f7ac94f19e6", "commit_rev":
"a252bab61a3811269fdca54ca84083856a7e2934"}

[commit-bot: I haz the power, 2017-05-23 00:55:36]

Description was changed from

==========
[EasyUnlock] Observe proximity changes and clean up TX power strategy.

With BLE, we only check the RSSI, so we can now remove all references to the TX
power proximity strategy.
==========

to

==========
[EasyUnlock] Observe proximity changes and clean up TX power strategy.

With BLE, we only check the RSSI, so we can now remove all references to the TX
power proximity strategy.

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

[commit-bot: I haz the power, 2017-05-23 00:55:37]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/a252bab61a3811269fdca54ca840...