Merge lp:~fboucault/unity-2d/nautilus_window_instead_of_wallpaper into lp:unity-2d

[Florian Boucault]

[workspace switcher] Draw Nautilus desktop window's screenshot instead of loading the GNOME wallpaper.

Greatly improves startup speed therefore contributing to fixing https://bugs.launchpad.net/unity-2d/+bug/745764

The following changes were necessary to have the desktop window screenshot taken only when necessary:
- Defined new 'visible' property on Unity2dDeclarativeView.
- Cleaned up descendant classes handling of hideEvent and showEvent.

[Florian Boucault (fboucault)]

Do not merge yet as the screenshoting of Nautilus is done as many times as we have desktops (taking in average 35ms on my laptop).

[Florian Boucault (fboucault)]

review: Needs Fixing

[Florian Boucault (fboucault)]

https://bugs.launchpad.net/unity-2d/+bug/900019 should also be fixed with this MR

[Florian Boucault (fboucault)]

> Do not merge yet as the screenshoting of Nautilus is done as many times as we
> have desktops (taking in average 35ms on my laptop).

A cache based on timestamp should solve that.

[lp:~fboucault/unity-2d/nautilus_window_instead_of_wallpaper]

832. By Andrea Cimitan on 2011-12-19

Some fixes to dash layout, especially on the filters pane. Fixes: https://bugs.launchpad.net/bugs/906235. Appoved by .

833. By Albert Astals Cid on 2011-12-20

[launcher] Move the RMB menu from clicked to pressed

Fixes LP bug #813036
. Fixes: https://bugs.launchpad.net/bugs/813036. Appoved by Lohith D Shivamurthy.

834. By Tiago Salem Herrmann on 2011-12-21

This merge request intends to fix lp:838708, making BFB behavior and visual match the design.. Fixes: https://bugs.launchpad.net/bugs/838708. Appoved by Lohith D Shivamurthy.

835. By Gerry Boland on 2012-01-02

[tests] Update visual asset of BFB - UNBLOCK. Fixes: . Appoved by .

836. By Albert Astals Cid on 2012-01-03

Use QScopedPointers to properly delete the lists returned from qtbamf. Fixes: . Appoved by .

837. By Albert Astals Cid on 2012-01-03

Reset window to NULL if it's not the one we are looking for

Otherwise in case we don't find the one with the xid we want the function returns the pointer of the last window of the list
. Fixes: . Appoved by .

838. By Albert Astals Cid on 2012-01-03

Do not invoke m_application->windows() multiple times

Two reasons for that:
 * the result of m_application->windows() needs to be deleted and if we call it in the for loop we can't
 * The size of the list itself can change (and more critically shrink) between calls ending up in a call to an invalid index (Bug 854634)
. Fixes: https://bugs.launchpad.net/bugs/854634. Appoved by Gerry Boland, Lohith D Shivamurthy.

839. By Tiago Salem Herrmann on 2012-01-03

The request to open the command lens (alt+f2) happens before the list of available lenses is properly populated when unity-2d-places is started by a dbus call.
This patch adds a simple "queue" that checks if the requested queue is available everytime a new lens is added to the "lenses" variable. Fixes: https://bugs.launchpad.net/bugs/883392. Appoved by Gerry Boland, Lohith D Shivamurthy.

840. By Michał Sawicz on 2012-01-04

[debian] package debug symbols. Fixes: . Appoved by Gerry Boland.

841. By Tiago Salem Herrmann on 2012-01-04

[dash] Remove underline from "Filter results" when it is highlighted. Fixes: https://bugs.launchpad.net/bugs/893061. Appoved by Gerry Boland.

842. By Gerry Boland on 2012-01-05

[test] [xdo] Ensure WM controls work by checking metacity's settings. Fixes: . Appoved by Lohith D Shivamurthy.

843. By Gerry Boland on 2012-01-05

[test] Fix Xdo to properly send function keys and re-enable Alt+F1 focus test. Fixes: . Appoved by Lohith D Shivamurthy.

844. By Didier Roche-Tolomelli on 2012-01-10

Dummy commit to rebuild against latest libunity and unity packaging
(libunity ABI bump) UNBLOCK. Fixes: . Appoved by Aurélien Gâteau.

845. By Aurélien Gâteau on 2012-01-12

* debian/control:
  - bump build-dep versions.

846. By Aurélien Gâteau on 2012-01-12

releasing version 5.2.0-0ubuntu1

847. By Didier Roche-Tolomelli on 2012-01-12

post-changelog edit

848. By Didier Roche-Tolomelli on 2012-01-13

remove TODO

