Merge lp:~smspillaz/unity/unity.less-paint-insanity into lp:unity

[Sam Spilsbury]

Removes the whole scanning of the list thing in getWindowPaintList and the whole "trying to figure out where the window was because we don't actually know" in glDraw by scanning the paint list again and again.

There's a much better way of doing this - just hook glPaint, skip the wrap chain to core (eg every single plugin skipped) and tell core not to do anything with the window.

I have no idea why there is a bunch of logic about whether or not to paint the panel in glDraw (that was moved to glPaint). Someone enlighten me?

[Sam Spilsbury (smspillaz)]

Bump?

I'd like to see this go in, because it should reduce lots of list copying within unity and also lots of unecessary list traversal, which should represent a nice speed boost. In addition, its far simpler in its implementation and far less bug prone.

I'm sure testing is probably an issue. I'll have another quick look at the code and see if I can think of a way to get it under test.

[Daniel van Vugt (vanvugt)]

Works for me. Though there are some indentation/tab mismatches around line 107. Looks like your tabs are not 8 characters wide like they should be?

review: Needs Fixing

[lp:~smspillaz/unity/unity.less-paint-insanity]

2373. By Sam Spilsbury on 2012-06-12

Fix indentation

2374. By Sam Spilsbury on 2012-06-12

Merge lp:unity

2375. By Sam Spilsbury on 2012-06-12

Merge lp:~vanvugt/unity/fix-1010348

2376. By Sam Spilsbury on 2012-06-14

Make the code slightly more sane, use some interfaces to demonstrate
what is happening and break dependencies a little

2377. By Sam Spilsbury on 2012-06-14

Ensure that we always paint on top of the topmost nux window

2378. By Sam Spilsbury on 2012-06-14

Also return false if someone tries to paint us with PAINT_WINDOW_NO_CORE_INSTANCE_MASK

2379. By Sam Spilsbury on 2012-06-14

Implement top window detection in terms of paint requestor

2380. By Sam Spilsbury on 2012-06-14

Merge lp:unity

2381. By Sam Spilsbury on 2012-06-14

Move paint requestor enforcement to UnityPaintDispatch

2382. By Sam Spilsbury on 2012-06-14

Move paint mask prohibition into UnityPaintDispatch through strategy

2383. By Sam Spilsbury on 2012-06-14

Ensure that the backing fbo is painted too

2384. By Sam Spilsbury on 2012-06-14

Work correctly with USE_MODERN_COMPIZ_GL too

2385. By Sam Spilsbury on 2012-06-14

Move UnityPaintDispatch to separate file

2386. By Sam Spilsbury on 2012-06-14

Use UnityPaintDispatch by composition

2387. By Sam Spilsbury on 2012-06-14

Rename variable

2388. By Sam Spilsbury on 2012-06-14

UnityPaint* -> ShellPaint*

2389. By Sam Spilsbury on 2012-06-14

mv UnityPaintSchedule -> ShellPaintSchedule

2390. By Sam Spilsbury on 2012-06-14

Initial tests for ShellPaintSchedule

2391. By Sam Spilsbury on 2012-06-14

Basic paint tests

2392. By Sam Spilsbury on 2012-06-14

Basic prohibition test

2393. By Sam Spilsbury on 2012-06-14

Inverse prohibition test case

2394. By Sam Spilsbury on 2012-06-14

Basic top requestor tests

2395. By Sam Spilsbury on 2012-06-14

Test suprvening requestor behaviour

2396. By Sam Spilsbury on 2012-06-14

Tests for RepaintPending value

2397. By Sam Spilsbury on 2012-06-14

Test prohibited paint requestor retreival

2398. By Sam Spilsbury on 2012-06-17

Merge lp:unity

2399. By Sam Spilsbury on 2012-06-19

Merge lp:unity

2400. By Sam Spilsbury on 2012-06-19

Change namespace to unity::compiz

2401. By Sam Spilsbury on 2012-06-19

unsigned int -> enum && move namespace compiz {} to namespace unity::compiz

2402. By Sam Spilsbury on 2012-06-19

Nuke FBOBindingQueryInterface

2403. By Sam Spilsbury on 2012-06-19

Basic documentation, not really a substitute for better code ...

2404. By Sam Spilsbury on 2012-06-25

Merge lp:unity

2405. By Sam Spilsbury on 2012-06-26

Remove unused method and rename methods to make more sense

Unmerged revisions

2405. By Sam Spilsbury on 2012-06-26

Remove unused method and rename methods to make more sense

2404. By Sam Spilsbury on 2012-06-25

Merge lp:unity

2403. By Sam Spilsbury on 2012-06-19

Basic documentation, not really a substitute for better code ...

2402. By Sam Spilsbury on 2012-06-19

Nuke FBOBindingQueryInterface

2401. By Sam Spilsbury on 2012-06-19

unsigned int -> enum && move namespace compiz {} to namespace unity::compiz

2400. By Sam Spilsbury on 2012-06-19

Change namespace to unity::compiz

2399. By Sam Spilsbury on 2012-06-19

Merge lp:unity

2398. By Sam Spilsbury on 2012-06-17

Merge lp:unity

2397. By Sam Spilsbury on 2012-06-14

Test prohibited paint requestor retreival

2396. By Sam Spilsbury on 2012-06-14

Tests for RepaintPending value

=== modified file 'plugins/unityshell/src/unityshell.cpp'
--- plugins/unityshell/src/unityshell.cpp	2012-05-31 09:28:02 +0000
+++ plugins/unityshell/src/unityshell.cpp	2012-06-07 07:22:25 +0000
@@ -2127,31 +2127,6 @@
   return "Unity";
 }
 
