I looked to the code in MatchURLFilters.jsm and I confirm that the correct behaviors seems to be the one that Seth describes in comment 2 code snippets (where if none of url doesn't a port that is included in the ports range than the url shouldn't match, but matching the port should not make the url to match the url filters if it doesn't also match the other criteria as well). The test case in https://searchfox.org/mozilla-central/rev/e66593593f3b356901011ea0fcdf9979728e9ae8/toolkit/components/extensions/test/mochitest/test_ext_webnavigation_filters.html#87-118 do not cover this expected behavior properly and should also be updated to fully cover the correct expected behavior when ports are included in the url filter.
Bug 1755263 Comment 3 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
I looked to the code in MatchURLFilters.jsm and I confirm that the correct behaviors seems to be the one that Seth describes in comment 2 code snippets (where if the url port is not included in the ports range than the url shouldn't match, but matching just the port should not make the url to match if the other criteria are not matched as well). The test case in https://searchfox.org/mozilla-central/rev/e66593593f3b356901011ea0fcdf9979728e9ae8/toolkit/components/extensions/test/mochitest/test_ext_webnavigation_filters.html#87-118 do not cover this expected behavior properly and should also be updated to fully cover the correct expected behavior when ports are included in the url filter.
I looked to the code in MatchURLFilters.jsm and I confirm that the correct behaviors seems to be the one that Seth describes in comment 2 code snippets (where if the url port is not included in the ports range than the url shouldn't match, but matching just the port should not make the url to match if the other criteria are not matched as well). The test case in https://searchfox.org/mozilla-central/rev/e66593593f3b356901011ea0fcdf9979728e9ae8/toolkit/components/extensions/test/mochitest/test_ext_webnavigation_filters.html#87-118 does not cover this expected behavior properly and should also be updated to fully cover the correct expected behavior when ports are included in the url filter.