Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spec: Add WebDriver extension #64

Merged
merged 3 commits into from
Jul 10, 2023
Merged

Spec: Add WebDriver extension #64

merged 3 commits into from
Jul 10, 2023

Conversation

alexmturner
Copy link
Collaborator

@alexmturner alexmturner commented Jun 14, 2023

This will allow for simpler testing, by avoiding the significant reporting delays. See also the related
WICG/attribution-reporting-api#843


Preview | Diff

@alexmturner
Copy link
Collaborator Author

@linnan-github, could you PTAL? Thanks!
cc @yoavweiss

spec.bs Outdated
@@ -352,6 +356,9 @@ A user agent holds a <dfn>contribution cache</dfn>, which is a [=list=] of
A user agent holds a <dfn>debug scope map</dfn>, which is an [=ordered map=]
from [=debug scopes=] to [=debug details=].

A user agent holds a <dfn>developer mode enabled</dfn>, which is a [=boolean=],

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shall we align with ARA and use "The user-agent holds an associated boolean automation local testing mode (default false)."

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed associated to align with the other storage language, but moved it to the user-agent automation section to align with ARA

spec.bs Outdated Show resolved Hide resolved
@alexmturner alexmturner added the spec Related to specification/standardization label Jun 28, 2023
@alexmturner
Copy link
Collaborator Author

@mathiasbynens, could you PTAL? Rick Byers suggested you as a reviewer. Thanks!

Copy link

@mathiasbynens mathiasbynens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

spec.bs Outdated

The [=remote end steps=] are:

1. If |parameters| is not a JSON [[ECMASCRIPT#sec-json-object|Object]], return a

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The section of the ECMAScript spec this points to is about the JSON object, i.e. globalThis.JSON, but that’s not what we’re interested in here. To fix this, link to https://tc39.es/ecma262/#sec-objects instead of #sec-json-object. (We could also change the phrasing to “is not an Object” or “is not a JSON-formatted object” to be more precise but that’s not a blocker.)

Relevant precedent in the WebDriver classic spec is e.g. here: https://w3c.github.io/webdriver/#add-cookie

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah good point! Fixed the link and updated the phrasing

This will allow for simpler testing, by avoiding the significant
reporting delays. See also the related
WICG/attribution-reporting-api#843
@alexmturner
Copy link
Collaborator Author

Thanks for the review!

@alexmturner alexmturner merged commit 3717d8a into main Jul 10, 2023
1 check passed
@alexmturner alexmturner deleted the web-driver branch July 10, 2023 17:45
github-actions bot added a commit that referenced this pull request Jul 10, 2023
SHA: 3717d8a
Reason: push, by alexmturner

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
alexmturner added a commit to WICG/attribution-reporting-api that referenced this pull request Jul 10, 2023
Fixes a link and clarifies the prose slightly. See patcg-individual-drafts/private-aggregation-api#64 (comment) for explanation.
alexmturner added a commit to WICG/attribution-reporting-api that referenced this pull request Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec Related to specification/standardization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants