XUL attribute persistence is broken in browser.xhtml

RESOLVED FIXED in Firefox 65

Status

()

enhancement
P3
normal
RESOLVED FIXED
7 months ago
6 months ago

People

(Reporter: bdahl, Assigned: bdahl)

Tracking

unspecified
Firefox 65
Points:
---

Firefox Tracking Flags

(firefox65 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 months ago
XUL attribute persistence is handled by the XUL document, so when we move to HTML the same behavior does not exist.

Example test that is broken:
browser/base/content/test/performance/browser_window_resize.js
(Assignee)

Updated

7 months ago
Assignee: nobody → bdahl
Priority: -- → P3
Status: NEW → ASSIGNED
(Assignee)

Comment 1

7 months ago
Previously, XUL attribute persistence was mainly handled by the XUL
document which doesn't work for browser.xhtml. Move the attribute watching
and persisting code to the XUL element and move the loading and setting
of the persisted attributes into XULStore, so they can be used by XUL and
HTML.

How the stored values are applied on document load is handled
slightly differently in XUL and HTML. In a XUL document, the attributes
are automatically applied after parse. In an HTML document we don't want
to have this behavior by default, so applyPersistentAttributes must be
called explicitly if the page wants to use this feature. This also gives
the page finer control over when to apply the persisted attributes.
Comment on attachment 9011856 [details]
Bug 1490464 - Fix XUL attribute persistence for browser.xhtml. r=smaug

Olli Pettay [:smaug] (r- if the bug doesn't explain what the change(s) are about.) has approved the revision.
Attachment #9011856 - Flags: review+
Comment on attachment 9011856 [details]
Bug 1490464 - Fix XUL attribute persistence for browser.xhtml. r=smaug

Olli Pettay [:smaug] (r- if the bug doesn't explain what the change(s) are about.) has requested changes to the revision.
Attachment #9011856 - Flags: review+

Comment 4

6 months ago
Pushed by bdahl@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4350340785e4
Fix XUL attribute persistence for browser.xhtml. r=smaug

Comment 5

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/4350340785e4
Status: ASSIGNED → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
You need to log in before you can comment on or make changes to this bug.