Mac OS X: Windows dragging is SLOW

VERIFIED FIXED in mozilla0.9.3

Status

VERIFIED FIXED
18 years ago
10 years ago

People

(Reporter: zs98, Assigned: mikepinkerton)

Tracking

({perf})

Trunk
mozilla0.9.3
PowerPC
Mac OS X

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

18 years ago
Dragging Mozilla windows around is unbearably slow.  This bug first appeared in
build 6/5/01 of Fizzilla.  Previously, dragging was in line with the rest of the OS.
Keywords: qawanted
Keywords: perf

Comment 1

18 years ago
Checked this out with QuartzDebug (application in the OSX dev tools). In the 6/5
build "live window resizing" is functional for the first time by default. The
problem appears to be that other carbon apps only use quartz to draw window
contents when resizing windows, make dropshadows, etc. but window dragging is
drawn by the GPU, not the CPU. In 6/5 quartz updates the entire window when
dragging, thus the massive slowdown.

fire up quartzdebug /MacOSX/Developer/Applications/QuartzDebug.app and compare
Fizz to other carbon apps.

Comment 2

18 years ago
But other Carbon apps have native interfaces. I wouldn't be surprised if this
was a side-effect of XUL.

Mac OS X is so slow that I'm not sure this will be a high-priority problem
anytime soon. But it is a problem (compared to dragging an IE window of the same
size around) so I'm going to confirm this.
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 3

18 years ago
So this is a regression from the live window resizing? Over to pink
Assignee: blakeross → pinkerton
(Assignee)

Comment 4

18 years ago
are we talking about moving windows or resizing them? we don't even get notified
when the windows are being moved, so i'm not quite sure how the resize code
could be getting involved.
(Assignee)

Comment 5

18 years ago
i have a patch to this. it appears that i wasn't checking the appropriate
parameter so the live-resize code was kicking in even when the window was moving.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.3
(Assignee)

Comment 6

18 years ago
Created attachment 37970 [details] [diff] [review]
[patch] only invalidate on resize, not move
(Assignee)

Comment 7

18 years ago
Created attachment 37972 [details] [diff] [review]
[patch #2] the real patch, ignore the other one
(Assignee)

Comment 8

18 years ago
r=pchen

Comment 9

18 years ago
I think only invalidating the area that differs between the two sizes is not 
enough. Resizing the window can cause the page to re-layout in a different way, 
necessitating an entire-window invalidate.
(Assignee)

Comment 10

18 years ago
Created attachment 37975 [details] [diff] [review]
[patch #3] go back to invalidating the whole window

Comment 12

18 years ago
a= asa@mozilla.org for checkin to the trunk.
(on behalf of drivers)
(Assignee)

Comment 13

18 years ago
fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 14

18 years ago
Marking verified in the June 13th build (2001061314)
Status: RESOLVED → VERIFIED

Updated

18 years ago
Keywords: qawanted
Product: Core → Mozilla Application Suite

Updated

10 years ago
Component: XP Apps: GUI Features → UI Design
You need to log in before you can comment on or make changes to this bug.