`includeGlobs` is not supported

[ryankashi]

Hi there! I am using your polyfill to register my content script via:

var content_script_registration_dict = {
"matches":["*://*.twitch.tv/*"],
"includeGlobs":['*player.twitch.tv*'],
"js":[{file:"my_javascript.js"}],
"css":[{file:"my_css.css"}]
}

chrome.contentScripts.register(content_script_registration_dict);

However I am noticing that my includeGlobs segment is being ignored. Instead, it ignores the includeGlobs match pattern, and injects my_javascript.js into all pages under matches regardless of whether it matches includeGlobs or not.

When I instead place my JavaScript code inside the manifest.json file, everything works as expected, thus it appears to be an issue with this package.

I think that this is an important fix for this package since includeGlobs is currently not behaving as expected.

Thanks!

[ryankashi]

Update, it appears that the excludeMatches pattern as well excludeGlobs are not working either

[fregante]

Yes, you can see which properties are supported here:

content-scripts-register-polyfill/ponyfill.ts

Lines 89 to 92 in 0263492

	matchAboutBlank,
	allFrames,
	frameId,
	runAt,

PR welcome to add what's missing

[fregante]

Support for exclude_matches added in v3.1

The globs depend on:

• Add globsToRegex webext-patterns#9