Skip to content

Commit

Permalink
[Digital Identity]: pref name should reflect new API name
Browse files Browse the repository at this point in the history
https://bugs.webkit.org/show_bug.cgi?id=269163
rdar://122740257

Reviewed by Chris Dumez.

Renamed DigitalCredentialsEnabled to DigitalIdentityEnable.

* LayoutTests/http/wpt/credential-management/setDigitalIdentityEnable.https-expected.txt: Renamed from LayoutTests/http/wpt/credential-management/setDigitalCredentialsEnabled.https-expected.txt.
* LayoutTests/http/wpt/credential-management/setDigitalIdentityEnable.https.html: Renamed from LayoutTests/http/wpt/credential-management/setDigitalCredentialsEnabled.https.html.
* Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml:
* Source/WebCore/Modules/credentialmanagement/CredentialsContainer.idl:
* Source/WebCore/Modules/credentialmanagement/DigitalIdentity.idl:

Canonical link: https://commits.webkit.org/274470@main
  • Loading branch information
marcoscaceres committed Feb 12, 2024
1 parent 48efea4 commit d44a5a2
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>
Check that Credential Management parts of the API are not exposed by
default
Check that parts of the Digital Identity API are not exposed by
setting
</title>
</head>
<script>
Expand All @@ -20,7 +20,7 @@
);
}

window.internals.settings.setDigitalCredentialsEnabled(true);
window.internals.settings.setDigitalIdentityEnabled(true);

async function checkIFrame() {
const iframeWin = document.querySelector("iframe").contentWindow;
Expand Down
6 changes: 3 additions & 3 deletions Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2393,12 +2393,12 @@ DiagnosticLoggingEnabled:
WebCore:
default: false

DigitalCredentialsEnabled:
DigitalIdentityEnabled:
type: bool
status: unstable
category: dom
humanReadableName: "Digital Credentials API"
humanReadableDescription: "Enable the experimental Digital Credentials API"
humanReadableName: "Digital Identity API"
humanReadableDescription: "Enable the experimental Digital Identity API"
defaultValue:
WebKitLegacy:
default: false
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,5 +34,5 @@
Promise<BasicCredential> store(BasicCredential credential);
Promise<BasicCredential?> create(optional CredentialCreationOptions options);
Promise<undefined> preventSilentAccess();
[EnabledBySetting=DigitalCredentialsEnabled] Promise<DigitalIdentity> requestIdentity(DigitalCredentialRequestOptions options);
[EnabledBySetting=DigitalIdentityEnabled] Promise<DigitalIdentity> requestIdentity(DigitalCredentialRequestOptions options);
};
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
[
Exposed=Window,
SecureContext,
EnabledBySetting=DigitalCredentialsEnabled,
EnabledBySetting=DigitalIdentityEnabled,
Conditional=WEB_AUTHN,
] interface DigitalIdentity : BasicCredential {
[SameObject] readonly attribute ArrayBuffer data;
Expand Down

0 comments on commit d44a5a2

Please sign in to comment.