Unity appears to enable decorations in _MOTIFY_WM_HINTS on its own

@bregma This is impacting valve users, could you please take a look? Thanks

Based on the behavior seen in the movie, I'm pretty sure this is the same issue reported in bug #1083186. I'll try to gather some more info on this bug before we duplicate it to the aforementioned bug.

I'm on Ubuntu 12.10 and this happens in Chrome as well when "Use System Title Bar and Borders" is unchecked.

This is fixed in 13.04, but is an issue in 12.10 and 12.04.

I'm currently working on a fix for those releases will soon be linking some branches and will work on getting them SRU'd.

Christopher:
if you look at the new code in PluginAdapter that is present in my branch linked here for trunk, it should help.
The whole code should be easily backportable with no regressions, while the usage of HasWindowDecorations/IsWindowDecorated in PanelMenuView and unityshell should be easy to switch.

@Marco,

Yep, I've already got a working branch for Unity/6.0:)

I just need to make sure I'm taking only what is absolutely needed in order to have the best chances of getting this SRU'd for both 12.10 and 12.04.

I now have a branch linked for Unity 5.0. My next steps are to get a test case in for each branch and then submit a merge proposal. Once the MP is approved, then I will nominate this for SRU in both Precise and Quantal.

Tomorrow I will have again access to my precise machine, so I will try testing and reviewing the 5.0 branch of the fix. We'll have to find someone from unity to take a look at the 6.0 branch as well. After these are merged in, we will try pushing them as SRUs in the nearest SRU scheduled release.

Hello Andrew, or anyone else affected,

Accepted unity into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/unity/5.20.0-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Hello Andrew, or anyone else affected,

Accepted unity into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/unity/5.20.0-0ubuntu2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

This bug was fixed in the package unity - 5.20.0-0ubuntu2

---------------
unity (5.20.0-0ubuntu2) precise; urgency=low

  * Check to see if the window is supposed to have decorations instead
    of the window actually being decorated when Undecorate() is called.
    This fixes a regression from the previous SRU where windows opened
    maximized upon login could have extra decorations (LP: #1195730)

unity (5.20.0-0ubuntu1) precise; urgency=low

  * New upstream release.
    - ResultRendererTile: fix a crash if row.renderer<TextureContainer*>()
      is null (LP: #926658)
    - Fix unity launcher vanishes when switching to mirrored displays
      (LP: #991637)
    - Fix window decorations being drawn when they should not be.
      This fixes icaclient window "dancing" (LP: #1083186) and
      hides decoration in a test case with Steam (LP: #1122478)
    - [multimonitor] Dash - Opening dash or HUD on one screen removes panel
      shadow on the other (LP: #892718)
    - Press Alt+F1, panel’s shadow dissapear (LP: #942965)
  * Cherry-pick a manual test for decorations fix
 -- Christopher Townsend <email address hidden> Mon, 08 Jul 2013 09:37:30 -0600

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.