849. By Albert Astals Cid on 2012-01-17

[test] Check that F1 toggles the launcher and brings the focus to the previous active window
. Fixes: . Appoved by Gerry Boland.

850. By Albert Astals Cid on 2012-01-17

[test] Check that showing the desktop shows also the launcher
. Fixes: . Appoved by .

851. By Gerry Boland on 2012-01-18

Rewrite launcher autohide_show_tests to use the verify* methods. Also add TmpWindow class manage creating and closing windows needed for tests.

Replacing the assert* with verify* methods mean we can remove the arbitrary sleep times, which means the test suite will run more smoothly (and faster!). The TmpWindow class provides book-keeping so that all created windows can be closed by the test teardown method, so that one failed test won't cause other tests to fail.. Fixes: . Appoved by Albert Astals Cid.

852. By Albert Astals Cid on 2012-01-18

Do not use "" to assign to QStrings, use QString() instead that is much faster or do not do nothing if they were just declared. Fixes: . Appoved by .

853. By Albert Astals Cid on 2012-01-18

Add some const &

It is true that those classes are refcounted so the win is not huge, but there's no reason not to save some locks in some atomic integers used for the ref/unrefing. Fixes: . Appoved by Gerry Boland.

854. By Albert Astals Cid on 2012-01-18

Add two more files to .bzrignore. Fixes: . Appoved by Gerry Boland.

855. By Albert Astals Cid on 2012-01-18

Wait for the Quit event to be delivered before really quitting

I think this is a problem either in Qt or in gnome-session, but we can "easily" workaround it ourselves, and could not find how to fix it "properly" so this seems like an acceptable workaround to me.. Fixes: https://bugs.launchpad.net/bugs/812104. Appoved by Gerry Boland.

856. By Albert Astals Cid on 2012-01-18

[test] Small improvements to autohide tests regarding timing on mouse moves
. Fixes: . Appoved by Gerry Boland.

857. By Albert Astals Cid on 2012-01-19

Remove Places related code as it is not used anymore. Fixes: . Appoved by Gerry Boland.

858. By Albert Astals Cid on 2012-01-19

[test] Check that the launcher is around for 1 second after removing an application tile
. Fixes: . Appoved by Michał Sawicz.

859. By Florian Boucault on 2012-01-21

Defined new 'visible' property on Unity2dDeclarativeView.
Cleaned up descendant classes handling of hideEvent and showEvent.

860. By Florian Boucault on 2012-01-21

[workspace switcher] Draw Nautilus desktop window's screenshot instead of loading the GNOME wallpaper.

861. By Florian Boucault on 2012-01-23

Added FIXME.

862. By Florian Boucault on 2012-01-26

Merged lp:unity-2d

863. By Florian Boucault on 2012-01-26

Fixes bug #900019

[Gerry Boland (gerboland)]

This improves the Spread visuals a lot.

It looks less good on multimonitor (the whole desktop of all screens is squeezed into each desktop), but as spread requires lots of MM work, I will accept this.

review: Approve

[Florian Boucault (fboucault)]

Thanks Gerry. There is still work to do on this MR though, work highlighted above. In summary:
* FIXME: Nautilus' window pixmap is reloaded as many times as there is workspaces
* writing automated tests:
  - make sure nautilus' pixmap is loaded only once
  - make sure nautilus XID retrieval works

[Florian Boucault (fboucault)]

> This improves the Spread visuals a lot.
>
> It looks less good on multimonitor (the whole desktop of all screens is
> squeezed into each desktop), but as spread requires lots of MM work, I will
> accept this.

Can you post a screenshot of what you described here. I sense that this is not acceptable as it would be a regression of what we currently have.

[Gerry Boland (gerboland)]

MM Screengrab: https://imgur.com/I4OVY

Yes it is a regression of what we have right now.

Since you plan to continue working on this branch, can you please mark it as Work in Progress until it's ready for review again?

Unmerged revisions

863. By Florian Boucault on 2012-01-26

Fixes bug #900019

862. By Florian Boucault on 2012-01-26

Merged lp:unity-2d

861. By Florian Boucault on 2012-01-23

Added FIXME.

860. By Florian Boucault on 2012-01-21

[workspace switcher] Draw Nautilus desktop window's screenshot instead of loading the GNOME wallpaper.

859. By Florian Boucault on 2012-01-21

Defined new 'visible' property on Unity2dDeclarativeView.
Cleaned up descendant classes handling of hideEvent and showEvent.

