Credential Manager: Reject credential request if an outstanding request is pending.

Credential Manager: Reject credential request if an outstanding request is pending.

If we're waiting on the PasswordStore (or the user, later) to give us
credential data, don't pile on new requests.

BUG=400674
Committed: https://crrev.com/4abf38fc383ad4a653233f6f6fb3ebd37b428c1c
Cr-Commit-Position: refs/heads/master@{#297595}

[Mike West, 2014-09-30 09:39:19]

mkwst@chromium.org changed reviewers:
+ vabr@chromium.org

[Mike West, 2014-09-30 09:39:20]

Moar. WDYT?

If you're happy, I'll find someone to stamp the IPC.

-mike

[vabr (Chromium), 2014-09-30 10:30:42]

LGTM.

The removed OVERRIDEs, that something Blink-specific, right?

Cheers,
Vaclav

[Mike West, 2014-09-30 10:34:00]

Yes. We shouldn't OVERRIDE functions defined in the Blink repository, as it
makes changes more or less impossible. Once we merge repos, we'll probably relax
that rule.

[Mike West, 2014-09-30 12:33:39]

mkwst@chromium.org changed reviewers:
+ palmer@chromium.org

[Mike West, 2014-09-30 12:33:39]

Chris, would you mind taking a quick look at this IPC change?

[palmer, 2014-09-30 21:14:59]

lgtm

[Mike West, 2014-10-01 04:18:16]

The CQ bit was checked by mkwst@chromium.org

[commit-bot: I haz the power, 2014-10-01 04:19:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/613993002/20001

[commit-bot: I haz the power, 2014-10-01 04:23:07]

Committed patchset #2 (id:20001) as 9870c1466e870cb9495a920f370901024068680d

[commit-bot: I haz the power, 2014-10-01 04:24:11]

Patchset 2 (id:??) landed as
https://crrev.com/4abf38fc383ad4a653233f6f6fb3ebd37b428c1c
Cr-Commit-Position: refs/heads/master@{#297595}