Update TaskTraits in client_cert_store_chromeos.cc.

Update TaskTraits in client_cert_store_chromeos.cc.

The .WithFileIO() and .WithWait() traits were deprecated in favor of
.MayBlock() and .WithSyncPrimitives(). See rationale in
https://docs.google.com/a/chromium.org/document/d/1ynRvQKah3Cx_eLqPS7KzIp8u73HeKIjX7HQqwSWOnF8/edit?usp=sharing

.MayBlock()
Tasks with this trait may block. This includes but is not limited to tasks
that wait on synchronous file I/O operations: read or write a file from
disk, interact with a pipe or a socket, rename or delete a file, enumerate
files in a directory, etc. This trait does not allow tasks to wait on
synchronization primitives (thread or process handles, waitable events,
condition variables).

R=gab@chromium.org
BUG=675660
Committed: https://crrev.com/3ebe0a19ea0edd9ee8fbd930b98a21032c8ac35f
Cr-Commit-Position: refs/heads/master@{#440167}

[fdoray, 2016-12-21 15:46:33]

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

[commit-bot: I haz the power, 2016-12-21 15:46:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[fdoray, 2016-12-21 16:05:30]

fdoray@chromium.org changed reviewers:
+ mattm@chromium.org, rsleevi@chromium.org

[fdoray, 2016-12-21 16:05:31]

PTAL

rsleevi@: mattm@ reviewed this related CL
https://codereview.chromium.org/2533473002/ but is now OOO

[gab, 2016-12-21 16:07:58]

lgtm (don't need to define .WithSyncPrimitives() in this CL's desc per it not
being used?)

[commit-bot: I haz the power, 2016-12-21 16:14:45]

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

[commit-bot: I haz the power, 2016-12-21 16:14:46]

Dry run: This issue passed the CQ dry run.

[fdoray, 2016-12-21 16:29:31]

Description was changed from

==========
Update TaskTraits in client_cert_store_chromeos.cc.

The .WithFileIO() and .WithWait() traits were deprecated in favor of
.MayBlock() and .WithSyncPrimitives(). See rationale in
https://docs.google.com/a/chromium.org/document/d/1ynRvQKah3Cx_eLqPS7KzIp8u73...

.MayBlock()
Tasks with this trait may block. This includes but is not limited to tasks
that wait on synchronous file I/O operations: read or write a file from
disk, interact with a pipe or a socket, rename or delete a file, enumerate
files in a directory, etc. This trait does not allow tasks to wait on
synchronization primitives (thread or process handles, waitable events,
condition variables).

.WithSyncPrimitives()
Tasks with this trait are allowed to block on synchronization primitives
(wait on a waitable event/condition variable, join a process/thread).
This trait implies MayBlock(), i.e. you do not need to specify MayBlock()
for a task that blocks exclusively on synchronization primitives.

R=gab@chromium.org
BUG=675660
==========

to

==========
Update TaskTraits in client_cert_store_chromeos.cc.

The .WithFileIO() and .WithWait() traits were deprecated in favor of
.MayBlock() and .WithSyncPrimitives(). See rationale in
https://docs.google.com/a/chromium.org/document/d/1ynRvQKah3Cx_eLqPS7KzIp8u73...

.MayBlock()
Tasks with this trait may block. This includes but is not limited to tasks
that wait on synchronous file I/O operations: read or write a file from
disk, interact with a pipe or a socket, rename or delete a file, enumerate
files in a directory, etc. This trait does not allow tasks to wait on
synchronization primitives (thread or process handles, waitable events,
condition variables).

R=gab@chromium.org
BUG=675660
==========

[fdoray, 2016-12-21 16:29:42]

On 2016/12/21 16:07:58, gab wrote:
> lgtm (don't need to define .WithSyncPrimitives() in this CL's desc per it not
> being used?)

Done

rsleevi@: PTAL

[Ryan Sleevi, 2016-12-21 18:34:50]

lgtm

[fdoray, 2016-12-21 18:43:24]

The CQ bit was checked by fdoray@chromium.org

[commit-bot: I haz the power, 2016-12-21 18:43:56]

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-21 18:49:09]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482345804467250, "parent_rev":
"dda90ed802c98104a48d56155fa6c36fcc40326f", "commit_rev":
"5fedc5406eca4b04b80c74a2a14104fdf1373e1d"}

[commit-bot: I haz the power, 2016-12-21 18:50:00]

Description was changed from

==========
Update TaskTraits in client_cert_store_chromeos.cc.

The .WithFileIO() and .WithWait() traits were deprecated in favor of
.MayBlock() and .WithSyncPrimitives(). See rationale in
https://docs.google.com/a/chromium.org/document/d/1ynRvQKah3Cx_eLqPS7KzIp8u73...

.MayBlock()
Tasks with this trait may block. This includes but is not limited to tasks
that wait on synchronous file I/O operations: read or write a file from
disk, interact with a pipe or a socket, rename or delete a file, enumerate
files in a directory, etc. This trait does not allow tasks to wait on
synchronization primitives (thread or process handles, waitable events,
condition variables).

R=gab@chromium.org
BUG=675660
==========

to

==========
Update TaskTraits in client_cert_store_chromeos.cc.

The .WithFileIO() and .WithWait() traits were deprecated in favor of
.MayBlock() and .WithSyncPrimitives(). See rationale in
https://docs.google.com/a/chromium.org/document/d/1ynRvQKah3Cx_eLqPS7KzIp8u73...

.MayBlock()
Tasks with this trait may block. This includes but is not limited to tasks
that wait on synchronous file I/O operations: read or write a file from
disk, interact with a pipe or a socket, rename or delete a file, enumerate
files in a directory, etc. This trait does not allow tasks to wait on
synchronization primitives (thread or process handles, waitable events,
condition variables).

R=gab@chromium.org
BUG=675660

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

[commit-bot: I haz the power, 2016-12-21 18:50:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-21 18:52:32]

Description was changed from

==========
Update TaskTraits in client_cert_store_chromeos.cc.

The .WithFileIO() and .WithWait() traits were deprecated in favor of
.MayBlock() and .WithSyncPrimitives(). See rationale in
https://docs.google.com/a/chromium.org/document/d/1ynRvQKah3Cx_eLqPS7KzIp8u73...

.MayBlock()
Tasks with this trait may block. This includes but is not limited to tasks
that wait on synchronous file I/O operations: read or write a file from
disk, interact with a pipe or a socket, rename or delete a file, enumerate
files in a directory, etc. This trait does not allow tasks to wait on
synchronization primitives (thread or process handles, waitable events,
condition variables).

R=gab@chromium.org
BUG=675660

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

to

==========
Update TaskTraits in client_cert_store_chromeos.cc.

The .WithFileIO() and .WithWait() traits were deprecated in favor of
.MayBlock() and .WithSyncPrimitives(). See rationale in
https://docs.google.com/a/chromium.org/document/d/1ynRvQKah3Cx_eLqPS7KzIp8u73...

.MayBlock()
Tasks with this trait may block. This includes but is not limited to tasks
that wait on synchronous file I/O operations: read or write a file from
disk, interact with a pipe or a socket, rename or delete a file, enumerate
files in a directory, etc. This trait does not allow tasks to wait on
synchronization primitives (thread or process handles, waitable events,
condition variables).

R=gab@chromium.org
BUG=675660

Committed: https://crrev.com/3ebe0a19ea0edd9ee8fbd930b98a21032c8ac35f
Cr-Commit-Position: refs/heads/master@{#440167}
==========

[commit-bot: I haz the power, 2016-12-21 18:52:33]

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