Closed Bug 1404668 Opened 7 years ago Closed 7 years ago

stylo: ensure we update style backend after document.open()

Categories

(Core :: CSS Parsing and Computation, enhancement)

Product:
Core
Component:
CSS Parsing and Computation
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WORKSFORME

People

(Reporter: heycam, Unassigned)

References

Details

From bug 1403077 comment 25:

> The one thing to consider is that document URI/principal can be reset via
> HTMLDocument::Open (document.open). I believe all the content is ejected there,
> but we should update the style backend, and verify that we end up with a new
> style set. Automated tests for this would probably be good.
Bobby, does this mean that the principal won't actually change when document.open() is called?

http://searchfox.org/mozilla-central/rev/298033405057ca7aa5099153797467eceeaa08b5/dom/html/nsHTMLDocument.cpp#1575-1603
Flags: needinfo?(bobbyholley)
(In reply to Cameron McCormack (:heycam) (away 4 Oct) from comment #1)
> Bobby, does this mean that the principal won't actually change when
> document.open() is called?
> 
> http://searchfox.org/mozilla-central/rev/
> 298033405057ca7aa5099153797467eceeaa08b5/dom/html/nsHTMLDocument.cpp#1575-
> 1603

Ah nice. WFM then.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(bobbyholley)
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.