Closed Bug 574702 Opened 9 years ago Closed 9 years ago

Dragging the window through glass areas gets jumpy


(Toolkit :: XUL Widgets, defect)

Not set





(Reporter: Felipe, Assigned: Felipe)




(1 file)

Follow-up from bug 555081. Since we apply/unapply the binding that makes glass areas draggable depending on theme/tabs-on-top, the mouse listeners might end up being added more than once. When the mouse moves, two or more events are queued with the X/Y information. When the first event is processed, the window is correctly moved, and then the following events have wrong offset information and ends up moving the window again, with incorrect deltas.

  - with latest nightly, change the visibility of menubar/app-button back and forth at least twice.
  - try dragging the window through the glass area of the menu bar
Attachment #454062 - Flags: review?(
Component: Theme → XUL Widgets
Product: Firefox → Toolkit
QA Contact: theme → xul.widgets
Is this still working around the missing destructor? Would this be fixed by bug 83635, which is blocking beta2?
The patch that works around the missing destructor hasn't landed yet. I believe that bug 83635 will make it possible to get rid of the workaround, but it's hard to tell as there's no WIP fix there that I could try. I'll keep an eye on it and be sure to test that when 83635 is fixed.
In Reply to Filipe:

I see this when having a unmaximized window.  And after I open the browser I was testing just dragging the window around the screen from top to bottom, side to side.  I didn't check the CPU usage there.
Dale: yeah at first I thought it was just slowness that made the window jump, but I found the problem after you customize the menu off and on again. You said that it happened only at the menubar, right? Can you confirm that you have tweaked the settings to hide and show the menu bar, and then the problem appeared? Or can you see this in an entire fresh window that has never been customized?

(The patch here should fix this if that's the case)
Attachment #454062 - Flags: review?( → review+
I tried the tab bar area too, though trying to confirm the rest.
Assignee: nobody → felipc
Closed: 9 years ago
Flags: in-testsuite-
OS: Windows 7 → All
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a6
We should file a bug on using a destructor and getting rid of these workarounds.
You need to log in before you can comment on or make changes to this bug.