qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 3/6] gtk: trace input grab reason


From: Marc-André Lureau
Subject: Re: [Qemu-devel] [PATCH 3/6] gtk: trace input grab reason
Date: Wed, 9 Sep 2015 16:53:57 +0200

On Wed, Sep 9, 2015 at 11:28 AM, Gerd Hoffmann <address@hidden> wrote:
> Add a reason to grab calls and trace points,
> so it is easier to debug grab related ui issues.
>
> Signed-off-by: Gerd Hoffmann <address@hidden>
> ---
>  trace-events |  3 ++-
>  ui/gtk.c     | 26 +++++++++++++-------------
>  2 files changed, 15 insertions(+), 14 deletions(-)
>

While at it, it doesn't make sense to add reasons for ungrab too?
could be done later if needed, so

Reviewed-by: Marc-André Lureau <address@hidden>

> diff --git a/trace-events b/trace-events
> index 0a82f0c..d6d1820 100644
> --- a/trace-events
> +++ b/trace-events
> @@ -1143,7 +1143,8 @@ ppm_save(const char *filename, void *display_surface) 
> "%s surface=%p"
>  gd_switch(const char *tab, int width, int height) "tab=%s, width=%d, 
> height=%d"
>  gd_update(const char *tab, int x, int y, int w, int h) "tab=%s, x=%d, y=%d, 
> w=%d, h=%d"
>  gd_key_event(const char *tab, int gdk_keycode, int qemu_keycode, const char 
> *action) "tab=%s, translated GDK keycode %d to QEMU keycode %d (%s)"
> -gd_grab(const char *tab, const char *device, bool on) "tab=%s, %s %d"
> +gd_grab(const char *tab, const char *device, const char *reason) "tab=%s, 
> dev=%s, reason=%s"
> +gd_ungrab(const char *tab, const char *device) "tab=%s, dev=%s"
>
>  # ui/vnc.c
>  vnc_key_guest_leds(bool caps, bool num, bool scroll) "caps %d, num %d, 
> scroll %d"
> diff --git a/ui/gtk.c b/ui/gtk.c
> index 5f87475..322d112 100644
> --- a/ui/gtk.c
> +++ b/ui/gtk.c
> @@ -165,9 +165,9 @@ struct GtkDisplayState {
>      bool ignore_keys;
>  };
>
> -static void gd_grab_pointer(VirtualConsole *vc);
> +static void gd_grab_pointer(VirtualConsole *vc, const char *reason);
>  static void gd_ungrab_pointer(GtkDisplayState *s);
> -static void gd_grab_keyboard(VirtualConsole *vc);
> +static void gd_grab_keyboard(VirtualConsole *vc, const char *reason);
>  static void gd_ungrab_keyboard(GtkDisplayState *s);
>
>  /** Utility Functions **/
> @@ -855,7 +855,7 @@ static gboolean gd_button_event(GtkWidget *widget, 
> GdkEventButton *button,
>              gtk_check_menu_item_set_active(GTK_CHECK_MENU_ITEM(s->grab_item),
>                                             TRUE);
>          } else {
> -            gd_grab_pointer(vc);
> +            gd_grab_pointer(vc, "relative-mode-click");
>          }
>          return TRUE;
>      }
> @@ -1092,7 +1092,7 @@ static gboolean gd_win_grab(void *opaque)
>      if (vc->s->ptr_owner) {
>          gd_ungrab_pointer(vc->s);
>      } else {
> -        gd_grab_pointer(vc);
> +        gd_grab_pointer(vc, "user-request-detached-tab");
>      }
>      return TRUE;
>  }
> @@ -1256,7 +1256,7 @@ static void gd_grab_devices(VirtualConsole *vc, bool 
> grab,
>  }
>  #endif
>
> -static void gd_grab_keyboard(VirtualConsole *vc)
> +static void gd_grab_keyboard(VirtualConsole *vc, const char *reason)
>  {
>      if (vc->s->kbd_owner) {
>          if (vc->s->kbd_owner == vc) {
> @@ -1277,7 +1277,7 @@ static void gd_grab_keyboard(VirtualConsole *vc)
>  #endif
>      vc->s->kbd_owner = vc;
>      gd_update_caption(vc->s);
> -    trace_gd_grab(vc->label, "kbd", true);
> +    trace_gd_grab(vc->label, "kbd", reason);
>  }
>
>  static void gd_ungrab_keyboard(GtkDisplayState *s)
> @@ -1295,10 +1295,10 @@ static void gd_ungrab_keyboard(GtkDisplayState *s)
>      gdk_keyboard_ungrab(GDK_CURRENT_TIME);
>  #endif
>      gd_update_caption(s);
> -    trace_gd_grab(vc->label, "kbd", false);
> +    trace_gd_ungrab(vc->label, "kbd");
>  }
>
> -static void gd_grab_pointer(VirtualConsole *vc)
> +static void gd_grab_pointer(VirtualConsole *vc, const char *reason)
>  {
>      GdkDisplay *display = gtk_widget_get_display(vc->gfx.drawing_area);
>
> @@ -1337,7 +1337,7 @@ static void gd_grab_pointer(VirtualConsole *vc)
>  #endif
>      vc->s->ptr_owner = vc;
>      gd_update_caption(vc->s);
> -    trace_gd_grab(vc->label, "ptr", true);
> +    trace_gd_grab(vc->label, "ptr", reason);
>  }
>
>  static void gd_ungrab_pointer(GtkDisplayState *s)
> @@ -1363,7 +1363,7 @@ static void gd_ungrab_pointer(GtkDisplayState *s)
>                               vc->s->grab_x_root, vc->s->grab_y_root);
>  #endif
>      gd_update_caption(s);
> -    trace_gd_grab(vc->label, "ptr", false);
> +    trace_gd_ungrab(vc->label, "ptr");
>  }
>
>  static void gd_menu_grab_input(GtkMenuItem *item, void *opaque)
> @@ -1372,8 +1372,8 @@ static void gd_menu_grab_input(GtkMenuItem *item, void 
> *opaque)
>      VirtualConsole *vc = gd_vc_find_current(s);
>
>      if (gd_is_grab_active(s)) {
> -        gd_grab_keyboard(vc);
> -        gd_grab_pointer(vc);
> +        gd_grab_keyboard(vc, "user-request-main-window");
> +        gd_grab_pointer(vc, "user-request-main-window");
>      } else {
>          gd_ungrab_keyboard(s);
>          gd_ungrab_pointer(s);
> @@ -1432,7 +1432,7 @@ static gboolean gd_enter_event(GtkWidget *widget, 
> GdkEventCrossing *crossing,
>      GtkDisplayState *s = vc->s;
>
>      if (gd_grab_on_hover(s)) {
> -        gd_grab_keyboard(vc);
> +        gd_grab_keyboard(vc, "grab-on-hover");
>      }
>      return TRUE;
>  }
> --
> 1.8.3.1
>
>



-- 
Marc-André Lureau



reply via email to

[Prev in Thread] Current Thread [Next in Thread]