Closed
Bug 507457
Opened 15 years ago
Closed 15 years ago
Firefox crashes when opening certain e-mails in aol webmail. [@nsStyleSet::AddImportantRules(nsRuleNode*, nsRuleNode*)]
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Tracking
()
RESOLVED
FIXED
mozilla1.9.2a1
Tracking | Status | |
---|---|---|
status1.9.1 | --- | .3-fixed |
People
(Reporter: katie7070, Assigned: dbaron)
References
()
Details
(Keywords: crash, regression, verified1.9.1)
Crash Data
Attachments
(5 files)
(deleted),
application/x-xpinstall
|
Details | |
(deleted),
text/plain; charset=UTF-8
|
Details | |
(deleted),
patch
|
bzbarsky
:
review+
samuel.sidler+old
:
approval1.9.1.3+
|
Details | Diff | Splinter Review |
(deleted),
text/plain; charset=UTF-8
|
Details | |
(deleted),
patch
|
bzbarsky
:
review+
samuel.sidler+old
:
approval1.9.1.3+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1.1) Gecko/20090715 Firefox/3.5.1
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1.1) Gecko/20090715 Firefox/3.5.1
Whenever I open emails from the apple newsletter, and some from family, Firefox crashes. Hard. Some have images, others do not, so it does not seem to matter. It only occurs with certain e-mail addresses though.
Reproducible: Always
Steps to Reproduce:
1. Log into AOL webmail.
2. Click on emails from News@InsideApple.apple.com (the newsletter address) or other bad address.
3. It says "Loading" at the bottom like normal, then Firefox crashes.
Actual Results:
Firefox closes in it's crash. Then, it says "Would you like to submit a report?" After restarting it is fine until I open another email from apple.
Expected Results:
Firefox should have opened the email in the webmail thing and I should have been able to read it.
I'm using the default theme. The emails open fine in Safari.
Comment 1•15 years ago
|
||
(In reply to comment #1)
> Please read
> https://developer.mozilla.org/en/How_to_get_a_stacktrace_for_a_bug_report
Like this?
f62eaed8-c06d-4228-bb5b-227682090731 7/31/09 7:49 AM
99c88bb2-cca4-4ba0-8885-95c7f2090730 7/30/09 3:02 PM
4ba4abc2-0213-40b0-b143-178832090723 7/23/09 5:47 PM
Comment 3•15 years ago
|
||
Thank you for the crash IDs :
The top of the stack for bp-62eaed8-c06d-4228-bb5b-227682090731 :
0 XUL nsStyleSet::AddImportantRules r.h:305
1 XUL nsStyleSet::FileRules layout/style/nsStyleSet.cpp:588
2 XUL nsStyleSet::ResolveStyleFor layout/style/nsStyleSet.cpp:691
3 XUL nsElementSH::PostCreate dom/src/base/nsDOMClassInfo.cpp:7539
4 XUL XPCWrappedNative::GetNewOrUsed js/src/xpconnect/src/xpcwrappednative.cpp:575
5 XUL XPCWrappedNative::GetNewOrUsed js/src/xpconnect/src/xpcwrappednative.cpp:432
6 XUL XPCConvert::NativeInterface2JSObject js/src/xpconnect/src/xpcconvert.cpp:1146
7 XUL XPCConvert::NativeData2JS js/src/xpconnect/src/xpcconvert.cpp:470
8 XUL nsXPCWrappedJSClass::CallMethod js/src/xpconnect/src/xpcwrappedjsclass.cpp:1595
9 XUL nsXPCWrappedJS::CallMethod js/src/xpconnect/src/xpcwrappedjs.cpp:561
10 XUL PrepareAndDispatch xpcom/reflect/xptcall/src/md/unix/xptcstubs_unixish_x86.cpp:93
11 XUL PrepareAndDispatch
12 XUL nsContentPolicy::ShouldLoad content/base/src/nsContentPolicy.cpp:157
13 XUL CSSLoaderImpl::CheckLoadAllowed tentPolicyUtils.h:221
14 XUL CSSLoaderImpl::LoadStyleLink layout/style/nsCSSLoader.cpp:1809
15 XUL nsStyleLinkElement::DoUpdateStyleSheet content/base/src/nsStyleLinkElement.cpp:313
16 XUL nsStyleLinkElement::UpdateStyleSheet content/base/src/nsStyleLinkElement.cpp:201
17 XUL HTMLContentSink::ProcessLINKTag content/html/document/src/nsHTMLContentSink.cpp:2943
18 XUL CNavDTD::HandleDefaultStartToken parser/htmlparser/src/CNavDTD.cpp:1080
Component: General → Style System (CSS)
Keywords: crash
Product: Firefox → Core
QA Contact: general → style-system
Summary: Firefox crashes when opening certain e-mails in aol webmail. → Firefox crashes when opening certain e-mails in aol webmail. [@nsStyleSet::AddImportantRules(nsRuleNode*, nsRuleNode*)
Version: unspecified → 1.9.1 Branch
Severity: normal → critical
Summary: Firefox crashes when opening certain e-mails in aol webmail. [@nsStyleSet::AddImportantRules(nsRuleNode*, nsRuleNode*) → Firefox crashes when opening certain e-mails in aol webmail. [@nsStyleSet::AddImportantRules(nsRuleNode*, nsRuleNode*)]
Comment 4•15 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.1) Gecko/20090715 Firefox/3.5.1
I can reproduce this. I'll try to get a testcase.
Comment 5•15 years ago
|
||
Only reproducible with Greasemonkey or certain other extensions installed. Might have something to do with the stylesheet service?
Assignee | ||
Comment 6•15 years ago
|
||
Steps to reproduce starting from a clean profile (i.e., which extensions to install, how they need to be configured) might be helpful.
Comment 7•15 years ago
|
||
Tentative STR:
1) install Greasemonkey
2) create an AOL email account
3) send email with the following content to AOL account (I used Thunderbird's Insert->HTML... feature to do this):
<style media="only screen and (max-device-width: 0px)">*{}</style>
4) open email in AOL webmail
I'm working on a reduced extension right now to replace the Greasemonkey step.
OS: Mac OS X → All
Hardware: x86 → All
Comment 8•15 years ago
|
||
Here is a somewhat reduced extension that you can use instead of Greasemonkey in the above steps. It consists of a content policy component and an install manifest.
I think this bug is the same as bug 492675 and bug 466024, which makes this top crasher #8 in the current 3.5.1 crash stats. That makes this bug a bit more exciting!
Assignee | ||
Comment 9•15 years ago
|
||
Assignee | ||
Updated•15 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Assignee | ||
Comment 10•15 years ago
|
||
I thought we used to have these, and we still should.
Assignee | ||
Comment 11•15 years ago
|
||
I'm a bit skeptical that this is the same problem as the topcrash, which I think has been around for a while.
Assignee | ||
Updated•15 years ago
|
Attachment #392120 -
Flags: review?(bzbarsky)
Assignee | ||
Comment 12•15 years ago
|
||
That said, I think the same assertions probably would catch the problem that is causing the topcrash.
Assignee | ||
Comment 13•15 years ago
|
||
This basically undoes patch 9 from bug 156716. At the time, that fixed a bug that was causing inconsistency between the DOM screen API and media queries when I was set up with mirrored monitors. But later the fix stopped working and I saw the relevant mochitest failures again when running with my laptop mirrored to a larger monitor. I'm not sure if I've still been seeing them; I actually think not. But the area is enough of a mess that given that it can cause flushes I think we're better off with that bug than this one, although I'm not even sure there's a real tradeoff.
Attachment #392122 -
Flags: review?(bzbarsky)
Updated•15 years ago
|
Blocks: 466057
Keywords: regression
Updated•15 years ago
|
Attachment #392120 -
Flags: review?(bzbarsky) → review+
Updated•15 years ago
|
Attachment #392122 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 14•15 years ago
|
||
(In reply to comment #7)
> 3) send email with the following content to AOL account (I used Thunderbird's
> Insert->HTML... feature to do this):
> <style media="only screen and (max-device-width: 0px)">*{}</style>
Out of curiosity, did you get that snippet of HTML from the email mentioned in comment 0, or from some other source of email?
In any case, fixed on mozilla-central:
http://hg.mozilla.org/mozilla-central/rev/341662af9f31
http://hg.mozilla.org/mozilla-central/rev/d98069ba3d39
Status: NEW → RESOLVED
Closed: 15 years ago
Priority: -- → P3
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Assignee | ||
Updated•15 years ago
|
Attachment #392122 -
Flags: approval1.9.1.3?
Assignee | ||
Updated•15 years ago
|
Attachment #392120 -
Flags: approval1.9.1.3?
Comment 15•15 years ago
|
||
It was the email Apple sends out when you register at store.apple.com. I never figured out how to get them to send the newsletter mentioned in comment 0, but I figure they probably use the same CSS in all their emails.
Updated•15 years ago
|
Assignee: nobody → dbaron
Comment 16•15 years ago
|
||
Comment on attachment 392120 [details] [diff] [review]
assertions to catch underlying problem
Approved for 1.9.1.3. a=ss
Attachment #392120 -
Flags: approval1.9.1.3? → approval1.9.1.3+
Comment 17•15 years ago
|
||
Comment on attachment 392122 [details] [diff] [review]
patch
Also approved for 1.9.1.3. a=ss
Attachment #392122 -
Flags: approval1.9.1.3? → approval1.9.1.3+
Assignee | ||
Comment 18•15 years ago
|
||
Fixed for 1.9.1.3:
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/5608ed9651d6
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/56b47701a175
status1.9.1:
--- → .3-fixed
Assignee | ||
Comment 19•15 years ago
|
||
I wonder if the first patch in bug 503791 helps with the underlying screen issue here.
Comment 20•15 years ago
|
||
I crafted the special e-mail in TB and used the test extension above and got the crash using Firefox 3.5.2. Verified fixed with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.3pre) Gecko/20090817 Shiretoko/3.5.3pre (.NET CLR 3.5.30729).
Keywords: verified1.9.1
Updated•13 years ago
|
Crash Signature: [@nsStyleSet::AddImportantRules(nsRuleNode*, nsRuleNode*)]
You need to log in
before you can comment on or make changes to this bug.
Description
•