Compiz Grid Plugin

Merge lp://staging/~compiz-team/compiz-grid-plugin/compiz-grid-plugin.fix_860257 into lp://staging/compiz-grid-plugin

  1. compiz-grid-plugin.fix_860257
  2. Merge into 0.9.5
Proposed by Sam Spilsbury
Status: Superseded
Proposed branch: lp://staging/~compiz-team/compiz-grid-plugin/compiz-grid-plugin.fix_860257
Merge into: lp://staging/compiz-grid-plugin
Diff against target: 112 lines (+34/-7)
2 files modified
src/grid.cpp (+31/-7)
src/grid.h (+3/-0)
To merge this branch: bzr merge lp://staging/~compiz-team/compiz-grid-plugin/compiz-grid-plugin.fix_860257
Reviewer Review Type Date Requested Status
Jason Smith (community) Approve
Review via email: mp+77093@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2011-09-29.

Description of the change

Fixes Bug 860257

With the new core changes attempting to move a window that is maximizing will result in a race condition resulting in a deadlock.

To post a comment you must log in.
lp://staging/~compiz-team/compiz-grid-plugin/compiz-grid-plugin.fix_860257 updated
99. By Sam Spilsbury

Move the window back to the desired size, explicitly moving it back by whatever we moved
it to will end up with us fighting ourselves in case core was configuring that window
while it was being moved, because we will get the moveNotify a little while after
we moved the window.

100. By Sam Spilsbury

Detect viewport changes and don't move windows back then.

101. By Sam Spilsbury

Merge trunk

Revision history for this message
Jason Smith (jassmith) :
review: Approve

Unmerged revisions

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

to all changes: