1810304 - Perma TEST-UNEXPECTED-PASS | /loading/early-hints/iframe-pdf.h2.window.html | Early hints for an iframe of which content is pdf should be ignored. - expected FAIL when Gecko 110 merges to Beta on 2023-01-16

Status: VERIFIED FIXED

(Core :: Networking: HTTP, defect, P3)

Description

[Iulian Moraru]

[Tracking Requested - why for this release]:

• Central-as-beta simulation.
• How to run these simulations.

This is only failing on Android 7.0 x86-64 Lite WebRender Shippable and Android 7.0 x86-64 Shippable WebRender.

Failure log:

[task 2023-01-14T12:16:23.150Z] 12:16:23     INFO - TEST-PASS | /loading/early-hints/iframe-coep-disallow.h2.html | Early hints for an iframe that violates Cross-Origin-Embedder-Policy should be ignored. 
[task 2023-01-14T12:16:23.150Z] 12:16:23     INFO - TEST-UNEXPECTED-PASS | /loading/early-hints/iframe-pdf.h2.window.html | Early hints for an iframe of which content is pdf should be ignored. - expected FAIL
[task 2023-01-14T12:16:23.150Z] 12:16:23     INFO - TEST-INFO | expected FAIL
[task 2023-01-14T12:16:23.220Z] 12:16:23     INFO - TEST-OK | /loading/early-hints/iframe-pdf.h2.window.html | took 773ms
[task 2023-01-14T12:16:23.591Z] 12:16:23     INFO - STDOUT: cleanup aborted: Unable to remount device
[task 2023-01-14T12:16:23.687Z] 12:16:23     INFO - STDOUT: cleanup aborted: Unable to remount device
[task 2023-01-14T12:16:23.688Z] 12:16:23     INFO - Closing logging queue
[task 2023-01-14T12:16:23.688Z] 12:16:23     INFO - queue closed
[task 2023-01-14T12:16:23.701Z] 12:16:23     INFO - Setting up ssl
[task 2023-01-14T12:16:23.718Z] 12:16:23     INFO - certutil | b''
[task 2023-01-14T12:16:23.739Z] 12:16:23     INFO - certutil | b''
[task 2023-01-14T12:16:23.750Z] 12:16:23     INFO - certutil | b'\nCertificate Nickname                                         Trust Attributes\n                                                             SSL,S/MIME,JAR/XPI\n\nweb-platform-tests                                           CT,, \n'
[task 2023-01-14T12:16:24.339Z] 12:16:24     INFO - adb Granting important runtime permissions to org.mozilla.geckoview.test_runner
[task 2023-01-14T12:16:25.578Z] 12:16:25     INFO - adb launch_application: am start -W -n org.mozilla.geckoview.test_runner/org.mozilla.geckoview.test_runner.TestRunnerActivity -a android.intent.action.MAIN --es env0 MOZ_CRASHREPORTER=1 --es env1 MOZ_CRASHREPORTER_NO_REPORT=1 --es env2 MOZ_CRASHREPORTER_SHUTDOWN=1 --es env3 MOZ_HIDE_RESULTS_TABLE=1 --es env4 MOZ_IN_AUTOMATION=1 --es env5 MOZ_LOG=signaling:3,mtransport:4,DataChannel:4,jsep:4 --es env6 R_LOG_LEVEL=6 --es env7 R_LOG_DESTINATION=stderr --es env8 R_LOG_VERBOSE=1 --es env9 MOZ_PROCESS_LOG=/tmp/tmpsy2rmdl2pidlog --es env10 MOZ_DISABLE_NONLOCAL_CONNECTIONS=1 --es env11 STYLO_THREADS=1 --es arg0 -no-remote --es arg1 -profile --es arg2 /data/local/tmp/test_root/profile --es arg3 --marionette --es arg4 about:blank --ez use_multiprocess True
[task 2023-01-14T12:16:26.894Z] 12:16:26     INFO - Starting runner
[task 2023-01-14T12:16:27.853Z] 12:16:27     INFO - TEST-START | /loading/early-hints/iframe-x-frame-options-deny.h2.window.html

Comment 1

[Iulian Moraru]

Hi Manuel! Can you please take a look at this?
Thank you!

Comment 2

[Manuel Bucher [:manuel]]

Nothing to worry about, we just need to update the meta .ini file to allow passing the test. I can prepare a patch. The test gets skipped (always pass) if the in-browser pdf reader is disables. So I guess that is the case for these two platforms, but will look into it as well.

Comment 3

[Manuel Bucher [:manuel]]

Attachment: Bug 1810304 - Allow iframe-pdf web-platform test to pass r=#necko

I'm not special casing the two platforms, because this test does nothing anyway until Bug 1809894 gets fixed

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/916c594abced
Allow iframe-pdf web-platform test to pass r=necko-reviewers,valentin

Comment 5

[Serban Stanca [:SerbanS]]

Patch is working in today's central as beta simulation.

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1799167

Comment 7

[Marian-Vasile Laza]

https://hg.mozilla.org/mozilla-central/rev/916c594abced

Comment 8

[Pascal Chevrel:pascalc]

Should we uplift to beta? Thanks

Comment 9

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Yes. It's test-only, no uplift request needed.

Comment 10

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:manuel, is this bug important enough to require an uplift?

• If yes, please nominate the patch for beta approval.
• If no, please set status-firefox110 to wontfix.

For more information, please visit auto_nag documentation.

Comment 11

[Noemi Erli[:noemi_erli]]

Verified on todays beta simulation: https://treeherder.mozilla.org/jobs?repo=try&resultStatus=success%2Ctestfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=d45253a0775735a7705419e4656da37dc3e2fc26&searchStr=android%2C7.0%2Cx86-64%2Clite%2Cwebrender%2Cshippable%2Copt%2Cweb%2Cplatform%2Ctests%2Cwithout%2Cfission%2Cenabled%2Ctest-android-em-7.0-x86_64-shippable-lite-qr%2Fopt-geckoview-web-platform-tests-nofis%2Cwpt21

Comment 12

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/releases/mozilla-beta/rev/c5767a4cb4e0