Make crypto::GetSystemNSSKeySlot asynchronous.

Make crypto::GetSystemNSSKeySlot asynchronous.

The system slot is set asynchronously, so the getting the system slot should happen asynchronously as well.

BUG=210525
[email protected]

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=286493

[mattm, 2014-07-29 23:16:46]

lgtm with nit

https://codereview.chromium.org/426983002/diff/20001/crypto/nss_util_internal.h
File crypto/nss_util_internal.h (right):

https://codereview.chromium.org/426983002/diff/20001/crypto/nss_util_internal...
crypto/nss_util_internal.h:51: // laoded and |callback| is non-null, the
|callback| will be run once the slot
laoded

[Ryan Sleevi, 2014-07-29 23:41:48]

On 2014/07/29 23:16:46, mattm wrote:
> lgtm with nit
> 
>
https://codereview.chromium.org/426983002/diff/20001/crypto/nss_util_internal.h
> File crypto/nss_util_internal.h (right):
> 
>
https://codereview.chromium.org/426983002/diff/20001/crypto/nss_util_internal...
> crypto/nss_util_internal.h:51: // laoded and |callback| is non-null, the
> |callback| will be run once the slot
> laoded

Defer to Matt :)

[pneubeck (no reviews), 2014-07-30 08:13:36]

The CQ bit was checked by [email protected]

[commit-bot: I haz the power, 2014-07-30 08:15:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/[email protected]/426983002/40001

[commit-bot: I haz the power, 2014-07-30 11:39:47]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[commit-bot: I haz the power, 2014-07-30 11:43:06]

The CQ bit was unchecked by [email protected]

[commit-bot: I haz the power, 2014-07-30 11:43:07]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[pneubeck (no reviews), 2014-07-30 12:19:56]

The CQ bit was checked by [email protected]

[commit-bot: I haz the power, 2014-07-30 12:20:58]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/[email protected]/426983002/40001

[commit-bot: I haz the power, 2014-07-30 12:24:33]

Change committed as 286493

[Ryan Sleevi, 2014-07-31 00:34:39]

retroactive LGTM