Merge lp://qastaging/~compiz-team/compiz-core/compiz-core.fix_853734 into lp://qastaging/compiz-core/0.9.5

Proposed by Sam Spilsbury
Status: Merged
Merged at revision: 2827
Proposed branch: lp://qastaging/~compiz-team/compiz-core/compiz-core.fix_853734
Merge into: lp://qastaging/compiz-core/0.9.5
Diff against target: 501 lines (+204/-53) (has conflicts)
3 files modified
src/privatewindow.h (+1/-0)
src/window.cpp (+195/-45)
src/windowgeometry.cpp (+8/-8)
Text conflict in src/window.cpp
To merge this branch: bzr merge lp://qastaging/~compiz-team/compiz-core/compiz-core.fix_853734
Reviewer Review Type Date Requested Status
Compiz Maintainers Pending
Review via email: mp+76020@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2011-09-19.

Description of the change

  Ensure that we update frame regions correctly by doing the following:
   * Only update frame regions when we get an event that something changed
     server side (eg don't immediately update the region once we issue
     a ConfigureWindow request)
   * Keep track of the changes in the input extents
   * Ensure that a synthetic ConfigureNotify is sent in cases where the resized
     frame window will exactly match its last size such that the client is
     positioned correctly inside the frame

To post a comment you must log in.
Revision history for this message
Mirco Müller (macslow) wrote : Posted in a previous version of this proposal

I'm not able to compile compiz here atm. But what I suggest is to initialize all declared variables.

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote : Posted in a previous version of this proposal

I can confirm that maximize->minimize works fine here. And generally everything seems to work fine. Haven't had time to do code review.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches