LayoutTests/http/tests/credentialmanager/credentialscontainer-notifysignedin-basics.html - Issue 661973002: Credential Manager: JavaScript arguments make more sense in a different order.

Side by Side Diff: LayoutTests/http/tests/credentialmanager/credentialscontainer-notifysignedin-basics.html

Issue 661973002: Credential Manager: JavaScript arguments make more sense in a different order. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git/+/master

Patch Set: Created 6 years, 2 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« no previous file with comments | « LayoutTests/http/tests/credentialmanager/credentialscontainer-notifyfailedsignin-basics.html ('k') | LayoutTests/http/tests/credentialmanager/federatedcredential-basics.html » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 <!DOCTYPE html>	1 <!DOCTYPE html>

2 <title>Credential Manager: notifySignedIn() basics.</title>	2 <title>Credential Manager: notifySignedIn() basics.</title>

3 <script src="../resources/testharness.js"></script>	3 <script src="../resources/testharness.js"></script>

4 <script src="../resources/testharnessreport.js"></script>	4 <script src="../resources/testharnessreport.js"></script>

5 <script>	5 <script>

6 function stubResolverChecker(c) {	6 function stubResolverChecker(c) {

7 assert_equals(c, undefined, "FIXME: We're currently always returning 'undefi ned'.");	7 assert_equals(c, undefined, "FIXME: We're currently always returning 'undefi ned'.");

8 this.done();	8 this.done();

9 }	9 }

10	10

11 function stubRejectionChecker(reason) {	11 function stubRejectionChecker(reason) {

12 assert_unreached("notifySignedIn() should not reject, but did: " + reason.na me);	12 assert_unreached("notifySignedIn() should not reject, but did: " + reason.na me);

13 }	13 }

14	14

15 var local = new LocalCredential('id', 'name', 'https://example.com/avatar.png', 'pencil');	15 var local = new LocalCredential('id', 'pencil', 'name', 'https://example.com/ava tar.png');

16 var federated = new FederatedCredential('id', 'name', 'https://example.com/avata r.png', 'https://federation.net');	16 var federated = new FederatedCredential('id', 'https://federation.net', 'name', 'https://example.com/avatar.png');

17	17

18 async_test(function () {	18 async_test(function () {

19 navigator.credentials.notifySignedIn().then(	19 navigator.credentials.notifySignedIn().then(

20 this.step_func(function () { assert_unreached("notifySignedIn() should r eject."); }),	20 this.step_func(function () { assert_unreached("notifySignedIn() should r eject."); }),

21 this.step_func(function (reason) {	21 this.step_func(function (reason) {

22 assert_equals(reason.name, "TypeError");	22 assert_equals(reason.name, "TypeError");

23 this.done();	23 this.done();

24 }));	24 }));

25 }, "Verify the basics of notifyFailedSignIn(): LocalCredential.");	25 }, "Verify the basics of notifyFailedSignIn(): LocalCredential.");

26	26

(...skipping 11 matching lines...) Expand all Loading...
38 this.step_func(stubResolverChecker.bind(this)),	38 this.step_func(stubResolverChecker.bind(this)),

39 this.step_func(stubRejectionChecker.bind(this)));	39 this.step_func(stubRejectionChecker.bind(this)));

40 }, "Verify the basics of notifySignedIn(): LocalCredential.");	40 }, "Verify the basics of notifySignedIn(): LocalCredential.");

41	41

42 async_test(function () {	42 async_test(function () {

43 navigator.credentials.notifySignedIn(federated).then(	43 navigator.credentials.notifySignedIn(federated).then(

44 this.step_func(stubResolverChecker.bind(this)),	44 this.step_func(stubResolverChecker.bind(this)),

45 this.step_func(stubRejectionChecker.bind(this)));	45 this.step_func(stubRejectionChecker.bind(this)));

46 }, "Verify the basics of notifySignedIn(): FederatedCredential.");	46 }, "Verify the basics of notifySignedIn(): FederatedCredential.");

47 </script>	47 </script>

OLD	NEW