Re: [PATCH 7/7] compositor: implement a state machine for display control

2012-02-28 Thread Andreas Ericsson
On 02/24/2012 08:02 PM, Bill Spitzak wrote: Tiago Vignatti wrote: DISPLAY_ON → DISPLAY_DIM → DISPLAY_SCREENSAVER → DISPLAY_OFF Won't there be some interest in making the backlight dim *after* the screensaver starts? I can imagine the control ui being a bunch of timing sliders, which the

[fullscreen v8] add fullscreen implementation and a test client

2012-02-28 Thread wuzhiwen
From: Alex Wu zhiwen...@linux.intel.com V7: Fullscreen surface will be atop panels and with a black surface underlying it. Only the WL_SHELL_SURFACE_FULLSCREEN_METHOD_SCALE method implemented in this version. We will implement other methods in another patch. V8: Move all the

[fullscreen-v8 PATCH 1/3] compositor: Export functions and add weston_suface::force_configure for fullscreen

2012-02-28 Thread zhiwen . wu
From: Alex Wu zhiwen...@linux.intel.com This will facilitate the implementation of fullscreen. Signed-off-by: Alex Wu zhiwen...@linux.intel.com --- src/compositor.c | 16 ++-- src/compositor.h |9 + 2 files changed, 23 insertions(+), 2 deletions(-) diff --git

[fullscreen-v8 PATCH 2/3] shell: Add implementation of fullscreen.

2012-02-28 Thread zhiwen . wu
From: Alex Wu zhiwen...@linux.intel.com All the fullscreen things (black surface, raise atop panels, transform, positioning) are handled in map() or configure(). Signed-off-by: Alex Wu zhiwen...@linux.intel.com Signed-off-by: Juan Zhao juan.j.z...@linux.intel.com --- src/shell.c | 231

[RFC] compositor: add dpms and backlight support

2012-02-28 Thread Tiago Vignatti
- hotplug doesn't work; it's easy fix though. - DPMS off only kicks in when wscreensaver is launched, cause then the compositor will trigger the second call to lock(), inside shell. It works anyway with the current wscreensaver broken. - for backlight, we're using the changes I've done here:

Re: [fullscreen-v8 PATCH 1/3] compositor: Export functions and add weston_suface::force_configure for fullscreen

2012-02-28 Thread Kristian Hoegsberg
On Tue, Feb 28, 2012 at 06:07:54PM +0800, zhiwen...@linux.intel.com wrote: From: Alex Wu zhiwen...@linux.intel.com Thanks, applied with a little edit to make the check in weston_surface_destroy an assert instead. If we end up in that case, it's an internal error and we need to catch it, not

Re: [PATCH RFC] protocol: add xslt stylesheet to prettify the protocol

2012-02-28 Thread Kristian Høgsberg
2012/2/25 Peter Hutterer peter.hutte...@who-t.net: On 25/02/12 01:04 , Kristian Høgsberg wrote: On Fri, Feb 24, 2012 at 2:42 AM, Peter Hutterer peter.hutte...@who-t.net  wrote: On Thu, Feb 23, 2012 at 04:26:23PM -0500, Kristian Hoegsberg wrote: On Thu, Feb 23, 2012 at 03:58:37PM +1000,

one drag-n-drop corner case

2012-02-28 Thread Nick Shaforostoff
Hi. I stumbled upon problems (including crashes) with drag-n-drop in X11 in case when a window disappears while dragging is being done (for example a program exits). https://bugs.kde.org/show_bug.cgi?id=192270 So I thought it would be useful for you to have such corner case in mind when you

[PATCH] compositor: make the input region of drag surfaces empty

2012-02-28 Thread Ander Conselvan de Oliveira
With the input region changes, drag surfaces were being picked and receiving events which led to problems on the client side. --- src/compositor.c | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/src/compositor.c b/src/compositor.c index b12c583..eabb34f

Re: [PATCH] protocol: Clarify the documentation for the fullscreen protocol

2012-02-28 Thread Rob Bradford
Juan, Does this align with what you intended in this request? I think it is a bit clearer now and resolves some of the items we discussed at the last hackfest and also on IRC. Cheers, Rob On 28 February 2012 16:48, Rob Bradford r...@robster.org.uk wrote: From: Rob Bradford

Re: [RFC] compositor: add dpms and backlight support