=== modified file 'libunity-2d-private/Unity2d/plugin.cpp'
--- libunity-2d-private/Unity2d/plugin.cpp	2012-01-24 16:46:52 +0000
+++ libunity-2d-private/Unity2d/plugin.cpp	2012-01-26 22:03:24 +0000
@@ -36,6 +36,7 @@
 #include "windowinfo.h"
 #include "windowslist.h"
 #include "screeninfo.h"
+#include "rootwindowinfo.h"
 #include "plugin.h"
 #include "cacheeffect.h"
 #include "iconutilities.h"
@@ -102,6 +103,7 @@
     qmlRegisterType<WindowsList>(uri, 0, 1, "WindowsList");
     qmlRegisterType<ScreenInfo>(); // Register the type as non creatable
     qmlRegisterType<WorkspacesInfo>(); // Register the type as non creatable
+    qmlRegisterType<RootWindowInfo>(); // Register the type as non creatable
 
     qmlRegisterType<CacheEffect>(uri, 0, 1, "CacheEffect");
     qmlRegisterType<QGraphicsBlurEffect>("Effects", 1, 0, "Blur");
@@ -175,6 +177,7 @@
        not creatable directly in QML */
     engine->rootContext()->setContextProperty("screen", ScreenInfo::instance());
     engine->rootContext()->setContextProperty("iconUtilities", new IconUtilities(engine));
+    engine->rootContext()->setContextProperty("rootWindow", RootWindowInfo::instance());
 
     /* Expose QConf objects as a context property not to initialize it multiple times */
     engine->rootContext()->setContextProperty("dash2dConfiguration", &dash2dConfiguration());
 
=== modified file 'libunity-2d-private/src/CMakeLists.txt'
--- libunity-2d-private/src/CMakeLists.txt	2012-01-24 11:50:38 +0000
+++ libunity-2d-private/src/CMakeLists.txt	2012-01-26 22:03:24 +0000
@@ -34,6 +34,7 @@
     windowinfo.cpp
     windowslist.cpp
     screeninfo.cpp
+    rootwindowinfo.cpp
     cacheeffect.cpp
     workspacesinfo.cpp
     signalwaiter.cpp
 
=== added file 'libunity-2d-private/src/rootwindowinfo.cpp'
--- libunity-2d-private/src/rootwindowinfo.cpp	1970-01-01 00:00:00 +0000
+++ libunity-2d-private/src/rootwindowinfo.cpp	2012-01-26 22:03:24 +0000
@@ -0,0 +1,121 @@
+/*
+ * Copyright (C) 2011 Canonical, Ltd.
+ *
+ * Authors:
+ *  Florian Boucault <[e-mail]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include "rootwindowinfo.h"
+
+#include <QX11Info>
+#include <QAbstractEventDispatcher>
+
+#include <X11/Xlib.h>
+#include <X11/Xatom.h>
+
+
+QAbstractEventDispatcher::EventFilter RootWindowInfo::s_oldEventFilter = NULL;
+Atom RootWindowInfo::NAUTILUS_DESKTOP_WINDOW_ID = 0;
+
+RootWindowInfo::RootWindowInfo(QObject *parent) :
+    QObject(parent),
+    m_nautilusDesktopWindowId(0)
+{
+    RootWindowInfo::internX11Atoms();
+
+    /* Setup a low-level event filter to receive X11 events directly,
+       then ask X11 to notify us of property changes on the root window.
+    */
+    QAbstractEventDispatcher *dispatcher = QAbstractEventDispatcher::instance();
+    RootWindowInfo::s_oldEventFilter = dispatcher->setEventFilter(RootWindowInfo::globalEventFilter);
+    XSelectInput(QX11Info::display(), QX11Info::appRootWindow(), PropertyChangeMask);
+
+    /* Initialize cached values of properties */
+    updateNautilusDesktopWindowId();
+}
+
+RootWindowInfo* RootWindowInfo::instance()
+{
+    static RootWindowInfo* singleton = new RootWindowInfo();
+    return singleton;
+}
+
+void RootWindowInfo::internX11Atoms()
+{
+    NAUTILUS_DESKTOP_WINDOW_ID = XInternAtom(QX11Info::display(),
+                                             "NAUTILUS_DESKTOP_WINDOW_ID",
+                                             False);
+}
+
+bool RootWindowInfo::globalEventFilter(void* message)
+{
+    /* Always call the old event handler before we do our thing, and
+       make sure to always return what it did return, as we don't really
+       "handle" any event, we just monitor them */
+    bool result;
+    result = (RootWindowInfo::s_oldEventFilter != NULL) ? RootWindowInfo::s_oldEventFilter(message) : false;
+
+    XEvent* event = static_cast<XEvent*>(message);
+    if (event->type == PropertyNotify) {
+        if (event->xproperty.atom == NAUTILUS_DESKTOP_WINDOW_ID) {
+            RootWindowInfo::instance()->updateNautilusDesktopWindowId();
+        }
+    }
+
+    return result;
+}
+
+bool RootWindowInfo::getX11Property(Atom property, long long_length, Atom req_type, unsigned long &value)
+{
+    Atom actual_type_return;
+    int actual_format_return;
+    unsigned long nitems_return;
+    unsigned long bytes_after_return;
+    unsigned char *prop_return;
+    int result;
+
+    result = XGetWindowProperty(QX11Info::display(), QX11Info::appRootWindow(),
+                                property,
+                                0, long_length, False, req_type,
+                                &actual_type_return, &actual_format_return, &nitems_return,
+                                &bytes_after_return, &prop_return);
+    if (result == Success && actual_type_return != None) {
+        value = ((unsigned long*) prop_return)[0];
+        XFree(prop_return);
+        return true;
+    }
+
+    return false;
+}
+
+void RootWindowInfo::updateNautilusDesktopWindowId()
+{
+    unsigned long value = 0;
+
+    getX11Property(NAUTILUS_DESKTOP_WINDOW_ID, 1, XA_WINDOW, value);
+
+    if (m_nautilusDesktopWindowId != value) {
+        m_nautilusDesktopWindowId = value;
+        Q_EMIT nautilusDesktopWindowIdChanged();
+    }
+}
+
+unsigned long RootWindowInfo::nautilusDesktopWindowId() const
+{
+    return m_nautilusDesktopWindowId;
+}
+
+
+#include "rootwindowinfo.moc"
 
=== added file 'libunity-2d-private/src/rootwindowinfo.h'
--- libunity-2d-private/src/rootwindowinfo.h	1970-01-01 00:00:00 +0000
+++ libunity-2d-private/src/rootwindowinfo.h	2012-01-26 22:03:24 +0000
@@ -0,0 +1,57 @@
+/*
+ * Copyright (C) 2011 Canonical, Ltd.
+ *
+ * Authors:
+ *  Florian Boucault <[e-mail]>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#ifndef ROOTWINDOWINFO_H
+#define ROOTWINDOWINFO_H
+
+#include <QObject>
+#include <QAbstractEventDispatcher>
+
+typedef unsigned long Atom;
+
+class RootWindowInfo : public QObject
+{
+    Q_OBJECT
+
+    Q_PROPERTY(unsigned long nautilusDesktopWindowId READ nautilusDesktopWindowId
+                                                     NOTIFY nautilusDesktopWindowIdChanged)
+public:
+    static RootWindowInfo* instance();
+
+    /* getters */
+    unsigned long nautilusDesktopWindowId() const;
+
+Q_SIGNALS:
+    void nautilusDesktopWindowIdChanged();
+
+private:
+    explicit RootWindowInfo(QObject *parent = 0);
+
+    static void internX11Atoms();
+    static bool globalEventFilter(void* message);
+    static bool getX11Property(Atom property, long long_length, Atom req_type, unsigned long &value);
+
+    void updateNautilusDesktopWindowId();
+
+    static QAbstractEventDispatcher::EventFilter s_oldEventFilter;
+    static Atom NAUTILUS_DESKTOP_WINDOW_ID;
+    unsigned long m_nautilusDesktopWindowId;
+};
+
+#endif // ROOTWINDOWINFO_H
 
=== modified file 'libunity-2d-private/src/unity2ddeclarativeview.cpp'
--- libunity-2d-private/src/unity2ddeclarativeview.cpp	2011-12-08 19:41:38 +0000
+++ libunity-2d-private/src/unity2ddeclarativeview.cpp	2012-01-26 22:03:24 +0000
@@ -158,6 +158,17 @@
     Q_EMIT globalPositionChanged(globalPosition());
 }
 
