Closed
Bug 466353
Opened 16 years ago
Closed 16 years ago
simple website with -moz-box-shadow uses ~60 megs of ram, and maxes out CPU
Categories
(Core :: Web Painting, defect, P2)
Core
Web Painting
Tracking
()
VERIFIED
FIXED
mozilla1.9.1b3
People
(Reporter: cheetahmorph, Assigned: ventnor.bugzilla)
References
()
Details
(4 keywords, Whiteboard: [fixed by bug 458031])
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b2pre) Gecko/20081118 Minefield/3.1b2pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b2pre) Gecko/20081118 Minefield/3.1b2pre
When loading http://www-cs-students.stanford.edu/~amitp/gameprog.html in minefield, the browser becomes unresponsive. Upon loading the task manager, Firefox is revealed to be using up all of a processor core.
Reproducible: Always
Steps to Reproduce:
1. Go to http://www-cs-students.stanford.edu/~amitp/gameprog.html
2. let page load
3.Laaaaaaaaaaaaaaaaaaaaaaaaaaaaaag
Actual Results:
Browser becomes very slow until tab containing page is closed
Expected Results:
Browser is responsive and allows me to read the page
Comment 1•16 years ago
|
||
Confirmed on Windows XP, latest trunk.
Comment 2•16 years ago
|
||
Comment 3•16 years ago
|
||
Maybe caused by Bug 212633 ?
On my Vista computer I see no hang, but vertical scrolling is nearly impossible.
I see also a superfluous horizontal scrollbar, but I think this is a different bug.
Blocks: 212633
Status: UNCONFIRMED → NEW
Component: General → Style System (CSS)
Ever confirmed: true
Product: Firefox → Core
QA Contact: general → style-system
Version: unspecified → Trunk
Updated•16 years ago
|
Flags: blocking1.9.1?
Reporter | ||
Comment 4•16 years ago
|
||
Vertical scrolling was very slow for me, as was changing tabs.
Comment 5•16 years ago
|
||
Since entering this:
javascript:void(document.getElementsByTagName("head")[0].innerHTML += "<style type='text/css'>*{-moz-box-shadow: none!important}</style>")
in the URL bar makes things faster for me, blaming -moz-box-shadow seems reasonable. And the page does use -moz-box-shadow.
Does that make things faster for you as well?
Component: Style System (CSS) → Layout: View Rendering
QA Contact: style-system → layout.view-rendering
Summary: simple website uses ~60 megs of ram, and maxes out CPU → simple website with -moz-box-shadow uses ~60 megs of ram, and maxes out CPU
The problem is almost certainly just that we create a temporary surface for blurring the whole box irrespective of the dirty area. We should optimize box-shadow drawing so that blur computation is restricted to the dirty area. It's a little tricky but not all that hard.
Reporter | ||
Comment 7•16 years ago
|
||
Yup, that made it nice and zippy.
The work in 458031 should fix this.
Depends on: 458031
Flags: blocking1.9.1? → blocking1.9.1+
Priority: -- → P2
Assignee: nobody → ventnor.bugzilla
Whiteboard: [depends on 458031]
Fixed by checkin for bug 458031.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [depends on 458031]
Comment 10•16 years ago
|
||
This was also an issue on OS X. Now I can verify that the fix on bug 458031 solves the hang. Tested with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20081203 Minefield/3.2a1pre
For now setting the TM to 1.9.2a1. We should adjust when the patch has been landed on 1.9.1 branch.
Status: RESOLVED → VERIFIED
OS: Windows XP → All
Hardware: PC → All
Target Milestone: --- → mozilla1.9.2a1
Comment 11•16 years ago
|
||
Bug 458031 has been checked in to 1.9.1 branch already. marking fixed1.9.1 here too and adjusting TM.
Keywords: fixed1.9.1
Whiteboard: [fixed by bug 458031]
Target Milestone: mozilla1.9.2a1 → mozilla1.9.1b3
Comment 12•16 years ago
|
||
Sorry for my last comment. We shouldn't change TM. Branch 1.9.1 and trunk are different nowadays. Setting the keyword is enough.
Target Milestone: mozilla1.9.1b3 → mozilla1.9.2a1
Comment 13•16 years ago
|
||
Verified on 1.9.1 with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre) Gecko/20081221 Shiretoko/3.1b3pre ID:20081221020430
Keywords: fixed1.9.1 → verified1.9.1
Updated•16 years ago
|
Target Milestone: mozilla1.9.2a1 → mozilla1.9.1b3
Updated•6 years ago
|
Component: Layout: View Rendering → Layout: Web Painting
You need to log in
before you can comment on or make changes to this bug.
Description
•