Closed Bug 1611961 Opened 5 years ago Closed 4 years ago

Move `UserActivationState` from BrowsingContext to WindowContext

Categories

(Core :: DOM: Core & HTML, task, P2)

task

Tracking

()

RESOLVED FIXED
81 Branch
Fission Milestone M6b
Tracking Status
firefox81 --- fixed

People

(Reporter: edgar, Assigned: edgar)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

https://html.spec.whatwg.org/#user-activation-data-model

When I implemented the user-activation-state, the WindowContext isn't available. It is a better place to put the user-activation-state.

Edgar, will you be working on this bug now that WindowContext (bug 1583863) has landed?

Tracking for Fission Nightly (M6)

Fission Milestone: --- → M6
Depends on: WindowContext
Flags: needinfo?(echen)
Priority: -- → P3

Yes, I will work on this bug.

Flags: needinfo?(echen)

(In reply to Edgar Chen [:edgar] from comment #2)

Yes, I will work on this bug.

Thanks! I will tentatively assign this bug to you so I know this bug has an owner.

Assignee: nobody → echen

Tracking for Fission Nightly M6b.

Would be nice to have this soon.

Fission Milestone: M6 → M6b

(Note to myself: this change might help https://bugzilla.mozilla.org/show_bug.cgi?id=1648887#c7)

Edgar, can you please get this patch ready for review?

Flags: needinfo?(echen)
Blocks: 1648887

(In reply to Neha Kochar [:neha] from comment #9)

Edgar, can you please get this patch ready for review?

Patch is ready for review now.

Flags: needinfo?(echen)
Severity: normal → N/A
Priority: P3 → P2
Pushed by echen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ab82a9c613f8 Move UserActivationState from BrowsingContext to WindowContext; r=smaug
Regressions: 1657625

Backed out for assertion failures on WindowContext.cpp

backout: https://hg.mozilla.org/integration/autoland/rev/de5aa7dd22b53113a65de864843257cfb9e53d60

push: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&searchStr=linux%2C18.04%2Cx64%2Cwebrender%2Cdebug%2Creftests%2Cwith%2Cfission%2Cenabled%2Ctest-linux1804-64-qr%2Fdebug-crashtest-fis-e10s%2Cc&revision=3719f7db339d59cc5f109ff84879f6fd745c40ce&selectedTaskRun=comz08giTsaawea2LS08cg.0

failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=312268933&repo=autoland&lineNumber=1924

[task 2020-08-06T13:52:35.187Z] 13:52:35 INFO - Assertion failure: mInProcess, at /builds/worker/checkouts/gecko/docshell/base/WindowContext.cpp:321
[task 2020-08-06T13:52:35.187Z] 13:52:35 INFO - #01: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x7c97ae2]
[task 2020-08-06T13:52:35.188Z] 13:52:35 INFO - #02: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x7c98190]
[task 2020-08-06T13:52:35.188Z] 13:52:35 INFO - #03: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x7c96759]
[task 2020-08-06T13:52:35.190Z] 13:52:35 INFO - #04: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x7c9ee67]
[task 2020-08-06T13:52:35.190Z] 13:52:35 INFO - #05: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x7c9e5c4]
[task 2020-08-06T13:52:35.191Z] 13:52:35 INFO - #06: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x7c9ef2c]
[task 2020-08-06T13:52:35.191Z] 13:52:35 INFO - #07: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x4cfe030]
[task 2020-08-06T13:52:35.191Z] 13:52:35 INFO - #08: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x4cfd9b3]
[task 2020-08-06T13:52:35.192Z] 13:52:35 INFO - #09: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x4cfc56e]
[task 2020-08-06T13:52:35.192Z] 13:52:35 INFO - #10: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x4cfd0f2]
[task 2020-08-06T13:52:35.196Z] 13:52:35 INFO - #11: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x4cfd67b]
[task 2020-08-06T13:52:35.199Z] 13:52:35 INFO - #12: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x421ba2a]
[task 2020-08-06T13:52:35.199Z] 13:52:35 INFO - #13: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x421ca09]
[task 2020-08-06T13:52:35.201Z] 13:52:35 INFO - #14: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x4512d0b]
[task 2020-08-06T13:52:35.205Z] 13:52:35 INFO - #15: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x4517a00]
[task 2020-08-06T13:52:35.206Z] 13:52:35 INFO - #16: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x453c35c]
[task 2020-08-06T13:52:35.207Z] 13:52:35 INFO - #17: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x462773e]
[task 2020-08-06T13:52:35.207Z] 13:52:35 INFO - #18: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x463f693]
[task 2020-08-06T13:52:35.208Z] 13:52:35 INFO - #19: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x40e23ab]
[task 2020-08-06T13:52:35.211Z] 13:52:35 INFO - #20: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x40e0d31]
[task 2020-08-06T13:52:35.212Z] 13:52:35 INFO - #21: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x40e017d]
[task 2020-08-06T13:52:35.212Z] 13:52:35 INFO - #22: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x40e02f5]
[task 2020-08-06T13:52:35.213Z] 13:52:35 INFO - #23: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x40e75e9]
[task 2020-08-06T13:52:35.214Z] 13:52:35 INFO - #24: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x40f4be5]
[task 2020-08-06T13:52:35.215Z] 13:52:35 INFO - #25: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x40f8b17]
[task 2020-08-06T13:52:35.216Z] 13:52:35 INFO - #26: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x4728d9c]
[task 2020-08-06T13:52:35.216Z] 13:52:35 INFO - #27: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x46d6a01]
[task 2020-08-06T13:52:35.220Z] 13:52:35 INFO - #28: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x46d695b]
[task 2020-08-06T13:52:35.220Z] 13:52:35 INFO - #29: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x6ee08e7]
[task 2020-08-06T13:52:35.222Z] 13:52:35 INFO - #30: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x8074418]
[task 2020-08-06T13:52:35.222Z] 13:52:35 INFO - #31: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x4729345]
[task 2020-08-06T13:52:35.223Z] 13:52:35 INFO - #32: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x46d6a01]
[task 2020-08-06T13:52:35.224Z] 13:52:35 INFO - #33: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x46d695b]
[task 2020-08-06T13:52:35.225Z] 13:52:35 INFO - #34: ??? [/builds/worker/workspace/build/application/firefox/libxul.so + 0x8073fca]
[task 2020-08-06T13:52:35.227Z] 13:52:35 INFO - #35: content_process_main(mozilla::Bootstrap*, int, char**) [ipc/contentproc/plugin-container.cpp:57]
[task 2020-08-06T13:52:35.228Z] 13:52:35 INFO - #36: main [browser/app/nsBrowserApp.cpp:303]
[task 2020-08-06T13:52:35.237Z] 13:52:35 INFO - #37: __libc_start_main [/lib/x86_64-linux-gnu/libc.so.6 + 0x21b97]
[task 2020-08-06T13:52:35.241Z] 13:52:35 INFO - #38: ??? [/builds/worker/workspace/build/application/firefox/firefox-bin + 0x40029]
[task 2020-08-06T13:52:35.242Z] 13:52:35 INFO - #39: ??? (???:???)

Flags: needinfo?(echen)

I could not reproduce this locally, it seems only to be reproducible on try.
Just got a pernosco session for debugging: https://pernos.co/debug/sZPlqy5NOCsLQ_IOGHOHTA/index.html (thanks to khuey).

Flags: needinfo?(echen)

The assertion happens when we try to load error page, it happens asynchronously, it could possible that we are in the middle of the switching process or the parent has already navigated to another page which would reset the WindowContext. It looks like that we don't really need to grab the user activation state for loading loading-error-page (about:neterror).

Depends on: 1658572
Depends on: 1658650
Pushed by echen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1f2300618024 Move UserActivationState from BrowsingContext to WindowContext; r=smaug
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 81 Branch
Regressions: 1679417
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: