CREDENTIAL: Update PasswordCredential's constructor.

CREDENTIAL: Update PasswordCredential's constructor.

Rather than a list of parameters, PasswordCredential is defined[1] to
take a PasswordCredentialData object. We defined that object long ago
(in [2]), but apparently never actually used it. This patch fixes that
oversight.

As a drive-by, this patch also stops exposing the PasswordCredential
and FederatedCredential to Workers, as per spec.

[1]: https://w3c.github.io/webappsec/specs/credentialmanagement/#interfaces-credential-types-passwordcredential
[2]: https://codereview.chromium.org/1162883002

BUG=494880
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201723

[Mike West, 2015-09-03 10:44:25]

mkwst@chromium.org changed reviewers:
+ philipj@opera.com, vabr@chromium.org

[Mike West, 2015-09-03 10:44:26]

Vaclav, Philip, would one of you mind taking a look at this small CL? We forgot
to change the spelling of the PasswordCredential object from a list of
parameters to a dictionary.

As a drive-by, this patch also stops exposing the PasswordCredential and
FederatedCredential to Workers, as per spec.

-mike

[vabr (Chromium), 2015-09-03 10:50:20]

On 2015/09/03 10:44:26, Mike West (buried) wrote:
> Vaclav, Philip, would one of you mind taking a look at this small CL? We
forgot
> to change the spelling of the PasswordCredential object from a list of
> parameters to a dictionary.
> 
> As a drive-by, this patch also stops exposing the PasswordCredential and
> FederatedCredential to Workers, as per spec.
> 
> -mike

LGTM!

The Expose=Window is what causes the exposition to Workes to stop?

Cheers,
Vaclav

[Mike West, 2015-09-03 11:12:23]

On 2015/09/03 at 10:50:20, vabr wrote:
> On 2015/09/03 10:44:26, Mike West (buried) wrote:
> > Vaclav, Philip, would one of you mind taking a look at this small CL? We
forgot
> > to change the spelling of the PasswordCredential object from a list of
> > parameters to a dictionary.
> > 
> > As a drive-by, this patch also stops exposing the PasswordCredential and
> > FederatedCredential to Workers, as per spec.
> > 
> > -mike
> 
> LGTM!
> 
> The Expose=Window is what causes the exposition to Workes to stop?

Correct. Thanks!

[Mike West, 2015-09-03 11:12:27]

The CQ bit was checked by mkwst@chromium.org

[commit-bot: I haz the power, 2015-09-03 11:12:34]

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

[Mike West, 2015-09-03 11:23:34]

The CQ bit was checked by mkwst@chromium.org

[Mike West, 2015-09-03 11:23:34]

The patchset sent to the CQ was uploaded after l-g-t-m from vabr@chromium.org
Link to the patchset: https://codereview.chromium.org/1314673007/#ps20001
(title: "Ugh. Missed one.")

[commit-bot: I haz the power, 2015-09-03 11:23:47]

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

[commit-bot: I haz the power, 2015-09-03 11:26:22]

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

[commit-bot: I haz the power, 2015-09-03 11:26:23]

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

[Mike West, 2015-09-03 11:45:47]

The CQ bit was checked by mkwst@chromium.org

[Mike West, 2015-09-03 11:45:47]

The patchset sent to the CQ was uploaded after l-g-t-m from vabr@chromium.org
Link to the patchset: https://codereview.chromium.org/1314673007/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2015-09-03 11:45:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1314673007/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1314673007/40001

[commit-bot: I haz the power, 2015-09-03 12:54:39]

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

[commit-bot: I haz the power, 2015-09-03 12:54:40]

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

[Mike West, 2015-09-03 13:01:56]

The CQ bit was checked by mkwst@chromium.org

[commit-bot: I haz the power, 2015-09-03 13:02:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1314673007/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1314673007/40001

[commit-bot: I haz the power, 2015-09-03 14:08:46]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201723