+void Unity2DDeclarativeView::showEvent(QShowEvent* event)
+{
+    QDeclarativeView::showEvent(event);
+    Q_EMIT visibleChanged(true);
+}
+
+void Unity2DDeclarativeView::hideEvent(QHideEvent* event)
+{
+    QDeclarativeView::hideEvent(event);
+    Q_EMIT visibleChanged(false);
+}
 
 /* Obtaining & Discarding Keyboard Focus for Window on Demand
  *
 
=== modified file 'libunity-2d-private/src/unity2ddeclarativeview.h'
--- libunity-2d-private/src/unity2ddeclarativeview.h	2011-12-08 19:41:38 +0000
+++ libunity-2d-private/src/unity2ddeclarativeview.h	2012-01-26 22:03:24 +0000
@@ -26,6 +26,7 @@
     Q_PROPERTY(bool useOpenGL READ useOpenGL WRITE setUseOpenGL NOTIFY useOpenGLChanged)
     Q_PROPERTY(bool transparentBackground READ transparentBackground WRITE setTransparentBackground NOTIFY transparentBackgroundChanged)
     Q_PROPERTY(QPoint globalPosition READ globalPosition NOTIFY globalPositionChanged)
+    Q_PROPERTY(bool visible READ isVisible NOTIFY visibleChanged)
 
 public:
     Unity2DDeclarativeView(QWidget *parent = 0);
@@ -44,10 +45,13 @@
     void useOpenGLChanged(bool);
     void transparentBackgroundChanged(bool);
     void globalPositionChanged(QPoint);
+    void visibleChanged(bool);
 
 protected:
     void setupViewport();
     virtual void moveEvent(QMoveEvent* event);
+    virtual void showEvent(QShowEvent* event);
+    virtual void hideEvent(QHideEvent* event);
 
 protected Q_SLOTS:
     void forceActivateWindow();
 
=== modified file 'places/app/dashdeclarativeview.cpp'
--- places/app/dashdeclarativeview.cpp	2012-01-26 13:12:12 +0000
+++ places/app/dashdeclarativeview.cpp	2012-01-26 22:03:24 +0000
@@ -134,7 +134,7 @@
 void
 DashDeclarativeView::showEvent(QShowEvent *event)
 {
-    QDeclarativeView::showEvent(event);
+    Unity2DDeclarativeView::showEvent(event);
     /* Note that this has to be called everytime the window is shown, as the WM
        will remove the flags when the window is hidden */
     setWMFlags();
 
=== modified file 'spread/Workspace.qml'
--- spread/Workspace.qml	2011-10-21 12:46:08 +0000
+++ spread/Workspace.qml	2012-01-26 22:03:24 +0000
@@ -16,7 +16,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-import QtQuick 1.0
+import QtQuick 1.1
 import "utils.js" as Utils
 import Unity2d 1.0
 
@@ -47,15 +47,26 @@
         height: switcher.height
         state: parent.state
 
-        GnomeBackground {
-            anchors.fill: parent
-            overlay_color: "black"
-            overlay_alpha: 0
-
-            clip: true
-            cached: false
-            offsetX: -screen.panelsFreeGeometry.x
-            offsetY: -screen.panelsFreeGeometry.y
+        clip: true
+
+        Image {
+            x: -declarativeView.globalPosition.x
+            y: -declarativeView.globalPosition.y
+            width: screen.geometry.width
+            height: screen.geometry.height
+
+            smooth: true
+            /* The window is scaled to a rectangle as large as possible inside
+               sourceSize while preserving its aspect ratio.
+               It saves video memory when using the OpenGL backend.
+               It makes scaling cheaper in the spread when using the raster backend.
+            */
+            sourceSize.width: 512
+            sourceSize.height: 512
+
+            source: declarativeView.visible ? "image://window/" + rootWindow.nautilusDesktopWindowId : ""
+            /* FIXME: Nautilus' window pixmap is reloaded as many times as there is workspaces */
+            cache: false
         }
 
         Windows {
 
=== modified file 'spread/app/spreadview.cpp'
--- spread/app/spreadview.cpp	2011-12-08 18:39:18 +0000
+++ spread/app/spreadview.cpp	2012-01-26 22:03:24 +0000
@@ -97,15 +97,3 @@
 
     return false;
 }
-
-void SpreadView::showEvent(QShowEvent *event)
-{
-    Q_UNUSED(event);
-    Q_EMIT visibleChanged(true);
-}
-
-void SpreadView::hideEvent(QHideEvent *event)
-{
-    Q_UNUSED(event);
-    Q_EMIT visibleChanged(false);
-}
 
=== modified file 'spread/app/spreadview.h'
--- spread/app/spreadview.h	2011-12-08 18:39:18 +0000
+++ spread/app/spreadview.h	2012-01-26 22:03:24 +0000
@@ -39,13 +39,10 @@
 protected:
     virtual void focusInEvent( QFocusEvent * event );
     virtual void focusOutEvent( QFocusEvent * event );
-    virtual void showEvent(QShowEvent *event);
-    virtual void hideEvent(QHideEvent *event);
     bool eventFilter(QObject *obj, QEvent *event);
 
 Q_SIGNALS:
     void outsideClick();
-    void visibleChanged(bool visible);
 };
 
 #endif // SPREADVIEW_H