Bug #1251777 “[regression] Semi-maximizing and then restoring th...” : Bugs : Unity

Christopher Townsend (townsend) on 2013-11-15

summary:

- Semi-maximizing andthen restoring the window does not return it to it's
+ Semi-maximizing and then restoring the window does not return it to it's
original place

Christopher Townsend (townsend) on 2013-11-16

summary:

- Semi-maximizing and then restoring the window does not return it to it's
- original place
+ [regression] Semi-maximizing and then restoring the window does not
+ return it to it's original place

Christopher Townsend (townsend) on 2013-11-16

description:

updated

Revision history for this message

MC Return (mc-return) wrote on 2013-11-16: Re: [regression] Semi-maximizing and then restoring the window does not return it to it's original place

#1

I cannot reproduce this one, but I am using Emerald.

When I click the restore button on the semi-maximized (left/right) grid window with the left mouse button it will maximize, using the middle one does nothing here, other than changing the button...

Revision history for this message

Christopher Townsend (townsend) wrote on 2013-11-19:

#2

I'm pretty sure this is caused by this change in Unity: http://bazaar.launchpad.net/~unity-team/unity/trunk/revision/3509

Changing this to Unity since this is where the bug originates.

affects:	compiz → unity
no longer affects:	compiz/0.9.10
affects:	compiz (Ubuntu) → unity (Ubuntu)
Changed in unity:
importance:	Undecided → Medium
Changed in unity (Ubuntu):
importance:	Undecided → Medium
Changed in unity:
assignee:	nobody → Christopher Townsend (townsend)
Changed in unity (Ubuntu):
assignee:	nobody → Christopher Townsend (townsend)
Changed in unity:
status:	New → In Progress
Changed in unity (Ubuntu):
status:	New → In Progress

Christopher Townsend (townsend) on 2013-11-20

summary:

[regression] Semi-maximizing and then restoring the window does not
- return it to it's original place
+ return it to it's original place using Ctrl-Super-Left/Right

Christopher Townsend (townsend) on 2013-12-04

Changed in compiz:
status:	New → In Progress
importance:	Undecided → Medium
assignee:	nobody → Christopher Townsend (townsend)
Changed in compiz (Ubuntu):
status:	New → In Progress
importance:	Undecided → Medium
assignee:	nobody → Christopher Townsend (townsend)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-12-06:

#3

Fix committed into lp:compiz at revision None, scheduled for release in compiz, milestone 0.9.11.0

Changed in compiz:
status:	In Progress → Fix Committed

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2013-12-06:

#4

Fix committed into lp:unity at revision None, scheduled for release in unity, milestone 7.2.0

Changed in unity:
status:	In Progress → Fix Committed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2014-02-17:

#6

Download full text (18.5 KiB)

This bug was fixed in the package unity - 7.1.2+14.04.20140214.1-0ubuntu1

