Make MockImageCaptureClient RefCountedThreadSafe

Make MockImageCaptureClient RefCountedThreadSafe

MockImageCaptureClient has non-thread-safe ref count, and touched from
multiple threads. Though the access ordering to the ref count is well
controlled and there's no data race, the usage causes a false positive
DCHECK failure on an upcoming strict execution sequence check by:
https://codereview.chromium.org/2666423002/

This CL converts MockImageCaptureClient to RefCountedThreadSafe to
avoid the failure.

BUG=688072
Review-Url: https://codereview.chromium.org/2699073002
Cr-Commit-Position: refs/heads/master@{#451562}
Committed: https://chromium.googlesource.com/chromium/src/+/d8359a676ba9f7c6446f5aa2449466ff517efc9c

[tzik, 2017-02-17 06:32:09]

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

[commit-bot: I haz the power, 2017-02-17 06:33:03]

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

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

[commit-bot: I haz the power, 2017-02-17 08:07:19]

Dry run: This issue passed the CQ dry run.

[tzik, 2017-02-17 12:14:03]

Description was changed from

==========
Make MockImageCaptureClient RefCountedThreadSafe
==========

to

==========
Make MockImageCaptureClient RefCountedThreadSafe

MockImageCaptureClient has non-thread-safe ref count, and touched from
multiple threads. Though the access ordering to the ref count is well
controlled and there's no data race, the usage causes a false positive
DCHECK failure on an upcoming strict execution sequence check by:
https://codereview.chromium.org/2666423002/

This CL converts MockImageCaptureClient to RefCountedThreadSafe to
avoid the failure.

BUG=688072
==========

[tzik, 2017-02-17 12:15:38]

tzik@chromium.org changed reviewers:
+ mcasas@chromium.org

[tzik, 2017-02-17 12:15:39]

PTAL

[mcasas, 2017-02-17 16:36:25]

lgtm

[tzik, 2017-02-20 04:00:58]

The CQ bit was checked by tzik@chromium.org

[tzik, 2017-02-20 04:00:59]

The patchset sent to the CQ was uploaded after l-g-t-m from mcasas@chromium.org
Link to the patchset: https://codereview.chromium.org/2699073002/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2017-02-20 04:01:11]

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-02-20 05:32:28]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487563258730050,
"parent_rev": "786184e85335e590fd0dd3b31d267f524633a90b", "commit_rev":
"d8359a676ba9f7c6446f5aa2449466ff517efc9c"}

[commit-bot: I haz the power, 2017-02-20 05:33:27]

Description was changed from

==========
Make MockImageCaptureClient RefCountedThreadSafe

MockImageCaptureClient has non-thread-safe ref count, and touched from
multiple threads. Though the access ordering to the ref count is well
controlled and there's no data race, the usage causes a false positive
DCHECK failure on an upcoming strict execution sequence check by:
https://codereview.chromium.org/2666423002/

This CL converts MockImageCaptureClient to RefCountedThreadSafe to
avoid the failure.

BUG=688072
==========

to

==========
Make MockImageCaptureClient RefCountedThreadSafe

MockImageCaptureClient has non-thread-safe ref count, and touched from
multiple threads. Though the access ordering to the ref count is well
controlled and there's no data race, the usage causes a false positive
DCHECK failure on an upcoming strict execution sequence check by:
https://codereview.chromium.org/2666423002/

This CL converts MockImageCaptureClient to RefCountedThreadSafe to
avoid the failure.

BUG=688072

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

[commit-bot: I haz the power, 2017-02-20 05:33:28]

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