2012-02-28 Thread Jesse Barnes
On Tue, 28 Feb 2012 13:52:21 +0200 Tiago Vignatti tiago.vigna...@intel.com wrote: @@ -907,11 +912,76 @@ sprite_handle_pending_buffer_destroy(struct wl_listener *listener, sprite-pending_surface = NULL; } +static void +drm_set_backlight(struct weston_output *output_base, uint32_t

Re: [RFC] compositor: add dpms and backlight support

2012-02-28 Thread Kristian Høgsberg
On Tue, Feb 28, 2012 at 1:19 PM, Jesse Barnes jbar...@virtuousgeek.org wrote: On Tue, 28 Feb 2012 13:52:21 +0200 Tiago Vignatti tiago.vigna...@intel.com wrote: @@ -907,11 +912,76 @@ sprite_handle_pending_buffer_destroy(struct wl_listener *listener,       sprite-pending_surface = NULL;  }

Re: [PATCH] compositor: make the input region of drag surfaces empty

2012-02-28 Thread Kristian Hoegsberg
On Tue, Feb 28, 2012 at 05:59:33PM +0200, Ander Conselvan de Oliveira wrote: With the input region changes, drag surfaces were being picked and receiving events which led to problems on the client side. --- src/compositor.c | 16 1 files changed, 16 insertions(+), 0

Re: [fullscreen-v8 PATCH 2/3] shell: Add implementation of fullscreen.

2012-02-28 Thread Kristian Hoegsberg
On Tue, Feb 28, 2012 at 06:08:38PM +0800, zhiwen...@linux.intel.com wrote: From: Alex Wu zhiwen...@linux.intel.com All the fullscreen things (black surface, raise atop panels, transform, positioning) are handled in map() or configure(). Ok, looking good, we're almost there now :) A few

Re: [RFC] [PATCH] Renamed client side wl_display_destroy() to wl_display_disconnect()

2012-02-28 Thread Kristian Hoegsberg
On Mon, Feb 27, 2012 at 05:10:03PM +0100, Samuel Rødal wrote: The only problem I see with this is that the disconnect naming can be confusing as the wl_display handle is actually freed as well. I like it, it pairs better with wl_display_connect. I don't think it's a problem that the name is

Re: [PATCH] Allow update function to not be set in wl_display_get_fd

2012-02-28 Thread Kristian Hoegsberg
On Mon, Feb 27, 2012 at 04:57:42PM +0100, Samuel Rødal wrote: Ignore previous patch, here's the correct version. From 4e1bedaaf05b576f5191f8fe3a34904ab9707414 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Samuel=20R=C3=B8dal?= samuel.ro...@nokia.com Date: Mon, 27 Feb 2012 15:17:20 +0100 Subject:

[PATCH] protocol: fix make distcheck

2012-02-28 Thread Peter Hutterer
Distribute all source files that we need for buildling. Plus, remove the html file on make clean. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- protocol/Makefile.am |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/protocol/Makefile.am

[PATCH 3/6] doc: Comment out some of the default chapters

2012-02-28 Thread Peter Hutterer
I'll leave them in for now as a template for how things looked originally, this can be removed later. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- doc/Wayland/en-US/Wayland.xml |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git

[PATCH 4/6] Hook up autotools for publican

2012-02-28 Thread Peter Hutterer
automake doesn't seem to provide a sensible method to install a directory of stuff in $(docdir). Do it manually then. CC: Gaetan Nadon gaetan.na...@videotron.ca Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- Gaetan - can you check this please, it seems less-than-ideal, especially

[PATCH 5/6] Generate the docbook description for the protocol from wayland.xml

2012-02-28 Thread Peter Hutterer
Convert the wayland.xml protocol description to a docbook-compatible format and hook it up to the publican sources. Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- doc/Wayland/.gitignore |1 + doc/Wayland/Makefile.am |9 ++-

[PATCH 6/6] Hack up distcheck for publican

2012-02-28 Thread Peter Hutterer
Publican requires a read-write source tree, see http://bugzilla.redhat.com/show_bug.cgi?id=798484 And it currently cannot build out-of-tree, so we need to copy the sources into the _build tree and generate Protocol.xml into that tree too (we'd have to do this anyway since automake creates a

Re: [PATCH] protocol: fix make distcheck

2012-02-28 Thread Christopher Michael
On 02/28/2012 09:50 PM, Peter Hutterer wrote: Distribute all source files that we need for buildling. Plus, remove the html file on make clean. Signed-off-by: Peter Huttererpeter.hutte...@who-t.net --- protocol/Makefile.am |7 --- 1 files changed, 4 insertions(+), 3 deletions(-)

Re: [PATCH] protocol: Clarify the documentation for the fullscreen protocol

2012-02-28 Thread Pekka Paalanen
On Tue, 28 Feb 2012 16:48:26 + Rob Bradford r...@robster.org.uk wrote: From: Rob Bradford r...@linux.intel.com --- protocol/wayland.xml | 53 - 1 files changed, 34 insertions(+), 19 deletions(-) diff --git a/protocol/wayland.xml