---------------
unity (7.1.2+14.04.20140214.1-0ubuntu1) trusty; urgency=low

  [ Sebastien Bacher ]
  * use unity-control-center by default.
  * lists keybinding in unity-control-center. (LP: #1271710)

  [ Brandon Schaefer ]
  * Bump to new libnux from this branch:
    https://code.launchpad.net/~brandontschaefer/nux/xim-preedit-
    support.
  * Adds Super+L to lock the screen, while keeping the older shortcut
    around in g-s-d (Ctrl+Alt+L). (LP: #830709)
  * Do not open the dash/hud on a monitor with a top most window that is
    fullscreen. (LP: #1267210)
  * Implement an EMConveter. This way with default settings such that
    DPI = 96.0f, and font_size = system font size. We can get the
    correct EM value for any pixel size. Once we have the correct EM
    value for any pixel size, the DPI value can be adjusted to the
    current logical one. From here, you can now get the correct pixel
    size based from of the EM value for the logical DPI of the screen.
  * Refactor EMConverter API. Now all thats needed is int
    ConvertPixels(int pixel); This will calculate the correct pixel size
    based on the DPI and font size.
  * Testing that the ibus anthy tests could possibly be causing strange
    issues on the nvidia machine. So skipping them to test if tihs is
    the source of the error.
  * Add Pt to Px function to em converter.
  * Move EMConverter over to unity settings.
  * Add multi monitor support for EMConverter in unity settings. Now you
    can grab a specific converter per monitor.
  * Simple RawPixel class. It adds 2 define literals, ex: 10_em,
    10.0_em. From there it turns them into raw pixels. RawPixels have CP
    (CovertPixel) function which takes in an EMConverter that allows you
    to use a converter specific to a monitor to convert the raw pixel to
    the correct value.

  [ Marco Trevisan (Treviño) ]
  * Don't re-present all of our windows on every frame. Only do that if
    damage intersects it. Use the new APIs exposed by compiz and nux to
    intelligently determine which windows need to be presented per-frame
    and only register damage for those windows. This fixes two things:
    1. BaseWindows being redrawn from scratch every time damage was
    registered over them. That was incorrect and should only be done in
    the case of background blurs. 2. BaseWindows being drawn to the
    screen on every frame, regardless of whether or not they needed to
    be. Now they will only be drawn if some damage intersects beneath
    them. Note that unity will expand the damage region to accomadate
    the base window since nux does not support geometry clipping. So if
    there is a partial intersection of the launcher for example, the
    area of the screen which contains the launcher will be re-painted
    (but the launcher itself won't be redrawn, just its texture) (LP:
    #1080947). (LP: #1080947)
  * Convert compiz regions / rects to nux::Geometry's and back easily.
  * UnityScreen: remove the useless and expensive gl{Push,Pop}Attrib
    calls For some reasons this code was copied by the opengl plugin as
    a workaround to fix the s...

This bug was fixed in the package unity - 7.1.2+14.04.20140214.1-0ubuntu1

---------------
unity (7.1.2+14.04.20140214.1-0ubuntu1) trusty; urgency=low

[ Sebastien Bacher ]
  * use unity-control-center by default.
  * lists keybinding in unity-control-center. (LP: #1271710)

[ Brandon Schaefer ]
  * Bump to new libnux from this branch:
    https://code.launchpad.net/~brandontschaefer/nux/xim-preedit-
    support.
  * Adds Super+L to lock the screen, while keeping the older shortcut
    around in g-s-d (Ctrl+Alt+L). (LP: #830709)
  * Do not open the dash/hud on a monitor with a top most window that is
    fullscreen. (LP: #1267210)
  * Implement an EMConveter. This way with default settings such that
    DPI = 96.0f, and font_size = system font size. We can get the
    correct EM value for any pixel size. Once we have the correct EM
    value for any pixel size, the DPI value can be adjusted to the
    current logical one. From here, you can now get the correct pixel
    size based from of the EM value for the logical DPI of the screen.
  * Refactor EMConverter API. Now all thats needed is int
    ConvertPixels(int pixel); This will calculate the correct pixel size
    based on the DPI and font size.
  * Testing that the ibus anthy tests could possibly be causing strange
    issues on the nvidia machine. So skipping them to test if tihs is
    the source of the error.
  * Add Pt to Px function to em converter.
  * Move EMConverter over to unity settings.
  * Add multi monitor support for EMConverter in unity settings. Now you
    can grab a specific converter per monitor.
  * Simple RawPixel class. It adds 2 define literals, ex: 10_em,
    10.0_em. From there it turns them into raw pixels. RawPixels have CP
    (CovertPixel) function which takes in an EMConverter that allows you
    to use a converter specific to a monitor to convert the raw pixel to
    the correct value.

[ Marco Trevisan (Treviño) ]
  * Don't re-present all of our windows on every frame. Only do that if
    damage intersects it. Use the new APIs exposed by compiz and nux to
    intelligently determine which windows need to be presented per-frame
    and only register damage for those windows. This fixes two things:
    1. BaseWindows being redrawn from scratch every time damage was
    registered over them. That was incorrect and should only be done in
    the case of background blurs. 2. BaseWindows being drawn to the
    screen on every frame, regardless of whether or not they needed to
    be. Now they will only be drawn if some damage intersects beneath
    them. Note that unity will expand the damage region to accomadate
    the base window since nux does not support geometry clipping. So if
    there is a partial intersection of the launcher for example, the
    area of the screen which contains the launcher will be re-painted
    (but the launcher itself won't be redrawn, just its texture) (LP:
    #1080947). (LP: #1080947)
  * Convert compiz regions / rects to nux::Geometry's and back easily.
  * UnityScreen: remove the useless and expensive gl{Push,Pop}Attrib
    calls For some reasons this code was copied by the opengl plugin as
    a workaround to fix the state of our screen after that nux has
    drawn. Actually this is not needed, the only thing we really need to
    do is to fix the current Viewport, because nux seems to leave it in
    a bad state which would lead to flickering menus, fullscreen
    windows, tooltip and missing windows thumbnails in switcher. Thanks
    to Sam Spilsbury for his precious support. (LP: #1251275)
  * Unity: always prefer passing [this] to lambdas than [&].
  * Introspectable: use IntrospectionData class for collecting data from
    children Now each introspectable object is called with an
    IntrospectionData parameter and calling one of its methods it's the
    only way to fill introspection data into unity. As bonus point,
    remove all the unneeded UnityCore/Variant.cpp inclusions. (LP:
    #1227131)
  * DebugDBusInterface: add local::xpathselect::NodeSelector to use the
    dloaded lib.
  * BackgroundEffectHelper: Specify the required blur area before
    drawing so selectively copy it at paint time This means that we
    don't have to waste fragment bandwidth copying the entire backbuffer
    when we could just do parts of it. Now BackgroundEffectHelper
    listens to windows and views geometry changes and updates a list of
    blurred regions that might be copied to the backup texture at every
    repaint that affects them. This avoids to copy large regions
    (especially when using big resolutions or multiple monitors), but
    only the ones we really need to blur.
  * SwitcherView: define a custom GeometryGetterFunc and notify helper
    on changes Thanks to this the switcher won't make
    BackgroundEffectHelper to create a blurred area as big as the
    current monitor (with just a small padding), but an area big enough
    to draw its background. This get updated automagically when the
    switcher view changes its geometry... Also, use nux::AnimateValue
    for switcher animations, making compiz to orchestrate it and only
    redraw the view if an icon changed since the last progress
    iteration. We don't want the switcher to be drawn unless its
    geometry or an icon changes. Also we don't want to redraw the
    switcher multiple times if only a not-animated parameter of an icon
    (such as a pip or the selection glow) has changed. As bonus, reduce
    lots of list copies.
  * {Dash,Hud}Controller: unset the keyfocus area only if one of their
    children has the focus Otherwise we could steal the focus to another
    view (such as the Alt+Tab switcher). (LP: #1252760)
  * IntrospectionData: use forward declaration for nux classes, add
    support to glib::Variant We don't need to repeat the inclusion
    everywhere we want debug introspection. This allows also to add
    glib::Variant adder without the need of any other cast.
  * decorations: added new unity-decorations fully gtk-css-themed inside
    unity. (LP: #691817, #827537)
  * PanelMenuView: show a dropdown menu of indicators, if there's no
    enough horizontal space PanelService: add new dbus method
    ShowEntriesDropdown that when called puts the requested indicator
    entires into a DropDown menu before showing them, also threat
    dropdown menus as special entries when the their geometry has been
    synced. UnityCore: add add ShowEntriesDropdown virtual method as
    well and implement in DBusIndicators, to call the relative method in
    PanelService. Add parents to indicator::Entry, now an entry can have
    one or more parents (that's mostly depending on how many monitors we
    have), and if parented, it notifies active, show_now and geometry
    status to them. API cleanups included, SO-name bumped. PanelView:
    add simpler PreLayoutManagement function, to avoid nux to do too
    many computations that will cause the Menu to relayout too much with
    no need. Added PanelIndicatorEntryDropdownView as new subtype of
    PanelIndicatorEntryView that will show a go-down icon on the
    PanelMenuView that when clicked will show a dropdown menu of
    indicators. This will happen only if there's not enough room to see
    them inline. PanelIndicatorsView: many cleanups, and add
    EnableDropdownMenu that creates a new
    PanelIndicatorEntryDropdownView and uses it to place the entries
    that don't fit the available space, until we don't get some more.
    Added and reworked a bunch of unit tests. (LP: #1274554)
  * DecorationsInputMixer: pass Timestamp to events This might be needed
    to perform some actions, and using CompScreen getCurrentTime() is
    way much more expensive than I thought.
  * MenuManager: add a wrapper for Indicators and KeyGrabber that
    registers mnemonics It also emits signals with accumulators that
    will just stop the emission when a client returns true, and holds
    menu general settings.
  * LauncherDragWindow: set the drag window background to transparent or
    it will be white by default This was causing a quick flickering of
    the icon as soon as it was drawn on the first place. (LP: #1279904)
  * {DecorationsGrabEdge, PanelTitlebarGrabAreaView}: show the
    WindowAction menu on right-click (LP: #1098419)
  * PanelMenuView: we don't need to escape the panel title anymore New
    decorations just use pango text. (LP: #1280077)

[ Chris Townsend ]
  * Don't re-present all of our windows on every frame. Only do that if
    damage intersects it. Use the new APIs exposed by compiz and nux to
    intelligently determine which windows need to be presented per-frame
    and only register damage for those windows. This fixes two things:
    1. BaseWindows being redrawn from scratch every time damage was
    registered over them. That was incorrect and should only be done in
    the case of background blurs. 2. BaseWindows being drawn to the
    screen on every frame, regardless of whether or not they needed to
    be. Now they will only be drawn if some damage intersects beneath
    them. Note that unity will expand the damage region to accomadate
    the base window since nux does not support geometry clipping. So if
    there is a partial intersection of the launcher for example, the
    area of the screen which contains the launcher will be re-painted
    (but the launcher itself won't be redrawn, just its texture) (LP:
    #1080947). (LP: #1080947)
  * In low graphics mode, only draw the Launcher background as needed
    instead of all of the time, even if the Launcher is hidden. (LP:
    #1179210)
  * Bump the Unity glib build dependency version.
  * Use IsVisibleOnMonitor() instead of IsVisible() for checking to
    close the Hud when clicking the Hud icon. This is due to the icon's
    quirk only being set for the current monitor. (LP: #1255241)
  * Add some introspection vector variables to account for per-monitor
    icon quirks to fix some failing Hud AP tests. (LP: #1244704)
  * Revert most of lp:unity revno. 3509 as the Compiz Grid plugin is a
    better place to handle the Ctrl-Super-Left/Right shortcuts. Expanded
    autopilot test to check for correct window placement when semi-
    maximizing and restoring. (LP: #1251777)
  * Re-enable Quicklist and Tooltip active blurring since it was removed
    due to the performance merge. (LP: #1253740)
  * Fix compile error due to deprecated gtk_widget_get_state() function.
    (LP: #1173219)
  * Fix a bunch of failing AP tests due to changes in Unity. (LP:
    #1244704)
  * Due to a change in xpathselect1.4, this fixes a sorting issue of
    Switcher icons for AP tests. (LP: #1244704)
  * Always set the interval timer to 0 when starting a new urgent
    animation timer. (LP: #1263775)
  * When hiding the Dash & Hud views, zero out the geometry and update
    the blur background size. This fixes the issue where the blur
    background wouldn't be updated when using static blur on subsequent
    opens of the Dash and Hud. (LP: #834777)
  * If the Dash or Hud is opened when entering Show Desktop, ensure they
    are closed. Added some Autopilot tests for this condition. (LP:
    #961890)
  * Add mesa-utils dependency to the unity-autopilot package since some
    of the autopilot tests use glxinfo to gather information. (LP:
    #1267902)
  * Fixes issue where a user places the mouse over a Launcher icon that
    has one unfocused window and then scrolls with the scroll wheel
    would focus that application when it should not according to Design.
    (LP: #1263786)
  * When using the mouse scrollwheel on inactive Launcher icons with
    unfocused windows, the unfocused window should not be focused. (LP:
    #1267888)
  * Add missing Python module declarations for multi-monitor setups.
    Fixes some AP test failures.
  * Fixes most Launcher based Autopilot tests due to the new object list
    sorting in xpathselect1.4.
  * Add back in vertical/horizontal maximized introspection objects
    since some Autopilot tests need them.
  * Removed the ZeitgeistUtils.h as this is no longer needed since
    Zeitgeist in main is now fixed.
  * Fix more Autopilot tests: Base icon center introspection on the
    current monitor instead of the hardcoded first monitor. Fix
    move_mouse_to_icon AP method to work correctly with multi-monitor.
    Fix some scroll tests for multi-monitor. Skip the Launcher only on
    primary monitor/test Launcher functionality on the second monitor
    tests since the tests cannot run in this scenario.
  * More Unity Autopilot Launcher test fixes: When initializing the
    Launcher tests, check whether the Launcher is available on the
    monitor being tested. If not, skip the test. Also, move the mouse to
    the monitor being tested before running the test to make sure the
    correct Launcher is being tested. Skip some Launcher reveal tests
    since the test is no longer valid due to how Autopilot moves the
    mouse now. Fix drag_icon_to_position method. Fix
    mouse_reveal_launcher method.
  * Fix more AP tests: Fix workspace emulator to get viewport info for
    more than the first monitor. Fix some Hud, Dash, and Switcher tests
    for multi-monitor.
  * Add a method to tell the Hud icon when there are multi-monitors and
    only a Launcher on the primary monitor. This will make sure the
    active and visible states for the icon are correct and allow the AP
    tests to pass. (LP: #1274985)
  * Fixes some Autopilot Panel test failures: Emit the signal to call
    OnIndicatorViewUpdated() when the active window position has
    changed. Account for an overlay (Dash/Hud) to be opened when testing
    if the window buttons should be shown in the Panel. This was removed
    some revisions ago and it looks like it shouldn't have been. Fix up
    a Panel test since it was failing in a part that didn't need to be
    ran anyways. (LP: #1277265)
  * Fix a couple of test_panel AP tests: The overlay window buttons did
    not take multi-monitor into account, so now account for that. Fix
    the AP test themselves to actually use the correct introspection
    object.
  * Fix a failing Dash AP test due to too many results being returned
    which caused the position of the Dash icon to be off the screen.
  * Fix where the mouse_disabled_ variable was unintialized which would
    cause the Switcher to not work correctly with the mouse and arrow
    keys. Fix the mouse filter list to account for only motion and
    button click/release so arrow key navigation will still work in the
    Switcher when mouse interaction is disabled. (LP: #1279499)
  * Fix Autopilot test issue where Show Desktop is somehow invoked and
    the recovery from this does not properly get the state back to Show
    Desktop off.

[ William Hua ]
  * Replace deprecated g_settings_list_schemas with
    g_settings_schema_source_list_schemas.
  * Launcher: Fix GVariant type error in Hud UBus message.
  * Implement the GNOME key grabber interface so that Compiz and gnome-
    settings-daemon no longer have to fight for key grabs. Also, fix the
    global menu bar mnemonics. (LP: #1113008, LP: #1206582, LP:
    #1226962). (LP: #1113008, #1226962, #1206582)

[ Stephen M. Webb ]
  * add multi-mointor support to a couple of tests where it got misssed.

[ George Karavasilev ]
  * LauncherIcon: always try to use themed icons first, before falling
    back to UnityTheme.

[ CI bot ]
  * Flush trunk to Ubuntu

[ Eleni Maria Stea ]
  * the branch fixes the following compile errors: 1.
    SetReferenceFramebuffer function gets 3 parameters
    (Nux/WindowCompositor.h) 2. RenderInterfaceFromForeignCmd gets const
    reference not pointer as argument (Nux/WindowThread.h).
  * adds virtual destructors to classes with virtual methods to prevent
    memory leaks - ABI break.
  * new gsetting in unityshell.xml.in: fonts-scale-factor.
  * Configuration Parser class for custom gsettings.
  * function that gets the monitor name from the GDKScreen in UScreen.

[ Andrea Azzarone ]
  * Don't re-present all of our windows on every frame. Only do that if
    damage intersects it. Use the new APIs exposed by compiz and nux to
    intelligently determine which windows need to be presented per-frame
    and only register damage for those windows. This fixes two things:
    1. BaseWindows being redrawn from scratch every time damage was
    registered over them. That was incorrect and should only be done in
    the case of background blurs. 2. BaseWindows being drawn to the
    screen on every frame, regardless of whether or not they needed to
    be. Now they will only be drawn if some damage intersects beneath
    them. Note that unity will expand the damage region to accomadate
    the base window since nux does not support geometry clipping. So if
    there is a partial intersection of the launcher for example, the
    area of the screen which contains the launcher will be re-painted
    (but the launcher itself won't be redrawn, just its texture) (LP:
    #1080947). (LP: #1080947)
  * Alt-Tab, shutdown dialog and shortcut overlay should not be
    transparent in Low GFX Mode. (LP: #1170508)

[ Sam Spilsbury ]
  * Don't re-present all of our windows on every frame. Only do that if
    damage intersects it. Use the new APIs exposed by compiz and nux to
    intelligently determine which windows need to be presented per-frame
    and only register damage for those windows. This fixes two things:
    1. BaseWindows being redrawn from scratch every time damage was
    registered over them. That was incorrect and should only be done in
    the case of background blurs. 2. BaseWindows being drawn to the
    screen on every frame, regardless of whether or not they needed to
    be. Now they will only be drawn if some damage intersects beneath
    them. Note that unity will expand the damage region to accomadate
    the base window since nux does not support geometry clipping. So if
    there is a partial intersection of the launcher for example, the
    area of the screen which contains the launcher will be re-painted
    (but the launcher itself won't be redrawn, just its texture) (LP:
    #1080947). (LP: #1080947)
  * BackgroundEffectHelper: Specify the required blur area before
    drawing so selectively copy it at paint time This means that we
    don't have to waste fragment bandwidth copying the entire backbuffer
    when we could just do parts of it. Now BackgroundEffectHelper
    listens to windows and views geometry changes and updates a list of
    blurred regions that might be copied to the backup texture at every
    repaint that affects them. This avoids to copy large regions
    (especially when using big resolutions or multiple monitors), but
    only the ones we really need to blur.
 -- Ubuntu daily release <ps-jenkins@lists.canonical.com>   Fri, 14 Feb 2014 23:06:09 +0000

Changed in unity (Ubuntu):
status:	In Progress → Fix Released

Revision history for this message

Launchpad Janitor (janitor) wrote on 2014-02-17:

#5

Download full text (44.6 KiB)

This bug was fixed in the package compiz - 1:0.9.11+14.04.20140214-0ubuntu1

---------------
compiz (1:0.9.11+14.04.20140214-0ubuntu1) trusty; urgency=low

[ Timo Jyrinki ]
* Bump version to 0.9.11

  [ Marco Trevisan (Treviño) ]
  * debian/00_remove_decor_in_unity_session.py: add migration script
    to avoid to load the decor plugin on compiz startup when using unity.
  * debian/compiz-gnome.gconf-defaults: disable decor plugin on unity session

  [ Sebastien Bacher ]
  * debian/compiz-gnome.links: lists keybinding in unity-control-center
  * typo fix in the previous commit. (LP: #1271710)

  [ Iven Hsu ]
  * Opacify: Only dim the windows above the active window.(LP:
    #1189374). (LP: #1189374)
  * KWD: Fix compile errors with KDE 4.11. The KWin developers made
    kdecorationbridge.h private. See:
    http://lists.freedesktop.org/archives/compiz/2013-March/003479.html
    (LP: #1193792). (LP: #1193792)

  [ Nikolay Martynov ]
  * When static switcher is enabled and has an option to show
    application icon turned on the icons are expected to be ~1/3 of a
    thumbnail (48px). Instead they are displayed in 512px size and
    completely cover everything. This change addresses this issue. See
    LP #1173914. (LP: #1173914, #1186426)

  [ BryanFRitt ]
  * Fixed the non-working Annotate 'Clear' Button. Moved this option's
    CCSM position upwards to keep the button shortcuts together. (LP:
    #1202907). (LP: #1202907)

[ CI bot ]
* Flush trunk to Ubuntu

  [ William Hua ]
  * Replace <Primary> with <Control> in CCSM. Fixes
    https://bugs.launchpad.net/compiz/+bug/1069121. (LP: #1069121)
  * Tweak support of key bindings of the form
    '<Modifier>Modifier_KeySym'. We tweak a bit the behaviour of key
    bindings such as '<Control>Shift_L' and '<Alt>Alt_R'. 1. We ignore
    the order of key pressing and releasing, so tapping
    '<Shift>Control_L' is the same as '<Control>Shift_L'. 2. We properly
    handle the double modifiers case, for example '<Control>Control_R'.
    3. We also parse key bindings with '<Primary>' being equivalent to
    '<Control>'.
  * Fix GSettings tests with extra slash.
  * Add an interface for plugins to provide non-option key actions that
    can be triggered.

  [ Eleni Maria Stea ]
  * It fixes the bug #1245886. In DecorScreen::handleEvent compiz
    shouldn't try to handle any events if there's no active window yet.
    (LP: #1245886)
  * Compiz static analysis shows that some compiz classes have virtual
    methods but not virtual destructors. Added the virtual destructors
    to get rid of warnings and potential memory leaks.
  * fixed cmake syntax errors.
  * CMake considered compiz a C++ project and couldn't find some
    dependencies like pthreads. Defined compiz as a C, CXX project to
    fix the issue.

  [ Povilas Kanapickas ]
  * Opacify: Properly initialize window drawing for new windows in
    Opacify plugin. (LP: #787814, part 2). (LP: #787814)
  * Opacify: Fix damage generation in the Opacify plugin. When setting
    opacity to some value, non-opacified windows need to be damaged
    regardless of opacity, whereas opacified windows need to be damaged
    only if opacity changes. Remove u...

This bug was fixed in the package compiz - 1:0.9.11+14.04.20140214-0ubuntu1

---------------
compiz (1:0.9.11+14.04.20140214-0ubuntu1) trusty; urgency=low

[ Timo Jyrinki ]
  * Bump version to 0.9.11

[ Marco Trevisan (Treviño) ]
  * debian/00_remove_decor_in_unity_session.py: add migration script
    to avoid to load the decor plugin on compiz startup when using unity.
  * debian/compiz-gnome.gconf-defaults: disable decor plugin on unity session

[ Sebastien Bacher ]
  * debian/compiz-gnome.links: lists keybinding in unity-control-center
  * typo fix in the previous commit. (LP: #1271710)

[ Iven Hsu ]
  * Opacify: Only dim the windows above the active window.(LP:
    #1189374). (LP: #1189374)
  * KWD: Fix compile errors with KDE 4.11. The KWin developers made
    kdecorationbridge.h private. See:
    http://lists.freedesktop.org/archives/compiz/2013-March/003479.html
    (LP: #1193792). (LP: #1193792)

[ Nikolay Martynov ]
  * When static switcher is enabled and has an option to show
    application icon turned on the icons are expected to be ~1/3 of a
    thumbnail (48px). Instead they are displayed in 512px size and
    completely cover everything. This change addresses this issue. See
    LP #1173914. (LP: #1173914, #1186426)

[ BryanFRitt ]
  * Fixed the non-working Annotate 'Clear' Button. Moved this option's
    CCSM position upwards to keep the button shortcuts together. (LP:
    #1202907). (LP: #1202907)

[ CI bot ]
  * Flush trunk to Ubuntu

[ William Hua ]
  * Replace <Primary> with <Control> in CCSM. Fixes
    https://bugs.launchpad.net/compiz/+bug/1069121. (LP: #1069121)
  * Tweak support of key bindings of the form
    '<Modifier>Modifier_KeySym'. We tweak a bit the behaviour of key
    bindings such as '<Control>Shift_L' and '<Alt>Alt_R'. 1. We ignore
    the order of key pressing and releasing, so tapping
    '<Shift>Control_L' is the same as '<Control>Shift_L'. 2. We properly
    handle the double modifiers case, for example '<Control>Control_R'.
    3. We also parse key bindings with '<Primary>' being equivalent to
    '<Control>'.
  * Fix GSettings tests with extra slash.
  * Add an interface for plugins to provide non-option key actions that
    can be triggered.

[ Eleni Maria Stea ]
  * It fixes the bug #1245886. In DecorScreen::handleEvent compiz
    shouldn't try to handle any events if there's no active window yet.
    (LP: #1245886)
  * Compiz static analysis shows that some compiz classes have virtual
    methods but not virtual destructors. Added the virtual destructors
    to get rid of warnings and potential memory leaks.
  * fixed cmake syntax errors.
  * CMake considered compiz a C++ project and couldn't find some
    dependencies like pthreads. Defined compiz as a C, CXX project to
    fix the issue.

[ Povilas Kanapickas ]
  * Opacify: Properly initialize window drawing for new windows in
    Opacify plugin. (LP: #787814, part 2). (LP: #787814)
  * Opacify: Fix damage generation in the Opacify plugin. When setting
    opacity to some value, non-opacified windows need to be damaged
    regardless of opacity, whereas opacified windows need to be damaged
    only if opacity changes. Remove uses of OPACIFY_WINDOW and
    OPACIFY_SCREEN. These macros cause significant loss of readability
    while providing no observable benefits.
  * OpenGL, fragment shader: Run the opacity filter after all other
    filters have been run. (LP: #1257223). (LP: #1257223)

[ Daniel van Vugt ]
  * This branch is 0.9.11

[ Hu Kang ]
  * Remove redundant src/logmessage/include/core/logmessage.h (LP:
    #1067246). (LP: #1067246)

[ Brandon Schaefer ]
  * Unrevert 3728, fix failing tests. Change the behaviour of
    undecorating windows. Previously when a window was undecorated, we
    would shift it back to an appropriate position according to its
    gravity member. That behaviour was problematic because in the
    StaticGravity case the window has to just stay in the same place.
    But then if you had a window with StaticGravity which then did get a
    decoration and later removed it, it would be placed as though it was
    decorated and appear to be in the wrong place. The correct behaviour
    is to place all windows as though they have decorations, and then
    when decorations are removed, to move the window back to the corner
    as indicated in its gravity and then expand its size to cover the
    obscured regions no longer hidden because the decorations went away.
    (LP: #1165343).   1. Completely remove decorOffsetMove and other
    related code from      decor.cpp. Put the logic to handle the
    window->input () - window->border ()      placement offset inside of
    setWindowFrameExtents instead. Now the window      will always be
    offset from its original non-decorated position to the new
         decorated position, rather than having to guess between
    decoration sizes.   2. Make saveGeometry and restoreGeometry work
    relative to window->border ()      as opposed to including it in the
    saved geometry. It is possible that the      border size might
    change during maximization, as such, we don't want to      save the
    position with the border before maximizing. Instead save the
    position      as if it were never decorated so that when the window
    is restored it can be      restored to its original position and
    then adjusted for its new border size.   3. Fix a few typoes in the
    tests.   4. Moved some commonly used matchers into compiz::testing
      5. Make COMPIZ_PLUGIN_DIR accept multiple directories and look in
    each one      of them for the plugin   6. Set COMPIZ_PLUGIN_DIR
    appropriately for each plugin that we wish to load      on startup
    so that we load locally built plugins as opposed to installed
         ones.   7. Uncomment compiz_discover_tests for the acceptance
    tests. Now they are      run by default. (LP: #1195522). (LP:
    #1195522, #1165343)
  * If we receive a stateNotifyChange, we check if we are in a max
    state. If so, overwrite the xwc with either X | W, or Y | H
    depending on the semi max state with the orig position before we
    went into a semi max state. This way when we leave any sort of max
    state the orig position is restored. If the window is being grabbed
    we do not overwrite the position, which it gets its own value at
    that point (instead of the orig pos). (LP: #892012)

[ Marco Trevisan (Treviño) ]
  * debian/patches/ubuntu-config.patch: remove grid custom keybindings
    for window management We handle these directly in unity
    (lp:~3v1n0/unity/super-arrows-shortcuts). (LP: #992697)
  * ActionMenu: weak ref the action menu on creation, and unref it on
    destroy Also, now the widget destruction automatically unset the
    menu. This makes the object to be correctly finalized (not only
    disposed) making sure the internal idle into WnckActionMenu gets
    stopped. (LP: #1191853)
  * debian/patches/ubuntu-config.patch: Don't use Ctrl+Alt+KP_0 for grid
    maximize Otherwise this clashes with default minimize key. (LP:
    #1236899)
  * Window: disable the size adjustments when decorating/undecorating
    windows. (LP: #1238111)
  * Window: resize a maximized window if the decorations changed, so
    that it will fill all the available space.
  * compiz-gnome: remove decor plugin from unity profile.

[ Michail Bitzes ]
  * Plugin wizard ported to OpenGL|ES. Use GLVertexBuffer. Enable
    building for GLES. Enable architectures armel and armhf. (LP:
    #1196003). (LP: #1196003)
  * Wizard plugin: No more segfaults/crashes when changing options in
    CCSM. (LP: #1195996) Gradual fade out of particles when terminating.
    (LP: #1196010) Typo fixed in wizard.xml.in. optionNotify added for
    some options that were missing it. Code cleanup: Declare variables
    outside of loops. Functions moved from class WizardScreen to class
    ParticleSystem. (similar to firepaint/showmouse) We just enable
    GL_BLEND if it is disabled and we just disable GL_BLEND if it was
    disabled before. Many other style and indentation fixes. (LP:
    #1196010, #1195996)
  * Plugin showrepaint ported to OpenGL|ES. Use GLVertexBuffer. Enable
    building for GLES. Enable architectures armel and armhf. .

[ Chris Townsend ]
  * Removed logic in the calculateWallOffset() function in the Expo
    plugin that only accounted for offsetting the expo animation on the
    left-most and top-most monitor. Also removed the
    TestNoOffsetIfOutputIsNotOrigin test since this test is now invalid.
    (LP: #1031710)
  * Ctrl-Alt-Del should open the gnome-system-monitor at the processes
    tab. This fix uses the commands plugin to override the default
    action. This includes: - Install the commands plugin by default. -
    Breaks/Replaces the compiz-plugins package for versions earlier than
    0.9.10. - Patch commands.xml and integrated.xml to map gnome-system-
    monitor to Ctrl-Alt-Del by default. - Start the command plugin upon
    Compiz starting. (LP: #890747)
  * Alt-Tabbing or Launcher selecting a window that is over 50% in a
    different viewport should not switch the viewport nor change the
    placement of the window. The fix is to add an option to turn this
    behavior on or off. By default, the option is on, but Ubuntu is
    patched to turn it off to fix this bug. (LP: #1092323)
  * Fix regression when selecting apps on a different viewport than the
    current one will no longer automatically switch the viewport. (LP:
    #1228352)
  * Fix issue where non-resizable windows and semi-maximized window
    would have an area near the top of the title bar that could not be
    grabbed. (LP: #1228507)
  * Remove seemingly useless code in the SnapWindow::resizeNotify()
    method where a snapped window that is being resized on the opposite
    side of the snap would cause orders of magnitude more calls to
    window->resize() and kill performance. (LP: #1019139)
  * Need to revert rev. 3798 because it has a bad side effect of
    "hanging" all of the windows when restarting Compiz and rendering
    the desktop unusable.
  * Bump ABI version due to some recent changes.
  * When saving window geometry when going into maximized/semi-
    maximized, also save for grabbed windows as well. This fixes a
    desktop hang condition. (LP: #1244754)
  * Second part of the fix to keep the workspace from automatically
    switching when more than half of the window is contained in another
    workspace. (LP: #1092323)
  * Add new Grid shortcuts for Ctrl-Super-Left/Right to left/right semi-
    maximize windows (removed from Unity). Patch Grid to use Ctrl-Super-
    Up to maximize a gridded window instead of using core. This helps in
    restoring a window back to it's original place before being gridded.
    Account for left and right borders when restoring a gridded window
    since the window would shrink and shift by this amount. (LP:
    #1251777)

[ James Jones ]
  * OpenGL contexts are reference counted and only destroyed once they
    are no longer current to any thread. To ensure the context really is
    destroyed, lose current from it before destroying it. (LP: #1213809)

[ Sami Jaktholm ]
  * Expo plugin: 1) Always fetch the dndButton action in dndFini. The
    action given to dndFini might be a key or edge binding action if
    expo was terminated during window DnD. We shouldn't set their state
    to CompAction::StateInitButton or the bindings break. 2) Pass NULL
    action to termExpo if invoked from handleEvent Previously we passed
    the action of expoKey to termExpo which in turn passed it to
    dndFini. DndFini changed the action state to
    CompAction::StateInitButton that broke the keybinding. Now that
    dndFini fetches the dndButton action itself, there's no need to
    fetch it in handleEvent too as dndFini is the only one using the
    action. (LP: #1131106)
  * Expo: Cast denominators of integer divisions to floats before
    performing a matrix translation to viewports with the results.
    Otherwise rounding errors in cases the results are not an integers
    cause the viewports to be painted offscreen. (LP: #438580)
  * Port dbus introspection to compiz 0.9. move xml creation to a
    separate class (IntrospectionResponse) for easier. memory management
    (allocate buffer and writer in ctor, free in dtor). move duplicated
    response sending code to a separate method that takes.
    IntrospectionResponse and sends the resulting xml. Refactor
    handle*IntrospectMessage to work with compiz 0.9 interfaces. This
    also fixes the broken list method which was a result of logic error.
    The code to invoke list handler was never reached. This fixes most
    of the issues noted in bug 749084. (LP: #749084)
  * Bias the current viewport in addWindowSizeChanges. If window is
    visible in the current viewport, use that viewport when calculating
    geometries in addWindowSizeChanges. Otherwise the current method is
    used. This way visible windows are maximized in the current
    viewport. However, offscreen windows won't jump back to the active
    viewport if window maximizes by itself or addWindowSizeChanges is
    called for some other reason. (LP: #776435)
  * Expo: Recompute glow quads if a desktop window is resized. (LP:
    #1090713). (LP: #1090713)
  * Decor: Use maximized border extents only if window is fully
    maximized. The decorator draws a normal border around semi-maximized
    windows. When maximized border extents were used for semi-maximized
    windows, compiz didn't reserve any space for the border in its
    geometry calculations. At least following problems are a result of
    this behavior: Semi-maximized windows have 1px borders drawn on
    adjacent workspaces. (LP: #986051). Grid placed window overlaps the
    adjacent viewport (LP: #898870). (LP: #986051, #898870)

[ MC Return ]
  * Fade plugin, minor code cleanup: Declaration and assignment of local
    variables in one line. Merged if condition checks. Use pre- instead
    of postfix de- and increments. Removed redundant brackets. Added and
    removed newlines. Fixed indentation.
  * Opacify code cleanup: Declaration of local variables outside of
    loops. Calculate activeOpacity and targetOpacity just once, not for
    each window. Use prefix instead of postfix increments. Merged if
    condition checks. Declaration and assignment of local variables in
    one line. Added and removed newlines. Removed redundant brackets.
    Fixed indentation. Opacify, functionality fix: Improved the toggling
    logic by setting isToggle in the constructor according to the
    startup setting, then calling setFunctions (isToggle);,which then
    calls screen->handleEventSetEnabled (os, os->isToggle); to fix
    Opacify ignoring the "Toggle Opacify on by default" setting. (LP:
    #787814). (LP: #787814)
  * Widget, minor code cleanup: Declaration and assignment of local
    variables in one line. Merged if condition checks. Added and removed
    newlines. Added default case to switch (just a style issue). Removed
    redundant brackets. Fixed indentation.
  * window.cpp code cleanup: Declare variables outside of loops. Reduced
    the scope of variables. Do not assign values to variables if those
    values are not used. Always bail out of functions ASAP, do not
    calculate stuff you might not need. Use pre- instead of postfix de-
    and increments. Declaration and assignment of variables in one line.
    Merged if condition checks. Added and removed brackets. Added and
    removed newlines. Fixed indentation. Added TODOs (removal of magic
    numbers).
  * Fix for cube.cpp, regression r3720 - Inner sides of cube invisible
    now. Wrong variable type was accidentially used, cullInv is of type
    int, not bool. (LP: #1183852). (LP: #1184852, #1183852)
  * src/output*.cpp cleanup: Return ASAP, do not calculate stuff you do
    not need. Declaration and assignment of variables in one line.
    Declare local variables outside of loops. Use prefix instead of
    postfix increment. Added and removed newlines. Fixed indentation.
  * Expo, code cleanup: Declaration of local variables outside of loops.
    Use prefix instead of postfix increments. Declaration and assignment
    of local variables in one line, if possible. Reduced the scope of
    some variables. Removed redundant brackets. Used static_cast <>
    (type) instead of (type) cast. Added and removed newlines if
    appropriate. Fixed indentation. Added TODOs. Expo, speed
    improvements: Do not calculate screen->vpSize ().width () and
    screen->vpSize ().height () multiple times, instead save them in the
    unsigned ints vpCountHorz and vpCountVert and use those variables
    instead. Speed up the curve calculations by using additional
    variables to save results to not have to re-calculate those all the
    time. The new variables introduced are degToRad, screenWidth,
    screenWidthSquared, curveDistSquaredPlusQuarter,
    pOne2MinusCurveDist, v0Squared and v2Squared. Also introduced const
    float halfGapX = gapX / 2.0 and used this variable in the
    calculations of curveDistance and curveRadius. Simplified
    calculations of this type: (M_PI / 180.0f) * curveAngle / 2.0, which
    is equal to: (M_PI / 360.0f) * curveAngle Do not call
    optionGetGroundSize (); twice, instead save the value in the float
    groundSize and use that variable in the following calculations.
    Removed the creation of the redundant bool hide, which is just used
    once in an if-condition check. It does not help to have this bool.
    Try to avoid redundant GL_BLEND state changes, they are expensive -
    only enable GL_BLEND if it is disabled and just disable it, if it
    was disabled before, otherwise do nothing. Try to avoid redundant GL
    filter changes - just query the filter state if the mipmap option in
    CCSM is enabled. Also just set back the filter to the previous state
    if we actually changed it, otherwise do nothing. -5.5f * 2 = -11.0f.
    No need to calculate M_PI / 180.0f 720 times: Calculate this value
    once instead and save it in the const float mpi, use mpi in the
    following looped calculations. Use const int scw in the same loop
    instead of calling screen->width () 360 times. glow.cpp: Massively
    increased calculation speed of the glow texture: No need for any
    macros here -> removed them and replaced them with local variables.
    Precalculate values, store them in local variables and use those in
    the following calculations (new variables are the ints winRealX,
    winRealY, winRealWidth, winRealHeight, halfWinRealWidth,
    halfWinRealHeight, xPlusHalfWidth, yPlusHalfHeight, xPlusGlowOff,
    yPlusGlowOff, xMinusGlowOff, yMinusGlowOff and the float glowPart).
    Used w->geometry ().widthIncBorders () and w->geometry
    ().heightIncBorders () to determine winRealWidth and winRealHeight.
    Expo, fixes: Do not force "One wall per output" on the user, if his
    displays use different resolutions. "One big wall" makes a lot of
    sense for many multi-screen configs, where not all of the screens
    have exactly the same resolution, so if the user explicitely chooses
    this mode, Compiz should respect the user's choice. (LP: #1009592).
    (LP: #1009592)
  * Compiz, redundant and outdated files cleanup: Removed outdated NEWS
    files. Removed outdated AUTHORS and ChangeLog files. Removed
    plugins/freewins/COPYING - we do not need another copy of the GPL v2
    here. Removed another copy of the GPL v2 from plugins/workarounds.
    This will reduce package sizes, download time and bring some order
    to chaos.
  * Expo, .xml.in fixes: Better description of the plugin. Improved and
    fixed tooltips. Moved the "Curve Strength" option closer to
    "Deformation". Refreshed the quilt patch for Ubuntu: There is no
    need to set the default distance to 0.005 as this setting has no
    effect without deformation. Set default multi-screen mode to "One
    wall per output" for Ubuntu, because "One big wall" works now for
    all display configurations and could potentially change the
    appearance for Ubuntu users. (LP: #1074487). (LP: #1074487)
  * Added contribute directory. Added QtCreatorConfig.xml, a
    configuration file for Qt Creator, which adjusts Qt Creator's C++
    indentation style to be the same that is used in the Compiz project
    (X11, mixed spaces and tabs).
  * Mousepoll: Fixed mousepoll version mismatch. (COMPIZ_MOUSEPOLL_ABI
    was missing) (LP: #1195659). (LP: #1195659)
  * Expo: Simplified the GL filter handling.
  * Fixed broken text in all plugins (missing COMPIZ_TEXT_ABI). (LP:
    #1196493). (LP: #1196493)
  * CCSM: Fixed vertical alignment of the text on the stylized keys.
    (Shift, Super, Ctrl, Alt) (LP: #1196345). (LP: #1196345)
  * Thumbnail, cleanup: Merged if condition checks. Declaration of
    variables when they are needed (C++ style). Removed redundant
    brackets. Removed useless "/* Could someone please explain how this
    works */" comment. Removed extern const unsigned short
    TEXT_DISTANCE, because this variable does not exist anymore (value
    is configurable already). Changed if (something > 0.0) to if
    (something), 10 times. Changed pointedWin = 0; to pointedWin =
    NULL;. Used centerX () and centerY () abstractions from
    compiz::window::Geometry to determine the icon center coordinates.
    Added and removed newlines, if appropriate. Fixed indentation.
    Thumbnail, code speedup: Return ASAP, do not calculate stuff you
    might not need. No need for WIN_W (w) and WIN_H (w) macros, removed
    those and replaced them with the local variables int winWidth, int
    winHeight, int dockWidth and int dockHeight. No need for WIN_X (w)
    and WIN_Y (w) macros, replaced those with int dockX and int dockY.
    Introduced int tHeight = thumb.height;, int tWidth = thumb.width;,
    int halfTWidth = tWidth / 2; and int halfTHeight = tHeight / 2; and
    used those variables in the following calculations. Use
    igMidPoint[0] and igMidPoint[1] as arguments in the screen-
    >outputDeviceForPoint (arg1, arg2) function, instead of re-
    calculating the arguments again. Thumbnail, GL speedup: Introduced
    GLfloat wxPlusWidth = wx + width;, GLfloat wyPlusHeight = wy +
    height;, GLfloat wxPlusWPlusOff = wxPlusWidth + off;, GLfloat
    wyPlusHPlusOff = wyPlusHeight + off;, GLfloat wxMinusOff = wx - off;
    and GLfloat wyMinusOff = wy - off; and used those coordinates in the
    vertexData arrays. No need to re- calculate those values multiple
    times. Just query optionGetShowDelay () once and use int showDelay
    later. We just enable blending if it is currently disabled and
    disable blending only, if it was disabled before.
  * src/session.cpp cleanup: Declaration and assignment of local
    variables in one line. (this *should* fix bug #1101405) Use pre-
    instead of postfix increments. Added and removed newlines. Added
    missing break (just a style issue). Fixed indentation. (LP:
    #1101405). (LP: #1101405)
  * Firepaint, code cleanup: Declare variables outside of loops.
    Declaration and assignment of local variables in one line. Use pre-
    instead of postfix increment. Removed redundant newlines. Removed
    redundant casts. (from (float) optionGetFireColor) Removed redundant
    brackets. Added missing break (style issue only). Added comment and
    TODO. Fixed indentation. Firepaint, speedup: We just enable GL_BLEND
    if it is disabled and we just disable GL_BLEND if it was disabled
    before. Introduced the GLfloats xMinusW, xPlusW, yMinusH and yPlusH
    to store calculated coordinates instead of recalculating them
    multiple times. Introduced float fireLife = optionGetFireLife ();,
    float fireWidth = optionGetFireSize ();, float fireHeight =
    fireWidth * 1.5f; and bool mystFire = optionGetFireMystical (); and
    used those variables inside the loop. Firepaint, .xml.in cleanup:
    Firepaint now has 2 tabs, 'General' and 'Particle Settings'. The
    name of this plugin is Firepaint, <short> should not contain the
    description. Better description of the plugin. Uppercase option
    titles. Punctuation for tooltips. Improved tooltips. Firepaint,
    fixes: Initialize all class member variables in the ParticleSystem::
    ParticleSystem () ctor (LP: #1101512, LP: #1101580). (LP: #1101512,
    #1101580)
  * img* plugins code cleanup: Always bail out of function ASAP, do not
    calculate stuff you might not need. Declare variables outside of
    loops so they won't be re-declared in each loop. Use prefix instead
    of postfix increments. Declaration and assignment of variables in
    one line. Merged if condition checks. Added and removed brackets.
    Added and removed newlines, if appropriate. Minor code structure
    improvements, declare variables when you need them, not much
    earlier. Fixed indentation.
  * Text, speed improvements: Introduced const float halfPi = PI / 2.0f;
    and const float triHalfPi = halfPi * 3; and used those to draw the
    rounded background. Introduced GLfloat xPlusWidth = x + width; and
    GLfloat yMinusHeight = y - height; and used those coordinates for
    the vertexData array. Text, cleanup: Declaration of local variables
    outside of loops. Fixed indentation.
  * cube.cpp, code cleanup: #include "privates.h", not <privates.h>.
    Merged if condition checks. Declaration of variables, where you need
    them to improve the code structure and readability. Use C++ standard
    bool for topDir, bottomDir, allCaps and mCapsPainted[output] (bool
    true/false instead Bool TRUE/FALSE). Initialize all of the class
    member variables in the constructor. Replaced if (priv->mSky.size ()
    > 0) with if (!priv->mSky.empty ()). Removed redundant brackets.
    Fixed indentation. Added TODO. cube.cpp, code speedup: Do not call
    optionGetInactiveOpacity () twice, instead save the value as float
    inactiveOpacity and use this variable in the following calculation.
    Introduced the variables float halfHsize = hsize / 2.0; and float
    tsSize = 360.0f / size; and used those in following calculations to
    not have to recalculate those values multiple times. Do not
    calculate M_PI / sides and 2 * M_PI / sides multiple times in loops,
    instead save those values in the GLfloats mps and tmps and use those
    values inside the loops. Use GLdouble mDist2 = 0.5 / mDistance; in
    the GLdouble clipPlane* arrays, instead of recalculating this value
    multiple times. Use GLfloat oneMinusFStepX = 1.0f - fStepX; and
    GLfloat oneMinusFStepY = 1.0f - fStepY; instead of calculating those
    values multiple times. Use float outputWidth = outputPtr->width ();
    and float outputHeight = outputPtr->height (); instead of calling
    the functions multiple times. Introduced the GLfloat normInvert =
    0.5f * invert; and used this variable instead of recalculating this
    value multiple times. Introduced the GLfloats mSkyWidth and
    mSkyHeight and used those in the following loops instead of calling
    mSkySize.width () and mSkySize.height () multiple times. Try to
    avoid redundant GL_BLEND state changes, because OpenGL will blindly
    change the global state, no matter what it currently is set to and
    that state change is expensive. So we query and save the actual
    blending state in the variable GLboolean glBlendEnabled =
    glIsEnabled (GL_BLEND); and just enable it if it isn't already and
    just disable it, if it was disabled before. (LP: #1101422, LP:
    #1195977) cube.cpp, other fixes: Return false in function
    PrivateCubeScreen::updateGeometry (int, int) if sides should ever be
    0, which should never happen, but should make Coverity happy. (LP:
    #1101541) Initialize all class member variables in the
    PrivateCubeScreen:: PrivateCubeScreen (CompScreen *) constructor.
    (mTc, mNOutput, mOutput, mOutputMask, mCleared, mCapsPainted) (LP:
    #1101575). (LP: #1195977, #1101541, #1101422, #1101575)
  * plugins/ezoom/src/ezoom.cpp: Also initialize xtrans and ytrans in
    the ctors. Just just enable blending if it is disabled and just
    disable it if it was disabled before. Introduced the variables
    oWidth, oHeight, halfOWidth and halfOHeight to speed up following
    calculations. Return ASAP, do not calculate possibly redundant
    stuff. Declaration and assignment of local variables in one line.
    Improved readability. Added and removed newlines. Minor indentation
    fixes. plugins/ezoom/ezoom.xml.in: Removed redundant whitespaces
    from a tooltip. (LP: #1101572). (LP: #1101572)
  * gears.cpp, gears.h: Do not render the gears if we are viewing the
    cube from the inside - the gears are otherwise obstructing the
    users' view on his workspaces. Print a warning to the console once
    in this case. (LP: #1185819). (LP: #1185819)
  * Animation code cleanup: Return ASAP, do not calculate stuff you
    might not need. Declare CompWindow *w, AnimWindow *animWin,
    PrivateAnimWindow *aw and Animation *curAnim outside the for loop.
    grid.cpp: Declare float x, y, topiyFloat outside the for loop.
    options.cpp: Declare unsigned int nOptions outside the foreach loop.
    Use pre- instead of postfix de- and increments. Declaration and
    assignment of variables in one line. Merged if condition checks.
    Added and removed brackets. Added and removed newlines. Fixed
    indentation. Also initialize the non-static class members
    "mGridWidth" and "mGridHeight" in the GridAnim::GridAnim ctor. (LP:
    #1101618). (LP: #1101618)

[ Andrea Azzarone ]
  * Satisfy matchers on getProperty as soon as they are set. The order
    of evaluation for matchers in Google Mock appears to be undefined -
    this means that we can't rely on the first argument being matched
    first and the second argument being matched afterwards. In turn,
    this means that any GValue may be passed to a GValueMatch which, by
    a design flaw, is unable to handle any values of a type it does not
    expect (at least not without an API change). It will silently pass
    the incorrect type to g_type_get_* which causes internal assertion
    failures. At the moment we're just interleaving the calls to
    getProperty and get_property - that way the expectations are
    satisfied and go away as soon as they're set. This in turn means
    that Google Mock only has to traverse one matcher rather than
    multiple matchers. (LP: #1187468). (LP: #1187468)
  * Extends invisible borders to the title bar too. (LP: #717444). (LP:
    #717444)
  * Disable grid/put_maximize_key on ubuntu.
  * Port to new gmock. Disable some faling tests because of weird leaks.
    (LP: #1185265)
  * Update window geo only on new decoration size only if mapped. (LP:
    #1198000). (LP: #1198000)
  * Disable show_desktop_key by default on ubuntu. Will provide the same
    option under the unityshell panel. (LP: #1204664)
  * Use Glib::RefPtr<Glib::Source> in glib_integration_test. After the
    fix of bug https://bugzilla.gnome.org/show_bug.cgi?id=561885
    Glib::Source::~Source is called when both Source::unreference() and
    SourceCallbackData::destroy_notify_callback() are called. (LP:
    #1214459)
  * Terminate scale on mouse click on an empty area. (LP: #1234624)

[ Sam Spilsbury ]
  * Change the behaviour of undecorating windows. Previously when a
    window was undecorated, we would shift it back to an appropriate
    position according to its gravity member. That behaviour was
    problematic because in the StaticGravity case the window has to just
    stay in the same place. But then if you had a window with
    StaticGravity which then did get a decoration and later removed it,
    it would be placed as though it was decorated and appear to be in
    the wrong place. The correct behaviour is to place all windows as
    though they have decorations, and then when decorations are removed,
    to move the window back to the corner as indicated in its gravity
    and then expand its size to cover the obscured regions no longer
    hidden because the decorations went away. (LP: #1165343). (LP:
    #1165343)
  * Added some new hooks to PluginClassHandler to allow a VTable to
    specify if loaded. PluginClassHandler::get () was designed to simply
    instantiate an instance of that class for the core structure, but it
    did this without checking if the plugin was loaded. Added some new
    methods to PluginClassHandler exposed by LoadedPluginClassBridge and
    only accessible by those who implement PluginKey to specify globally
    whether or not a plugin is actually loaded, so that
    PluginClassHandler can return accordingly. Integration and unit
    tests added as appropriate (LP: #1169620) (LP: #1101026). (LP:
    #1101026, #1169620)
  * Satisfy matchers on getProperty as soon as they are set. The order
    of evaluation for matchers in Google Mock appears to be undefined -
    this means that we can't rely on the first argument being matched
    first and the second argument being matched afterwards. In turn,
    this means that any GValue may be passed to a GValueMatch which, by
    a design flaw, is unable to handle any values of a type it does not
    expect (at least not without an API change). It will silently pass
    the incorrect type to g_type_get_* which causes internal assertion
    failures. At the moment we're just interleaving the calls to
    getProperty and get_property - that way the expectations are
    satisfied and go away as soon as they're set. This in turn means
    that Google Mock only has to traverse one matcher rather than
    multiple matchers. (LP: #1187468). (LP: #1187468)
  * Revert revision 3278.
  * Immediately update the window matrices and regions if the number of
    textures changed. (LP: #1189369). (LP: #1189369)
  * Provide a basic decor plugin acceptance test suite. This change
    provides a simple acceptance test suite for the decor plugin. It
    creates a fake window decorator (cdt::FakeDecorator) and allows
    users to create fake decorations (eg cdt::FakePixmapDecoration)
    which can be serialized as decoration properties and set on windows
    under xorg-gtest. It also launches compiz with the opengl, composite
    and decor plugins loaded and runs some basic tests. Among them:
    BaseDecorAcceptance. Startup: Basic canary "can we start compiz with
    these plugins" test
    FakeDecoratorSessionOwnerNameSetOnSelectionOwner: Create a fake
    decorator, ensure that libdecoration sets our session name "fake" on
    the selection owner FakeDecoratorReceiveClientMessage: Create fake
    decorator, ensure that libdecoration posts a client message to the
    root window announcing that the new decorator exists.
    DecorationSupportsWindowType: Create a fake decorator, announce
    support for the WINDOW type decorations and ensure that the correct
    atom is set on the session owner window.
    DecorationSupportsPixmapType: Create a fake decorator, announce
    support for the PIXMAP type decorations and ensure that the correct
    atom is set on the session owner window.
    DecorFakeDecoratorAcceptance. WindowDefaultFallbackNoExtents: By
    default, newly created windows should recieve a fallback decoration
    but they should not have any frame extents.
    DecorWithPixmapDefaultsAcceptance. These tests create a default
    pixmap decoration to use in the hypothetical situation that the
    window decorator hasn't yet generated a decoration for this window.
    FallbackRecieveInputFrameNotify: Verify that we get a
    _COMPIZ_WINDOW_DECOR_INPUT_FRAME property set on the client when it
    is created and mapped (eg, an input frame window was annonuced to be
    created) FallbackHasInputFrameInParent: Verify that a second window
    exists in the frame window after this message was recieved.
    FallbackNormalWindowExtentOnDecoration: Verify that
    _NET_FRAME_EXTENTS is set to the default window extents for the
    default decoration when the window was mapped.
    FallbackNormalWindowInputOnFrame: Verify that the input window
    matches the extents set. PixmapDecoratedWindowAcceptance. These
    tests create a new window and an associated unique decoration for
    that window. MaximizeBorderExtentsOnMaximize: Maximize the window
    and ensure that the correct border extents are used.
    MaximizeBorderExtentsOnVertMaximize: Vertically maximize the window
    and ensure that the correct border extents are used.
    MaximizeBorderExtentsOnHorzMaximize: Horizontally maximize the
    window and ensure that the correct border extents are used.
    MaximizeFrameWindowSizeEqOutputSize: Maximie the window and ensure
    that the frame window exactly equals the output size.
    VertMaximizeFrameWindowSizeEqOutputYHeight: Maximize the window and
    ensure that the frame window's Y and Height values are consistent
    with the output size. HorzMaximizeFrameWindowSizeEqOutputXWidth:
    Ditto horizontal maximization.
    DISABLED_VertMaximizeFrameWindowSizeSameXWidth: Maximize the window
    vertically and ensure that the frame window's border-relative X
    position and width does not change. This test is disabled, as the
    behavior in compiz is currently broken, (but it means that we can
    fix it later and enable the test).
    DISABLED_HorzMaximizeFrameWindowSizeSameYHeight Maximize the window
    horizontally and ensure that the frame window's border-relative Y
    position and height does not change. This test is disabled, as the
    behavior in compiz is currently broken, (but it means that we can
    fix it later and enable the test). (LP: #1188900). (LP: #1188900)
  * Only mark for no further instantiations once we've finished
    destructing the plugin (LP: #1193596). (LP: #1193596)
  * Add more acceptance tests for the decor plugin.
    PixmapDecoratedWindowAcceptance. UndecoratedWindowExpandToOrigSize
    Test that upon undecoration, the window has exactly the same
    geometry as it did before decoration.
    DISABLED_UndecorateStaticGravityWindow Tests that for windows with a
    static gravity, the window has exactly the same geometry as it did
    before decoration. Disabled, as core is currently not moving the
    window with the static gravity back to where it started.
    AdjustmentExtents/PixmapDecorationAdjustment.
    AdjustRestoredWindowBorderMovesClient/P Test that changing the
    border extents causes the client window's absolute geometry to
    change. DISABLED_AdjustRestoredWindowBorderShrinkClient/P Tests that
    changing the border extents causes the client window's absolute
    geometry to shrink by the amount of border. Disabled, as the current
    behavior is to expand the frame window as opposed to shrinking the
    client (but this will be changed).
    DISABLED_ClientExpandsAsBorderShrinks/P Tests that as the border
    shrinks away, the client expands back to its original size.
    Disabled, as the current behavior is to expand the frame window as
    opposed to shrinking the client (but this will be changed).
    DISABLED_ClientExpandsAsBorderShrinksWhilstMaximized/P Tests that
    even if the client is maximized, if the restored border shrinks away
    it demaximizes back to the same position while also accounting for
    any change in the restored window border size. Disabled, as the
    current behavior is to expand the frame window as opposed to
    shrinking the client (but this will be changed).
    DISABLED_ClientExpandsAsBorderShrinksWhilstUndecorated/P Tests that
    even when the client is not permitted to be decorated because the
    decoration hint was removed, when it is redecorated it will be moved
    to the correct position taking into account any changes in its
    border size. Disabled, as the current behavior is to expand the
    frame window as opposed to shrinking the client (but this will be
    changed). DISABLED_AdjustRestoredWindowInputNoMoveClient/P Tests
    that adjusting the input extents (as independent from the border
    extents) of the client does not cause the client to move. Disabled,
    as this behavior appears to be broken in core. Move the
    "GetImmediateParent" function into the compiz::testing namespace so
    that it can be used by other tests. (LP: #1188900)
  * Unrevert 3728, fix failing tests. Change the behaviour of
    undecorating windows. Previously when a window was undecorated, we
    would shift it back to an appropriate position according to its
    gravity member. That behaviour was problematic because in the
    StaticGravity case the window has to just stay in the same place.
    But then if you had a window with StaticGravity which then did get a
    decoration and later removed it, it would be placed as though it was
    decorated and appear to be in the wrong place. The correct behaviour
    is to place all windows as though they have decorations, and then
    when decorations are removed, to move the window back to the corner
    as indicated in its gravity and then expand its size to cover the
    obscured regions no longer hidden because the decorations went away.
    (LP: #1165343).   1. Completely remove decorOffsetMove and other
    related code from      decor.cpp. Put the logic to handle the
    window->input () - window->border ()      placement offset inside of
    setWindowFrameExtents instead. Now the window      will always be
    offset from its original non-decorated position to the new
         decorated position, rather than having to guess between
    decoration sizes.   2. Make saveGeometry and restoreGeometry work
    relative to window->border ()      as opposed to including it in the
    saved geometry. It is possible that the      border size might
    change during maximization, as such, we don't want to      save the
    position with the border before maximizing. Instead save the
    position      as if it were never decorated so that when the window
    is restored it can be      restored to its original position and
    then adjusted for its new border size.   3. Fix a few typoes in the
    tests.   4. Moved some commonly used matchers into compiz::testing
      5. Make COMPIZ_PLUGIN_DIR accept multiple directories and look in
    each one      of them for the plugin   6. Set COMPIZ_PLUGIN_DIR
    appropriately for each plugin that we wish to load      on startup
    so that we load locally built plugins as opposed to installed
         ones.   7. Uncomment compiz_discover_tests for the acceptance
    tests. Now they are      run by default. (LP: #1195522). (LP:
    #1195522, #1165343)
  * Add a simple script for making releases.
  * Port to new gmock. Disable some faling tests because of weird leaks.
    (LP: #1185265)
  * Don't add the frame to the toplevel stack if it hasn't been created
    yet. In the event that a window is unreparented or destroyed, we
    usually need to add its frame window to the toplevel window stack
    until the time at which we recieve a DestroyNotify for it, as there
    may be incoming ConfigureNotify events puporting to stack other
    windows relative to that frame. However, this does not apply in the
    case where we have not yet received a CreateNotify for the frame
    window. In that case, it is not possible for any stacking requests
    to be made relative to this window, so it does not need to be added
    immediately. Instead, we can add it at the time that we recieve a
    CreateNotify for it as a regular override redirect window until the
    time that it is later destroyed. (LP: #1171314). (LP: #1171314)
  * Remove redundant src/logmessage/include/core/logmessage.h (LP:
    #1067246). (LP: #1067246)
  * Adjust acceptance tests for the fix to LP: #1198000 1. Always set
    frame extents before updating the frame 2. Added
    COMPIZ_NO_CONFIGURE_BUFFER_LOCKS so that we don't have to deal with
    the delayed configure requests logic in the tests where it does not
    help 3. Split restoredDecorationSize in to
    restoredDecorationBorderSize and restoredDecorationInputSize so that
    we can measure the size of the decoration both in terms of its
    visible and invisible border 4. Wait for the default decoration
    properly 5. Capture the initial geometry of the window with and
    without decorations when the window gets its first decoration. Use
    these values to calculate the size changes in the window from then
    on instead of its creation values 6. Just flag when the window was
    first decorated in setWindowFrameExtents rather than checking if it
    is mapped. We only want to not resize the window once. (LP:
    #1198000)
  * Release version 0.9.10.0
  * Bump VERSION to 0.9.10.2
  * Ensure that the frame region is always set as soon as the window is
    decorated. Further ensure that the window decoration isn't
    needlessly reset if the window already had one. Refactored XShape
    usage into a common function. Added tests to verify the behaviour of
    shape set on initially creating a decorated window and also upon
    changing the input frame window shape (LP: #1158267). (LP: #1158267)
  * Add support for the GLX_EXT_buffer_age extension on hardware that
    supports it, which should result in a nice speed-boost due to the
    fact that we no longer have to abuse fillrate by retaining old frame
    contents, and instead asking the driver how defined the current
    frame is. (LP: #1024304)
 -- Ubuntu daily release <ps-jenkins@lists.canonical.com>   Fri, 14 Feb 2014 22:54:33 +0000

Changed in compiz (Ubuntu):
status:	In Progress → Fix Released

Stephen M. Webb (bregma) on 2014-04-03

Changed in unity:
status:	Fix Committed → Fix Released

Christopher Townsend (townsend) on 2015-04-06

Changed in compiz:
status:	Fix Committed → Fix Released

Unity

[regression] Semi-maximizing and then restoring the window does not return it to it's original place using Ctrl-Super-Left/Right

Bug Description

Related branches

Other bug subscribers

Remote bug watches

	Status	Importance	Assigned to
Compiz	Fix Released	Medium	Christopher Townsend
Unity	Fix Released	Medium	Christopher Townsend
compiz (Ubuntu)	Fix Released	Medium	Christopher Townsend
unity (Ubuntu)	Fix Released	Medium	Christopher Townsend