Move off of deprecated base::NonThreadSafe in device/

Move off of deprecated base::NonThreadSafe in device/

BUG=676387
Review-Url: https://codereview.chromium.org/2905153002
Cr-Commit-Position: refs/heads/master@{#475368}
Committed: https://chromium.googlesource.com/chromium/src/+/f03b5126e924f11cca9e5cc2243d29d861fc52b8

[gab, 2017-05-25 21:44:36]

Description was changed from

==========
Move off of deprecated base::NonThreadSafe in device/

Opted for a ThreadChecker even though a SequenceChecker is generally
preferred because device/serial and device/usb somehow feels very third-party
and potential thread-affine... let me know if this code is merely thread-unsafe
and would be better off with a SequenceChecker.

BUG=676387
==========

to

==========
Move off of deprecated base::NonThreadSafe in device/

Opted for a ThreadChecker even though a SequenceChecker is generally
preferred because device/serial and device/usb somehow feels very third-party
and potentially thread-affine... let me know if this code is merely
thread-unsafe
and would be better off with a SequenceChecker.

BUG=676387
==========

[gab, 2017-05-25 21:44:48]

Patchset #1 (id:1) has been deleted

[gab, 2017-05-25 21:44:53]

The CQ bit was checked by gab@chromium.org to run a CQ dry run

[gab, 2017-05-25 21:45:49]

gab@chromium.org changed reviewers:
+ reillyg@chromium.org

[gab, 2017-05-25 21:45:50]

Reilly PTaL, thanks!

[commit-bot: I haz the power, 2017-05-25 21:46:01]

Dry run: 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-25 21:56:06]

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

[commit-bot: I haz the power, 2017-05-25 21:56:07]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[gab, 2017-05-25 22:31:58]

The CQ bit was checked by gab@chromium.org to run a CQ dry run

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

Dry run: 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-26 00:39:26]

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

[commit-bot: I haz the power, 2017-05-26 00:39:27]

Dry run: This issue passed the CQ dry run.

[Reilly Grant (use Gerrit), 2017-05-26 00:52:38]

lgtm

[Reilly Grant (use Gerrit), 2017-05-26 00:53:45]

A sequence checker would also work here.

[gab, 2017-05-29 17:43:21]

Patchset #3 (id:60001) has been deleted

[gab, 2017-05-29 17:43:39]

Description was changed from

==========
Move off of deprecated base::NonThreadSafe in device/

Opted for a ThreadChecker even though a SequenceChecker is generally
preferred because device/serial and device/usb somehow feels very third-party
and potentially thread-affine... let me know if this code is merely
thread-unsafe
and would be better off with a SequenceChecker.

BUG=676387
==========

to

==========
Move off of deprecated base::NonThreadSafe in device/

BUG=676387
==========

[gab, 2017-05-29 17:44:38]

On 2017/05/26 00:53:45, Reilly Grant wrote:
> A sequence checker would also work here.

Thanks, flipped to sequence (thread-safety) checking.

[gab, 2017-05-29 17:44:42]

The CQ bit was checked by gab@chromium.org

[gab, 2017-05-29 17:44:43]

The patchset sent to the CQ was uploaded after l-g-t-m from reillyg@chromium.org
Link to the patchset: https://codereview.chromium.org/2905153002/#ps80001
(title: "ThreadChecker => SequenceChecker")

[commit-bot: I haz the power, 2017-05-29 17:44:58]

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-29 18:44:39]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1496079882920370,
"parent_rev": "1f329a33828547c2a8d08cf10089283ef75fa702", "commit_rev":
"f03b5126e924f11cca9e5cc2243d29d861fc52b8"}

[commit-bot: I haz the power, 2017-05-29 18:44:48]

Description was changed from

==========
Move off of deprecated base::NonThreadSafe in device/

BUG=676387
==========

to

==========
Move off of deprecated base::NonThreadSafe in device/

BUG=676387

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

[commit-bot: I haz the power, 2017-05-29 18:44:50]

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