Skip to content

[Technical review] BiDi script module - disown, addPreloadScript, removePreloadScript - #46094

Open
dipikabh wants to merge 4 commits into
mdn:mainfrom
dipikabh:bidi-script-4
Open

dipikabh wants to merge 4 commits into
mdn:mainfrom
dipikabh:bidi-script-4

Conversation

@dipikabh

Copy link
Copy Markdown
Contributor

Description

  • Added pages for the following commands of the script module (with corresponding updates to the sidebar and script module landing page):

    • script.disown
    • script.addPreloadScript
    • script.removePreloadScript
  • Added sections to the script module landing page to explain "preload scripts" and "channels" in the context of BiDi.

  • Some minor consistency fixes in browsingContext.create and session.subscribe to move the ID lookup sentence to end of the field description

Spec links

Related issue

Doc issue: mdn/mdn#851

@dipikabh
dipikabh requested a review from a team October 10, 2026 14:16
@dipikabh
dipikabh requested review from a team as code owners October 10, 2026 14:16
@dipikabh
dipikabh requested review from chrisdavidmills and pepelsbey and removed request for a team October 10, 2026 14:16
@github-actions github-actions Bot added the Content:WebDriver WebDriver docs label Oct 10, 2026
@dipikabh
dipikabh requested review from a team and pepelsbey and removed request for a team, chrisdavidmills and pepelsbey October 10, 2026 14:16
@github-actions github-actions Bot added the size/l [PR only] 501-1000 LoC changed label Oct 10, 2026
@dipikabh
dipikabh removed the request for review from pepelsbey October 10, 2026 14:16
@github-actions

Copy link
Copy Markdown
Contributor
Preview URLs (6 pages)
Flaws (19)

Note! 2 documents with no flaws that don't need to be listed. 🎉

Found an unexpected or unresolvable flaw? Please report it here.

URL: /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script
Title: script module
Flaw count: 5

  • broken_links:
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/evaluate doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/callFunction doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/evaluate#target doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/callFunction#target doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/callFunction doesn't resolve

URL: /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/addPreloadScript
Title: script.addPreloadScript command
Flaw count: 6

  • broken_links:
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/message doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/message doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/message doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/message doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/evaluate doesn't resolve
    • and 1 more flaws omitted

URL: /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/disown
Title: script.disown command
Flaw count: 6

  • broken_links:
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/evaluate doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/evaluate#resultownership doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/evaluate doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/evaluate#resultownership doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/evaluate doesn't resolve
    • and 1 more flaws omitted

URL: /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/removePreloadScript
Title: script.removePreloadScript command
Flaw count: 2

  • broken_links:
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/evaluate doesn't resolve
    • Link /en-US/docs/Web/WebDriver/Reference/BiDi/Modules/script/callFunction doesn't resolve

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:WebDriver WebDriver docs size/l [PR only] 501-1000 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant