Skip to content

fix: honor disabled site rules in embedded frames - #1643

Open
jay23606 wants to merge 1 commit into
igrigorik:masterfrom
jay23606:fix/1634-site-rules-in-frames
Open

jay23606 wants to merge 1 commit into
igrigorik:masterfrom
jay23606:fix/1634-site-rules-in-frames

Conversation

@jay23606

@jay23606 jay23606 commented Oct 9, 2026

Copy link
Copy Markdown

Summary

  • Apply disabled site rules to the current frame and its ancestor origins.
  • Prevent the Video Speed Controller from initializing inside an embedded player when the enclosing site is disabled.

Closes #1634

Validation

  • Focused content bridge and site pattern tests: 37 passed.
  • Direct ESLint checks pass for all source and test JavaScript.
  • Prettier check passes for both changed files.
  • Release build passes with RELEASE=1 node scripts/build.mjs.
  • Full test suite: 552 passed; two release-helper tests cannot run on this Windows environment because unzip is unavailable.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Site rules do not disable the extension on looptube.io

1 participant