402781 - ASSERTION: Principals should be the same, with data url @import rules

Status: RESOLVED WORKSFORME

(Core :: Security, defect)

Description

[Martijn Wargers (dead)]

Attachment: zipped testcase

See zipped up testcase, to reproduce the assertion in your debug build, unzip the zipped up testcase and then open the file named 'principals should be the same.htm'.

Boris, you might be interested in this assertion, since you seem to have added in bug 221428.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Yeah, this is a consequence of nsIPrincipal::Equals being asymmetric ever since the patch for bug 230606 landed.  When looking up a new sheet load in the hashtable, we key on URI and loading principal.

In this case, the same URI is being loaded by both the document directly and by a.css.

When doing the hashtable lookup we presumably do document->Equals(a.css), which tests true.  Then the assertion tests the other direction, which tests false due to the abovementioned asymmetry.

The real issue is that the patch for bug 230606 is wrong as-is...

Comment 2

[Martijn Wargers (dead)]

No assertion anymore in current trunk build.