Skip to content

Editorial: Remove inaccurate SecurityError note from createImageBitmap#12478

Open
Devansh232406 wants to merge 1 commit into
whatwg:mainfrom
Devansh232406:fix-imagebitmap-note
Open

Editorial: Remove inaccurate SecurityError note from createImageBitmap#12478
Devansh232406 wants to merge 1 commit into
whatwg:mainfrom
Devansh232406:fix-imagebitmap-note

Conversation

@Devansh232406
Copy link
Copy Markdown

@Devansh232406 Devansh232406 commented May 22, 2026

  • At least two implementers are interested (and none opposed):
  • Tests are written and can be reviewed and commented upon at:
  • Implementation bugs are filed:
    • Chromium: …
    • Gecko: …
    • WebKit: …
    • Deno (only for timers, structured clone, base64 utils, channel messaging, module resolution, web workers, and web storage): …
    • Node.js (only for timers, structured clone, base64 utils, channel messaging, and module resolution): …
  • Corresponding HTML AAM & ARIA in HTML issues & PRs:
  • MDN issue is filed: …
  • The top of this comment includes a clear commit message to use.

(See WHATWG Working Mode: Changes for more details.)


/imagebitmap-and-animations.html ( diff )

Copy link
Copy Markdown
Member

@annevk annevk left a comment

Choose a reason for hiding this comment

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

Did you also check in web-platform-tests if any of the createImageBitmap() tests expect this exception?

Comment thread source
Comment on lines 128143 to +128144

<p>Rejects the promise with a <span>"<code>SecurityError</code>"</span>
<code>DOMException</code> if the script is not allowed to access the image data of the source
image (e.g. a <code>video</code> that is <span>CORS-cross-origin</span>, or a
<code>canvas</code> being drawn on by a script in a worker from another
<span>origin</span>).</p>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should remove these two newlines as well.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants