userScripts / scripting API 调整，增强兼容性 ( 重做 #704 ) #925
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概述 Descriptions
chrome.scripting.registerContentScripts#704之前
@early-start的代码修改，导致 #704 的修改被倒回去了现在重做 #704
历史原因，
scripting.RegisteredContentScript的设计不是用来做 自订脚本 用途，不支持动态代码。它只支持
url网址，，而不支持code代码这个PR做了在网址取 flags 的处理，因此不需要 code
但相反 chrome 不支持网址参数，因此还是用 userScripts API 做注入
已测试过能在 Firefox & Chrome 顺利执行
变更内容 Changes
截图 Screenshots