~ps-jenkins/compiz/latestsnapshot-10.9.10+13.10.20131004-0ubuntu1 : changes

~ps-jenkins/compiz/latestsnapshot-10.9.10+13.10.20131004-0ubuntu1 » Changes from revision 3795

From Revision 3775 to 3756

Rev	Summary	Authors	Tags	Date
3775	Decor: Use maximized border extents only if window is fully ... 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). Fixes: https://bugs.launchpad.net/bugs/898870, https://bugs.launchpad.net/bugs/986051. Approved by PS Jenkins bot, Sam Spilsbury.	Sami Jaktholm		11 years ago
3774	Bump VERSION to 0.9.10.2 Bump VERSION to 0.9.10.2	Sam Spilsbury		11 years ago
3773	Release version 0.9.10.0 Release version 0.9.10.0	Sam Spilsbury	v0.9.10.0	11 years ago
3772	Expo: Recompute glow quads if a desktop window is resized. Expo: Recompute glow quads if a desktop window is resized. (LP: #1090713). Fixes: https://bugs.launchpad.net/bugs/1090713. Approved by PS Jenkins bot, MC Return, Sam Spilsbury.	Sami Jaktholm		11 years ago
3771	Text, speed improvements: 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. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return		11 years ago
3770	img* plugins code cleanup: 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. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return		11 years ago
3769	Firepaint, code cleanup: 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). Fixes: https://bugs.launchpad.net/bugs/1101512, https://bugs.launchpad.net/bugs/1101580. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return		11 years ago
3768	src/session.cpp cleanup: 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). Fixes: https://bugs.launchpad.net/bugs/1101405. Approved by PS Jenkins bot, Sam Spilsbury.	MC Return		11 years ago
3767	Fixed the non-working Annotate 'Clear' Button. Fixed the non-working Annotate 'Clear' Button. Moved this option's CCSM position upwards to keep the button shortcuts together. (LP: #1202907). Fixes: https://bugs.launchpad.net/bugs/1202907. Approved by PS Jenkins bot, MC Return.	BryanFRitt		11 years ago
3766	Bias the current viewport in addWindowSizeChanges. 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. Fixes: https://bugs.launchpad.net/bugs/776435. Approved by PS Jenkins bot, Brandon Schaefer, Sam Spilsbury.	Sami Jaktholm		11 years ago
3765	Adjust acceptance tests for the fix to LP: #1198000 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. Approved by PS Jenkins bot, Sam Spilsbury, Andrea Azzarone.	Sam Spilsbury		11 years ago
3764	Remove redundant src/logmessage/include/core/logmessage.h Remove redundant src/logmessage/include/core/logmessage.h (LP: #1067246). Fixes: https://bugs.launchpad.net/bugs/1067246. Approved by PS Jenkins bot, Sam Spilsbury.	Sam Spilsbury		11 years ago
3763	Don't add the frame to the toplevel stack if it hasn't been ... 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). Fixes: https://bugs.launchpad.net/bugs/1171314. Approved by PS Jenkins bot, MC Return.	Sam Spilsbury		11 years ago
3762	If we receive a stateNotifyChange, we check if we are in a m... 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). Fixes: https://bugs.launchpad.net/bugs/892012. Approved by PS Jenkins bot, Sam Spilsbury, MC Return, Sami Jaktholm.	Brandon Schaefer		11 years ago
3761	Update window geo only on new decoration size only if mapped... Update window geo only on new decoration size only if mapped. (LP: #1198000). Fixes: https://bugs.launchpad.net/bugs/1198000. Approved by PS Jenkins bot, Sam Spilsbury, MC Return.	Andrea Azzarone		11 years ago
3760	Port to new gmock. Disable some faling tests because of weir... Port to new gmock. Disable some faling tests because of weird leaks. Fixes: https://bugs.launchpad.net/bugs/1185265. Approved by PS Jenkins bot, Brandon Schaefer.	Sam Spilsbury		11 years ago
3759	Thumbnail, cleanup: 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. Approved by PS Jenkins bot, MC Return, Sami Jaktholm, Sam Spilsbury.	MC Return		11 years ago
3758	Port dbus introspection to compiz 0.9. 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. Fixes: https://bugs.launchpad.net/bugs/749084. Approved by PS Jenkins bot, MC Return.	Sami Jaktholm		11 years ago
3757	Expo: Cast denominators of integer divisions to floats befor... 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. Fixes: https://bugs.launchpad.net/bugs/438580. Approved by PS Jenkins bot, MC Return.	Sami Jaktholm		11 years ago
3756	Plugin wizard ported to OpenGL\|ES. Plugin wizard ported to OpenGL\|ES. Use GLVertexBuffer. Enable building for GLES. Enable architectures armel and armhf. (LP: #1196003). Fixes: https://bugs.launchpad.net/bugs/1196003. Approved by PS Jenkins bot, MC Return.	Michail Bitzes		11 years ago