Closed Bug 444894 Opened 17 years ago Closed 16 years ago

Crashes when zooming on some pages under linux [@ nsRuleNode::GetStyleBackground]

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Version:
1.9.0 Branch
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: glandium, Unassigned, NeedInfo)

References

(
URL
)

Details

(Keywords: crash)

Crash Data

Attachments

(3 files)

simple html file to reproduce the bug
113 bytes, text/html
Details
Full backtrace of epiphany crash
35.23 KB, application/octet-stream
Details
Full stack traces of consecutive nsStyleSet::BeginReconstruct calls
112.92 KB, text/plain
Details
When going at the given url and modifying zoom level, a crash occurs. Interestingly, this doesn't happen in iceweasel/firefox, but in gtkmozembed clients, such as galeon and epiphany. I couldn't reproduce with a specially crafted version of TestGtkEmbed, though. FWIW, I could reproduce the crash with the Ubuntu 8.04.1 i386 livecd. Here is the stacktrace with an optimized build: #0 0x00007fa3e57c6ccc in nsRuleNode::GetStyleBackground (this=0x4267cb8, aContext=<value optimized out>, aComputeData=<value optimized out>) at nsRuleNode.h:215 #1 0x00007fa3e57ca0e4 in nsStyleContext::CalcStyleDifference (this=0x4006810, aOther=0x426b278) at nsStyleStructList.h:79 #2 0x00007fa3e5706863 in CaptureChange (aOldContext=0x7fa3eaca19e0, aNewContext=0x7fa3eaa6f870, aFrame=0x41, aContent=0x0, aChangeList=0x41dcae0, aMinChange=nsChangeHint_RepaintFrame, aChangeToAssume=0) at nsFrameManager.cpp:1058 #3 0x00007fa3e57075b5 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x40068d0, aParentContent=0x3fac8d0, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1196 #4 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x4006700, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #5 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x40065f0, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #6 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x4006460, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #7 0x00007fa3e57078d9 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x40063e8, aParentContent=0x3fac6c0, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1125 #8 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x4006188, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #9 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x4006078, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #10 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x4011970, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #11 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x40117f0, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #12 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x40115f0, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #13 0x00007fa3e57078d9 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x4011578, aParentContent=0x3f97da0, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1125 #14 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x4067f28, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #15 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x403d8b0, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #16 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x403d7a0, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #17 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x403d5c0, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #18 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x403cd08, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #19 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x3f94038, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #20 0x00007fa3e57078d9 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x3f93fc0, aParentContent=0x3f90a90, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1125 #21 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x3f93440, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #22 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x3f8de50, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #23 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x3f8d7e8, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #24 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x3db6228, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #25 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x3db6460, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #26 0x00007fa3e5707961 in nsFrameManager::ReResolveStyleContext (this=0x3eede08, aPresContext=0x35a6240, aFrame=0x3db6130, aParentContent=<value optimized out>, aChangeList=0x7ffffb501160, aMinChange=7) at nsFrameManager.cpp:1404 #27 0x00007fa3e5707cdf in nsFrameManager::ComputeStyleChangeFor (this=0x3eede08, aFrame=0x3db6130, aChangeList=0x7ffffb501160, aMinChange=<value optimized out>) at nsFrameManager.cpp:1470 #28 0x00007fa3e56ef7c6 in nsCSSFrameConstructor::RebuildAllStyleData (this=0x3dfa6f0, aExtraHint=7) at nsCSSFrameConstructor.cpp:13422 #29 0x00007fa3e5704c3e in DocumentViewerImpl::SetTextZoom (this=0x3c08f40, aTextZoom=1.18920708) at nsPresContext.h:475 #30 0x0000000000485755 in EphyBrowser::SetZoom ()
FWIW, in a debug build, it displays these lines: WARNING: Unable to test style tree integrity -- no content node: file nsCSSFrameConstructor.cpp, line 10043 WARNING: Unable to test style tree integrity -- no content node: file nsCSSFrameConstructor.cpp, line 10043 WARNING: Unable to test style tree integrity -- no content node: file nsCSSFrameConstructor.cpp, line 10043 WARNING: Unable to test style tree integrity -- no content node: file nsCSSFrameConstructor.cpp, line 10043 WARNING: Unable to test style tree integrity -- no content node: file nsCSSFrameConstructor.cpp, line 10043 WARNING: Unable to test style tree integrity -- no content node: file nsCSSFrameConstructor.cpp, line 10043 WARNING: Unable to test style tree integrity -- no content node: file nsCSSFrameConstructor.cpp, line 10043 WARNING: Unable to test style tree integrity -- no content node: file nsCSSFrameConstructor.cpp, line 10043 ###!!! ASSERTION: Unmatched begin/end?: '!mOldRuleTree', file nsStyleSet.cpp, line 107 WARNING: Unable to test style tree integrity -- no content node: file nsCSSFrameConstructor.cpp, line 10043 The stacktrace with a non-optimized debug build looks like: #0 0x00007fe163f420e7 in nsRuleNode::WalkRuleTree (this=0x4193a50, aSID=eStyleStruct_Background, aContext=0x41acec8, aRuleData=0x7fff7a5dd390, aSpecificData=0x7fff7a5dd2f0) at nsRuleNode.cpp:1388 #1 0x00007fe163f44810 in nsRuleNode::GetBackgroundData (this=0x4193a50, aContext=0x41acec8) at nsRuleNode.cpp:1211 #2 0x00007fe163f4490b in nsRuleNode::GetStyleBackground (this=0x4193a50, aContext=0x41acec8, aComputeData=1) at nsStyleStructList.h:79 #3 0x00007fe163f4bb5c in nsStyleContext::GetStyleBackground (this=0x41acec8) at nsStyleStructList.h:79 #4 0x00007fe163f4eed3 in nsStyleContext::CalcStyleDifference (this=0x3b07018, aOther=0x41acec8) at nsStyleContext.cpp:472 #5 0x00007fe163da6dcd in CaptureChange (aOldContext=0x3b07018, aNewContext=0x41acec8, aFrame=0x3b070d8, aContent=0x3a49cc0, aChangeList=0x7fff7a5e11e0, aMinChange=7, aChangeToAssume=0) at nsFrameManager.cpp:1058 #6 0x00007fe163da739f in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3b070d8, aParentContent=0x3a49c20, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1196 #7 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3b06f00, aParentContent=0x3a499f0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #8 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3aba088, aParentContent=0x3a49a90, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #9 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3aab338, aParentContent=0x3a49a90, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #10 0x00007fe163da6fdc in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3aab2b8, aParentContent=0x3a499a0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1125 #11 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3aab048, aParentContent=0x3a499a0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #12 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3aaaf30, aParentContent=0x3a37d40, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #13 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3ab9938, aParentContent=0x3a37c60, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #14 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3ab96a8, aParentContent=0x3a37c00, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #15 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3ab95a8, aParentContent=0x3a37c00, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #16 0x00007fe163da6fdc in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3ab9528, aParentContent=0x3a37ae0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1125 #17 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3b308c8, aParentContent=0x3a37a90, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #18 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3aee140, aParentContent=0x3a37a90, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #19 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3aee028, aParentContent=0x3a379a0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #20 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3aede40, aParentContent=0x2694700, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #21 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3aed368, aParentContent=0x26944b0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #22 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3a2c6b0, aParentContent=0x26944b0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #23 0x00007fe163da6fdc in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3a2c630, aParentContent=0x3a2e9a0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1125 #24 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3a2ba78, aParentContent=0x3a0bb30, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #25 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3a2b870, aParentContent=0x26bd580, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #26 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x3a294f8, aParentContent=0x26bd580, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #27 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x38b35a8, aParentContent=0x26bd580, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #28 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x38b37e8, aParentContent=0x0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #29 0x00007fe163da7dd3 in nsFrameManager::ReResolveStyleContext (this=0x268e3a8, aPresContext=0x26b0240, aFrame=0x38b34a0, aParentContent=0x0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1404 #30 0x00007fe163da802b in nsFrameManager::ComputeStyleChangeFor (this=0x268e3a8, aFrame=0x38b34a0, aChangeList=0x7fff7a5e11e0, aMinChange=7) at nsFrameManager.cpp:1470 #31 0x00007fe163d6b912 in nsCSSFrameConstructor::RebuildAllStyleData (this=0x268ea70, aExtraHint=7) at nsCSSFrameConstructor.cpp:13422 #32 0x00007fe163db7a46 in nsPresContext::RebuildAllStyleData (this=0x26b0240, aExtraHint=7) at nsPresContext.cpp:1409 #33 0x00007fe163da5442 in nsPresContext::SetTextZoom (this=0x26b0240, aZoom=1.18920708) at nsPresContext.h:475 #34 0x00007fe163d989d1 in DocumentViewerImpl::SetTextZoom (this=0x2a30520, aTextZoom=1.18920708) at nsDocumentViewer.cpp:2674 #35 0x0000000000485755 in EphyBrowser::SetZoom () Please tell me if you need more information
Keywords: crash
Also reproducible at http://erlang.org/
See also http://bugzilla.gnome.org/show_bug.cgi?id=525357 . Does this happen only on certain sites, or does it happen any time you increase text zoom? It seems surprising that it's specific to Epiphany/Galeon. Are you sure? (You were changing text zoom in the comparison cases, not full zoom?) Do you have a pointer to source code and build instructions for Epiphany/Galeon?
(In reply to comment #4) > were changing text zoom in the comparison cases, not full zoom?) Do you have a > pointer to source code and build instructions for Epiphany/Galeon? Specifically, to whichever one and version thereof you think is easiest to build/test.
It only happens on some sites, the url attached to this bug is very reliable to trigger the crash. I never got firefox to crash on it, despite switching between full and text zoom. Epiphany is using nsIMarkupContentViewer->SetTextZoom. At first I thought it could be related to the zoom factors Epiphany was using (Firefox uses 1.1, 1.2, 1.3... epiphany uses some strange values (some square roots, I think)) and changed the factors in epiphany, but that didn't change anything. I tried adding the relevant code portions to the TestGtkEmbed code, and while zooming works with my hack, it doesn't crash, which is quite disappointing. Anyways, it looks very much like stack or heap corruption of some sort from where i stand. I would suggest trying with the latest Epiphany from the 2.22 branch (2.22.3 iirc), and check lib/ephy-zoom.* and embed/mozilla/EphyBrowser.cpp.
Are there instructions somewhere for how to build Epiphany with a Gecko built normally from source and not installed?
Summary: Crashes when zooming on some pages under linux → Crashes when zooming on some pages under linux [@ nsRuleNode::GetStyleBackground]
I don't think there are. The best you can you is to have pkgconfig files pointing at your local version of libxul. You can use PKG_CONFIG_PATH to tell pkg-config where it should look for .pc files.
Did you manage to reproduce the issue ?
I can always reproduce the crash with the attached html file. If you remove the "#"-symbol in the background attribute the segfault will not occure. Referencing an image in the background attribute produces the same result as the "#".
David (dbaron), did you have a chance to reproduce the issue yet ?
No, I haven't had a chance to attempt getting the necessary builds set up. But I realized I might be able to debug on a Fedora 9 system by installing the appropriate packages + *-debuginfo RPMs.
Er, never mind... I forgot Fedora 9 is using Firefox 2.
So, to somebody who does have the setup to debug this: I'd say the first thing to look at would be this assertion: > ###!!! ASSERTION: Unmatched begin/end?: '!mOldRuleTree', file nsStyleSet.cpp, > line 107 Are there in fact unmatched calls to nsStyleSet::BeginReconstruct and nsStyleSet::EndReconstruct? (They should always be matched, and should never be nested... really the assertion text should be complaining about "unmatched or nested begin/end".) If so, what are the stacks that show how we're ending up with unmatched (the stack of the begin that has no end) or nested (the stack of both begins) calls?
Er... Fedora 9 uses Firefox 3.
Er, sorry, my Fedora box runs Fedora 8, not 9; Fedora 9 won't boot because of the way Dell set up the partition table back when I bought the machine a number of years ago (though all the previous Fedora releases worked fine), and since one of the main purposes of the machine is backups, I'm not about to repartition the disk.
(In reply to comment #15) > Are there in fact unmatched calls to nsStyleSet::BeginReconstruct and > nsStyleSet::EndReconstruct? (They should always be matched, and should never > be nested... really the assertion text should be complaining about "unmatched > or nested begin/end".) Bingo, it looks like they are either unmatched or nested. > If so, what are the stacks that show how we're ending up with unmatched (the > stack of the begin that has no end) or nested (the stack of both begins) calls? I’m attaching the stacks of the two consecutive begins.
Looks like epiphany is to blame here. It appears that during the reconstruct, gecko sends a MOZILLA_EMBED_LOAD_LOADING signal for one background image, and epiphany's handler calls ephy_base_embed_restore_zoom_level, which ends up doing a SetTextZoom because at the time ephy_base_embed_restore_zoom_level is called, GetTextZoom returns the old zoom level, which means ephy_base_embed_restore_zoom_level believes it's different from the new zoom level in which case it fires SetTextZoom. I guess EphyBrowser should keep the zoom level itself instead of relying on GetTextZoom...
Actually, on the contrary, GetTextZoom returns the new zoom level, while ephy_base_embed_restore_zoom_level compares to the old one...
Based on comment 19, sounds like this is invalid as a Gecko bug.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → INVALID
Crash Signature: [@ nsRuleNode::GetStyleBackground]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: