Open Bug 1250503 Opened 9 years ago Updated 4 years ago

Only listen for app-theme-changed for pageshow() on system-principal documents

Categories

(Core :: DOM: Core & HTML, defect, P5)

Product:
Core
Component:
DOM: Core & HTML
Type:
defect

Tracking

()

Tracking Flags:
Tracking Status
firefox47 --- affected

People

(Reporter: Gijs, Unassigned)

References

(Blocks 1 open bug)

Details

We only care about these when the document has the system principal: https://dxr.mozilla.org/mozilla-central/rev/789a12291942763bc1e3a89f97e0b82dc1c9d00b/dom/base/nsDocument.cpp#12337-12347 so we might as well actually check that before we even add the observer - presumably, when pageshow fires and we add the observer, there's a pretty good guarantee that we'll actually have a principal. This would potentially miss out documents where we change the document's principal after pageshow, but I should hope that we don't do that... :-\

Bulk-downgrade of unassigned, >=3 years untouched DOM/Storage bug's priority.

If you have reason to believe this is wrong, please write a comment and ni :jstutte.

Severity: normal → S4
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.