Bind ContentVerifer::VerifyFailed as OnceCallback.

Bind ContentVerifer::VerifyFailed as OnceCallback.

The consumer ContentVerifyJob, calls this only once and ContentVerifier
is ref counted, so this is a good thing to do.

BUG=712945
Test=No visible changes expected.

Review-Url: https://codereview.chromium.org/2824333003
Cr-Commit-Position: refs/heads/master@{#465731}
Committed: https://chromium.googlesource.com/chromium/src/+/e289daa9258b8646c54ae1dfcbda3f8843ba4012

[lazyboy, 2017-04-19 02:45:18]

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

[commit-bot: I haz the power, 2017-04-19 02:45:51]

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-04-19 03:25:27]

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

[commit-bot: I haz the power, 2017-04-19 03:25:28]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[lazyboy, 2017-04-19 17:33:19]

Description was changed from

==========
Make 2 bound methods in ref counted ContentVerifier to use OnceCallback.

ContentVerifyJob::FailureCallback and
ContentHashFetcher::FetchCallback are bound to ref counted class
ContentVerifier's methods.

Also make ContentHashFetcherJob::CompletionCallback a OnceCallback,
although it doesn't contribute to any ref counting.

BUG=712945
Test=No visible changes expected.
==========

to

==========
Bind ContentVerifer::VerifyFailed as OnceCallback.

The consumer ContentVerifyJob, calls this only once and ContentVerifier
is ref counted, so this is a good thing to do.

BUG=712945
Test=No visible changes expected.
==========

[lazyboy, 2017-04-19 17:33:35]

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

[lazyboy, 2017-04-19 17:33:47]

lazyboy@chromium.org changed reviewers:
+ rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2017-04-19 17:34:17]

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-04-19 18:22:53]

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

[commit-bot: I haz the power, 2017-04-19 18:22:53]

Dry run: This issue passed the CQ dry run.

[Devlin, 2017-04-19 20:19:33]

lgtm

[lazyboy, 2017-04-19 20:25:04]

The CQ bit was checked by lazyboy@chromium.org

[commit-bot: I haz the power, 2017-04-19 20:25: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, 2017-04-19 20:29:57]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1492633504860430,
"parent_rev": "c161437abe4b327a90de3c32520dbd85bf7ce89e", "commit_rev":
"e289daa9258b8646c54ae1dfcbda3f8843ba4012"}

[commit-bot: I haz the power, 2017-04-19 20:30:04]

Description was changed from

==========
Bind ContentVerifer::VerifyFailed as OnceCallback.

The consumer ContentVerifyJob, calls this only once and ContentVerifier
is ref counted, so this is a good thing to do.

BUG=712945
Test=No visible changes expected.
==========

to

==========
Bind ContentVerifer::VerifyFailed as OnceCallback.

The consumer ContentVerifyJob, calls this only once and ContentVerifier
is ref counted, so this is a good thing to do.

BUG=712945
Test=No visible changes expected.

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

[commit-bot: I haz the power, 2017-04-19 20:30:16]

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