Remove //components/proximity_auth/remote_device.h. It was supposed to have been removed in https:/…

Remove //components/proximity_auth/remote_device.h. It was supposed to have been removed in https://codereview.chromium.org/2560713002.

BUG=672263
TBR=tengs@chromium.org

Committed: https://crrev.com/9da90eb2fe57c3f047f4f4461ab6c68884233211
Cr-Commit-Position: refs/heads/master@{#439515}

[Kyle Horimoto, 2016-12-19 17:42:53]

khorimoto@chromium.org changed reviewers:
+ ben@chromium.org

[Kyle Horimoto, 2016-12-19 17:56:36]

khorimoto@chromium.org changed reviewers:
+ msarda@chromium.org
- ben@chromium.org

[Kyle Horimoto, 2016-12-19 17:58:49]

Description was changed from

==========
Remove //components/proximity_auth/remote_device.h. It was supposed to have been
removed in https://codereview.chromium.org/2560713002.

BUG=672263
==========

to

==========
Remove //components/proximity_auth/remote_device.h. It was supposed to have been
removed in https://codereview.chromium.org/2560713002.

BUG=672263

TBR=tengs@chromium.org
==========

[Kyle Horimoto, 2016-12-19 17:58:49]

khorimoto@chromium.org changed reviewers:
+ hansberry@chromium.org
- msarda@chromium.org

[msarda, 2016-12-19 18:12:27]

msarda@chromium.org changed reviewers:
+ msarda@chromium.org

[msarda, 2016-12-19 18:12:27]

LGTM as discussed offline. Please answer my questions before landing this CL.

https://codereview.chromium.org/2586983003/diff/1/components/cryptauth/DEPS
File components/cryptauth/DEPS (right):

https://codereview.chromium.org/2586983003/diff/1/components/cryptauth/DEPS#n...
components/cryptauth/DEPS:4: "+components/proximity_auth/logging",
Why this change? This CL does not include anything from
components/proximity_auth/logging.

https://codereview.chromium.org/2586983003/diff/1/components/proximity_auth/r...
File components/proximity_auth/remote_device.h (left):

https://codereview.chromium.org/2586983003/diff/1/components/proximity_auth/r...
components/proximity_auth/remote_device.h:13: struct RemoteDevice {
Was this file moved? I do not see the added on in this CL.

[Kyle Horimoto, 2016-12-19 18:15:02]

https://codereview.chromium.org/2586983003/diff/1/components/cryptauth/DEPS
File components/cryptauth/DEPS (right):

https://codereview.chromium.org/2586983003/diff/1/components/cryptauth/DEPS#n...
components/cryptauth/DEPS:4: "+components/proximity_auth/logging",
On 2016/12/19 18:12:27, msarda wrote:
> Why this change? This CL does not include anything from
> components/proximity_auth/logging.

Other files in this directory depend on //components/proximity_auth/logging
target. This CL removes the only dependency left on //components/proximity_auth.

https://codereview.chromium.org/2586983003/diff/1/components/proximity_auth/r...
File components/proximity_auth/remote_device.h (left):

https://codereview.chromium.org/2586983003/diff/1/components/proximity_auth/r...
components/proximity_auth/remote_device.h:13: struct RemoteDevice {
On 2016/12/19 18:12:27, msarda wrote:
> Was this file moved? I do not see the added on in this CL.

It was moved in the original CL which is linked to in this CL's description.

[Kyle Horimoto, 2016-12-19 18:15:11]

The CQ bit was checked by khorimoto@chromium.org

[commit-bot: I haz the power, 2016-12-19 18:15:40]

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, 2016-12-19 19:00:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482171311286270, "parent_rev":
"41badb6a30039930ea8fb8163bb421a7e5a9a8c7", "commit_rev":
"56bedb351a7b230217bfeda6bb07d3d81af60387"}

[commit-bot: I haz the power, 2016-12-19 19:00:34]

Description was changed from

==========
Remove //components/proximity_auth/remote_device.h. It was supposed to have been
removed in https://codereview.chromium.org/2560713002.

BUG=672263

TBR=tengs@chromium.org
==========

to

==========
Remove //components/proximity_auth/remote_device.h. It was supposed to have been
removed in https://codereview.chromium.org/2560713002.

BUG=672263

TBR=tengs@chromium.org

Review-Url: https://codereview.chromium.org/2586983003
==========

[commit-bot: I haz the power, 2016-12-19 19:00:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-19 19:03:09]

Description was changed from

==========
Remove //components/proximity_auth/remote_device.h. It was supposed to have been
removed in https://codereview.chromium.org/2560713002.

BUG=672263

TBR=tengs@chromium.org

Review-Url: https://codereview.chromium.org/2586983003
==========

to

==========
Remove //components/proximity_auth/remote_device.h. It was supposed to have been
removed in https://codereview.chromium.org/2560713002.

BUG=672263

TBR=tengs@chromium.org

Committed: https://crrev.com/9da90eb2fe57c3f047f4f4461ab6c68884233211
Cr-Commit-Position: refs/heads/master@{#439515}
==========

[commit-bot: I haz the power, 2016-12-19 19:03:10]

Patchset 1 (id:??) landed as
https://crrev.com/9da90eb2fe57c3f047f4f4461ab6c68884233211
Cr-Commit-Position: refs/heads/master@{#439515}

[Ben Goodger (Google), 2016-12-19 20:08:35]

ben@chromium.org changed reviewers:
+ ben@chromium.org

[Ben Goodger (Google), 2016-12-19 20:08:35]

lgtm