[Merge to M43 branch] Postpone check if password form is loaded from store

In some cases we encounter situation when in processing onSubmit of password form we didn't receive information from password store yet. For example this can happen when JavaScript creates password form after submission and submit it (as for nytimes.com).

To process such situation this CL postpones checking if password form fetched from store until moment when we are going to decide if we should save it.

In order to process this correctly creating pending credentials in PasswordManager was moved to separate method and it is called only when both events happen - form submission and fetching from store finished.

BUG=470322
Review URL: https://codereview.chromium.org/1050903002

Cr-Commit-Position: refs/heads/master@{#324400}
(cherry picked from commit f46ff48284131eba449796a932ec32290b03a403)

R=vabr@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/037c0924bcdc0372df29ee77891f6653880b6ed4

[dvadym, 2015-04-13 10:55:06]

Hi Vaclav,

 Could you please review this merge issue?

Best regards,
Vadym

[dvadym, 2015-04-13 10:57:56]

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

[commit-bot: I haz the power, 2015-04-13 10:58:01]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1086563002/1

[commit-bot: I haz the power, 2015-04-13 10:58:02]

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

[commit-bot: I haz the power, 2015-04-13 10:58:03]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[dvadym, 2015-04-13 11:41:02]

The CQ bit was checked by dvadym@chromium.org

[commit-bot: I haz the power, 2015-04-13 11:41:23]

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

[commit-bot: I haz the power, 2015-04-13 11:41:25]

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

[commit-bot: I haz the power, 2015-04-13 11:41:26]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[vabr (Chromium), 2015-04-13 11:46:27]

This merge of https://codereview.chromium.org/1050903002 LGTM.

Thanks!
Vaclav

[dvadym, 2015-04-13 11:53:03]

The CQ bit was checked by dvadym@chromium.org

[commit-bot: I haz the power, 2015-04-13 11:53:08]

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

[commit-bot: I haz the power, 2015-04-13 11:53:10]

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

[commit-bot: I haz the power, 2015-04-13 11:53:11]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[vabr (Chromium), 2015-04-13 13:34:18]

Committed patchset #1 (id:1) manually as
037c0924bcdc0372df29ee77891f6653880b6ed4 (presubmit successful).

[vabr (Chromium), 2015-04-13 13:35:55]

On 2015/04/13 13:34:18, vabr (Chromium) wrote:
> Committed patchset #1 (id:1) manually as
> 037c0924bcdc0372df29ee77891f6653880b6ed4 (presubmit successful).

I landed this CL on dvadym's behalf. He is already a committer, but is not on
some manually updated list yet, and therefore could not "git cl land" this.

Cheers,
Vaclav