1639328 - Fission iframes in display: none subtrees don't throttle requestAnimationFrame and such.

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, defect)

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Frame

Comment 2

[Emilio Cobos Álvarez [:emilio]]

https://crisal.io/tmp/fission-frame-raf/ contains the difference between a same origin and cross origin iframe.

Comment 3

[Chris Peterson [:cpeterson]]

This bug is a Fission Nightly blocker (milestone M6c).

Comment 4

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

The severity field is not set for this bug.
:neha, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 5

[Neha Kochar [:neha]]

Emilio, this seems a very important M6c issue, would you be able to work on this?

Comment 6

[Emilio Cobos Álvarez [:emilio]]

I can poke, will unassign if I can't get to it.

Comment 7

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1639328 - Make sure BrowserChilds for OOP iframes start in a consistent state. r=smaug

Right now they start with a FullyHidden() effect info, but with a
"visible" widget, and thus active docshell and so on.

That's no good :)

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/954bf9f3dca5
Make sure BrowserChilds for OOP iframes start in a consistent state. r=smaug

Comment 9

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1639328 - Try to make the test more reliable.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/be370bea0702
Try to make the test more reliable. CLOSED TREE

Comment 11

[Razvan Maries]

Backed out for perma failures.

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=c742e992415fb5d467ab812caf70c5d30d37d387&failure_classification_id=2

Logs:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=312620891&repo=autoland&lineNumber=2218
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=312621078&repo=autoland&lineNumber=2341

Backout: https://hg.mozilla.org/integration/autoland/rev/682bc156c9719ab66f8ba182999c69210afb1528

Comment 12

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1639328 - Fix browser_hidden_iframe.js. r=yzen

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8628c5cddd14
Make sure BrowserChilds for OOP iframes start in a consistent state. r=smaug
https://hg.mozilla.org/integration/autoland/rev/23130282a937
Fix browser_hidden_iframe.js. r=yzen

Comment 14

[Razvan Maries]

Backed out for perma failures.

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedTaskRun=e9pG76iPRUCkKbyTleYVig.0&resultStatus=testfailed%2Cbusted%2Cexception&searchStr=linux%2C18.04%2Cx64%2Cwebrender%2Copt%2Cmochitests%2Ctest-linux1804-64-qr%2Fopt-mochitest-plain-e10s%2C4&revision=23130282a937f703c23003353b362a503be6e182

Log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=312680272&repo=autoland&lineNumber=3061

Backout: https://hg.mozilla.org/integration/autoland/rev/154c7195ede58a20ae6da49576d3e59e65c971ac

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e7058f34ca94
Make sure BrowserChilds for OOP iframes start in a consistent state. r=smaug
https://hg.mozilla.org/integration/autoland/rev/b9141033e4a4
Fix browser_hidden_iframe.js. r=yzen

Comment 16

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/e7058f34ca94
https://hg.mozilla.org/mozilla-central/rev/b9141033e4a4