-bool isNuxWindow (CompWindow* value)
-{
-  std::vector<Window> const& xwns = nux::XInputWindow::NativeHandleList();
-  auto id = value->id();
-
-  // iterate loop by hand rather than use std::find as this is considerably faster
-  // we care about performance here becuase of the high frequency in which this function is
-  // called (nearly every frame)
-  unsigned int size = xwns.size();
-  for (unsigned int i = 0; i < size; ++i)
-  {
-    if (xwns[i] == id)
-      return true;
-  }
-  return false;
-}
-
-const CompWindowList& UnityScreen::getWindowPaintList()
-{
-  CompWindowList& pl = _withRemovedNuxWindows = cScreen->getWindowPaintList();
-  pl.remove_if(isNuxWindow);
-
-  return pl;
-}
-
 void UnityScreen::RaiseInputWindows()
 {
   std::vector<Window> const& xwns = nux::XInputWindow::NativeHandleList();
@@ -2164,13 +2139,11 @@
   }
 }
 
-/* detect occlusions
- *
- * core passes down the PAINT_WINDOW_OCCLUSION_DETECTION
- * mask when it is doing occlusion detection, so use that
- * order to fill our occlusion buffer which we'll flip
- * to nux later
- */
+/* we want to paint underneath other windows here,
+ * so we need to find if this window is actually
+ * stacked on top of one of the nux input windows
+ * and if so paint nux and stop us from painting
+ * other windows or on top of the whole screen */
 bool UnityWindow::glPaint(const GLWindowPaintAttrib& attrib,
                           const GLMatrix& matrix,
                           const CompRegion& region,
@@ -2199,25 +2172,6 @@
     }
   }
 
-  return gWindow->glPaint(wAttrib, matrix, region, mask);
-}
-
-/* handle window painting in an opengl context
- *
- * we want to paint underneath other windows here,
- * so we need to find if this window is actually
- * stacked on top of one of the nux input windows
- * and if so paint nux and stop us from painting
- * other windows or on top of the whole screen */
-bool UnityWindow::glDraw(const GLMatrix& matrix,
-#ifndef USE_MODERN_COMPIZ_GL
-                         GLFragment::Attrib& attrib,
-#else
-                         const GLWindowPaintAttrib& attrib,
-#endif
-                         const CompRegion& region,
-                         unsigned int mask)
-{
   if (uScreen->doShellRepaint && !uScreen->paint_panel_ && window->type() == CompWindowTypeNormalMask)
   {
     guint32 id = window->id();
@@ -2234,30 +2188,39 @@
     }
   }
 
-  if (uScreen->doShellRepaint && !uScreen->forcePaintOnTop ())
+  std::vector<Window> const& xwns = nux::XInputWindow::NativeHandleList();
+  for (unsigned int i = 0; i < xwns.size(); i++)
   {
-    std::vector<Window> const& xwns = nux::XInputWindow::NativeHandleList();
-    unsigned int size = xwns.size();
-
-    for (CompWindow* w = window; w && uScreen->doShellRepaint; w = w->prev)
+    if (xwns[i] == window->id ())
     {
-      auto id = w->id();
-
-      for (unsigned int i = 0; i < size; ++i)
-      {
-        if (xwns[i] == id)
-        {
-#ifdef USE_MODERN_COMPIZ_GL
-          uScreen->paintDisplay();
+      if (uScreen->doShellRepaint &&
+	        !uScreen->forcePaintOnTop () &&
+	        !(mask & PAINT_WINDOW_OCCLUSION_DETECTION_MASK))
+#ifndef USE_MODERN_COMPIZ_GL
+	      uScreen->paintDisplay (region, matrix, mask);
 #else
-          uScreen->paintDisplay(region, matrix, mask);
+        uScreen->paintDisplay ();
 #endif
-          break;
-        }
-      }
+	    /* Skip to end and don't paint */
+	    gWindow->glPaintSetCurrentIndex (std::numeric_limits<unsigned int>::max ());
+	    return gWindow->glPaint (attrib, matrix, emptyRegion, mask | PAINT_WINDOW_NO_CORE_INSTANCE_MASK);
     }
   }
 
+  return gWindow->glPaint(wAttrib, matrix, region, mask);
+}
+
+/* handle window painting in an opengl context
+ */
+bool UnityWindow::glDraw(const GLMatrix& matrix,
+#ifndef USE_MODERN_COMPIZ_GL
+                         GLFragment::Attrib& attrib,
+#else
+                         const GLWindowPaintAttrib& attrib,
+#endif
+                         const CompRegion& region,
+                         unsigned int mask)
+{
   if (window->type() == CompWindowTypeDesktopMask)
     uScreen->setPanelShadowMatrix(matrix);
 

=== modified file 'plugins/unityshell/src/unityshell.h'
--- plugins/unityshell/src/unityshell.h	2012-05-28 03:19:35 +0000
+++ plugins/unityshell/src/unityshell.h	2012-06-07 07:22:25 +0000
@@ -132,9 +132,6 @@
                                 CompOutput*,
                                 unsigned int);
 
-  /* Pop our InputOutput windows from the paint list */
-  const CompWindowList& getWindowPaintList();
-
   /* handle X11 events */
   void handleEvent(XEvent*);
 
@@ -279,7 +276,6 @@
   bool    damaged;
   bool    _key_nav_mode_requested;
   CompOutput* _last_output;
-  CompWindowList _withRemovedNuxWindows;
 
   nux::Property<nux::Geometry> primary_monitor_;