[CrOS Tether] TetherHostFetcher: When invoking callbacks, use a copy of the device list.

[CrOS Tether] TetherHostFetcher: When invoking callbacks, use a copy of the device list.

Previously, it used a reference which had already been deleted. The single callback case correctly used the copy, but the list callback did not.

BUG=672263
Review-Url: https://codereview.chromium.org/2777343006
Cr-Commit-Position: refs/heads/master@{#460445}
Committed: https://chromium.googlesource.com/chromium/src/+/aea32ab18a36aa5bb829b3095e6ddd776bf62e67

[Kyle Horimoto, 2017-03-29 00:21:11]

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

[Ryan Hansberry, 2017-03-29 17:02:23]

The CQ bit was checked by hansberry@chromium.org

[Ryan Hansberry, 2017-03-29 17:02:23]

lgtm

[commit-bot: I haz the power, 2017-03-29 17:03: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-03-29 17:03:38]

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

[commit-bot: I haz the power, 2017-03-29 17:03:39]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[Jeremy Klein, 2017-03-29 17:07:22]

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

[Jeremy Klein, 2017-03-29 17:07:22]

lgtm

[Kyle Horimoto, 2017-03-29 17:12:51]

The CQ bit was checked by khorimoto@chromium.org

[commit-bot: I haz the power, 2017-03-29 17:13:18]

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-03-29 17:45:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490807571146460, "parent_rev":
"b17ee3d6890cf542ff39366a5f6579d540314829", "commit_rev":
"aea32ab18a36aa5bb829b3095e6ddd776bf62e67"}

[commit-bot: I haz the power, 2017-03-29 17:46:19]

Description was changed from

==========
[CrOS Tether] TetherHostFetcher: When invoking callbacks, use a copy of the
device list.

Previously, it used a reference which had already been deleted. The single
callback case correctly used the copy, but the list callback did not.

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

to

==========
[CrOS Tether] TetherHostFetcher: When invoking callbacks, use a copy of the
device list.

Previously, it used a reference which had already been deleted. The single
callback case correctly used the copy, but the list callback did not.

BUG=672263

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

[commit-bot: I haz the power, 2017-03-29 17:46:20]

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