Closed Bug 1223751 Opened 7 years ago Closed 6 years ago

If the top level content document is assuming all frames are visible, subdocuments should be too


(Core :: Layout: Images, Video, and HTML Frames, defect)

Not set



Tracking Status
firefox48 --- fixed


(Reporter: seth, Unassigned)


(Blocks 1 open bug)



(1 file)

Right now we can get in a nasty situation where we won't update frame visibility in subdocuments if the top level content document is assuming all frames are visible:

[23:22:25]  <seth>	*** nsPresShell 0x188446800: ScheduleFrameVisibilityUpdate
[23:22:25]  <seth>	***    - scheduling visibility update for root content doc with pres shell 0x180ad4000
[23:22:25]  <seth>	*** nsPresShell 0x180ad4000: ScheduleFrameVisibilityUpdate
[23:22:25]  <seth>	*** AssumeAllFramesVisible: yes, because we're a xul doc
[23:22:25]  <seth>	***    - early return because we're assuming all frames are visible
[23:23:02]  <seth>	so if the root content doc is a xul doc, we don't update frame visibility even for nested docs?
[23:31:20]  <seth>	this looks pretty bogus

That means that images in those subdocuments won't animated, and won't be locked, among other things.

Bug 1223747 eliminates the problem for XUL documents by no longer assuming that all images are visible in them, but there are still other kinds of documents like resource documents that we make this assumption for. We should prevent this problem from happening in those cases as well. The solution is to have AssumeAllFramesVisible() calls in subdocuments return true if the top level content document's pres shell is returning true for AssumeFramesVisible().
Attachment #8685970 - Flags: review?(tnikkel) → review+
This seems like something that would be good to get landed ASAP, as then any changes it causes can be easily tracked and noticed sooner.
Bug 1223751 - Assume all frames are visible in subdocuments of a top level content document that's assuming all frames are visible. r=tn
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Product: Core → Core Graveyard
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.