Closed Bug 1869718 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 43643 - Make the ignore case is respected in the SW static routing URLPattern.

Categories

(Core :: DOM: Service Workers, task, P4)

Product:
Core
Component:
DOM: Service Workers
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
122 Branch
Tracking Flags:
Tracking Status
firefox122 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 43643 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/43643
Details from upstream follow.

Yoshisato Yanagisawa <yyanagisawa@chromium.org> wrote:

Make the ignore case is respected in the SW static routing URLPattern.

This CL set the case sensitive option to false if ignore case flag is
set in the SafeUrlPattern.

Removed TODOs because the conversion is actually done in the URLPatern
class and the ignore case option is respected.

Change-Id: I4e2197484c3e31aab3df54abf979adff944879c5
Bug: 1371756
Reviewed-on: https://chromium-review.googlesource.com/5118999
WPT-Export-Revision: 308277a2a413e51045c29b002f8cbea000f362e0

Component: web-platform-tests → DOM: Service Workers
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 2 tests and 13 subtests

Status Summary

Firefox

OK : 1[Gecko-windows11-32-2009-qr-opt] 2[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub]
FAIL : 13[Gecko-windows11-32-2009-qr-opt] 17[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub]
TIMEOUT: 2
NOTRUN : 3

Chrome

OK : 1
PASS : 3
FAIL : 10
TIMEOUT: 2
NOTRUN : 3

Safari

OK : 2
FAIL : 17

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

  • /service-workers/service-worker/tentative/static-router/static-router-main-resource.https.html [wpt.fyi]: OK [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub], TIMEOUT [Gecko-windows11-32-2009-qr-opt] (Chrome: TIMEOUT, Safari: OK)
    • Main resource load matched with the condition: FAIL [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub], TIMEOUT [Gecko-windows11-32-2009-qr-opt] (Chrome: TIMEOUT, Safari: FAIL)
    • Main resource load matched with the ignore case condition: FAIL [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub], NOTRUN [Gecko-windows11-32-2009-qr-opt] (Chrome: NOTRUN, Safari: FAIL)
    • Main resource load matched without the ignore case condition: FAIL [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub], NOTRUN [Gecko-windows11-32-2009-qr-opt] (Chrome: NOTRUN, Safari: FAIL)
    • Main resource load not matched with the condition: FAIL [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub], NOTRUN [Gecko-windows11-32-2009-qr-opt] (Chrome: NOTRUN, Safari: FAIL)
  • /service-workers/service-worker/tentative/static-router/static-router-subresource.https.html [wpt.fyi]
    • Subresource load not matched with URLPattern condition: FAIL (Chrome: PASS, Safari: FAIL)
    • Subresource load matched with URLPattern condition: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource cross origin load matched with URLPattern condition via constructed object: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource load matched with ignoreCase URLPattern condition: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource load matched without ignoreCase URLPattern condition: FAIL (Chrome: PASS, Safari: FAIL)
    • Subresource load matched with URLPattern condition via URLPatternInit: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource cross origin load not matched with URLPattern condition via URLPatternInit: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource load matched with URLPattern condition via string: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource cross origin load not matched with URLPattern condition via string: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource load matched with RequestMode condition: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource load matched with the nested or condition: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource load matched with the next or condition: FAIL (Chrome: FAIL, Safari: FAIL)
    • Subresource load not matched with or condition: FAIL (Chrome: PASS, Safari: FAIL)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9d489674a456 [wpt PR 43643] - Make the ignore case is respected in the SW static routing URLPattern., a=testonly https://hg.mozilla.org/integration/autoland/rev/45ca5167faab [wpt PR 43643] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/9d489674a456
https://hg.mozilla.org/mozilla-central/rev/45ca5167faab

Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox122: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
You need to log in before you can comment on or make changes to this bug.