Closed
Bug 428767
Opened 16 years ago
Closed 16 years ago
Unbreak ourselves after bug 292789 removed message access to messageBody.css
Categories
(Thunderbird :: Security, defect)
Thunderbird
Security
Tracking
(Not tracked)
VERIFIED
FIXED
Thunderbird 3
People
(Reporter: philor, Assigned: philor)
References
()
Details
(Keywords: regression)
Attachments
(2 files)
1.10 KB,
patch
|
mkmelin
:
review+
dveditz
:
review+
|
Details | Diff | Splinter Review |
1.40 KB,
patch
|
iannbugzilla
:
review+
dveditz
:
review+
|
Details | Diff | Splinter Review |
Mirror the fix made to the only-application-that-matters to keep the bug 292789 fix to prevent content from sniffing for extensions from breaking us - in the case I noticed so far, breaking us by preventing the load of messageBody.css for displayed messages.
Attachment #315349 -
Flags: review?(mkmelin+mozilla)
Comment 2•16 years ago
|
||
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008041302 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4) Same regression with SM: {{ Security Error: Content at imap://... may not load or link to chrome://messenger/skin/messageBody.css. }}
Updated•16 years ago
|
Attachment #315377 -
Flags: review+
Comment 4•16 years ago
|
||
Comment on attachment 315349 [details] [diff] [review] Fix v.1 [Checkin: Comment 7] r=dveditz -- very sorry I didn't catch this!
Attachment #315349 -
Flags: review+
Comment 5•16 years ago
|
||
sr=dveditz, even, if you need it to check in (I imagine not for mozilla/mail, dunno about mozilla/mailnews)
Comment 6•16 years ago
|
||
Comment on attachment 315349 [details] [diff] [review] Fix v.1 [Checkin: Comment 7] sure, r=mkmelin
Attachment #315349 -
Flags: review?(mkmelin+mozilla) → review+
Assignee | ||
Comment 7•16 years ago
|
||
mail/base/jar.mn 1.108
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Attachment #315349 -
Attachment description: Fix v.1 → Fix v.1
[Checkin: Comment 7]
Comment on attachment 315377 [details] [diff] [review] (Bv1-SM) <jar.mn> [Checkin: Comment 9] r=me for that simple change
Attachment #315377 -
Flags: review?(mnyromyr) → review+
Updated•16 years ago
|
Comment 9•16 years ago
|
||
mozilla/mailnews/jar.mn 1.126
Keywords: checkin-needed
Whiteboard: [c-n: Bv1-SM]
Updated•16 years ago
|
Attachment #315377 -
Attachment description: (Bv1-SM) <jar.mn> → (Bv1-SM) <jar.mn>
[Checkin: Comment 9]
Comment 10•16 years ago
|
||
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9pre) Gecko/2008041401 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4) V.Fixed, with SM. (Assuming as much with TB.)
Status: RESOLVED → VERIFIED
Comment 11•16 years ago
|
||
FTR, bug 428996 backed these out and solved the issue in an improved/narrowed way, per bug 292789 comment 110.
You need to log in
before you can comment on or make changes to this bug.
Description
•