Merge lp://qastaging/~desrt/appmenu-gtk/ignore-gtkmodelmenu into lp://qastaging/appmenu-gtk/12.10

Proposed by desrt
Status: Merged
Approved by: Charles Kerr
Approved revision: 157
Merged at revision: 157
Proposed branch: lp://qastaging/~desrt/appmenu-gtk/ignore-gtkmodelmenu
Merge into: lp://qastaging/appmenu-gtk/12.10
Diff against target: 24 lines (+8/-0)
1 file modified
src/bridge.c (+8/-0)
To merge this branch: bzr merge lp://qastaging/~desrt/appmenu-gtk/ignore-gtkmodelmenu
Reviewer Review Type Date Requested Status
jenkins (community) continuous-integration Approve
Ted Gould (community) Approve
Review via email: mp+124947@code.qastaging.launchpad.net

Commit message

Ignore menus from gtkmodelmenu, in two ways:

    - if we notice a GtkMenuShell from a gtkmodelmenu then don't try to
      monitor its toplevel

    - when searching for menubars inside of toplevels, ignore ones created
      by gtkmodelmenu

Description of the change

  Ignore menus from gtkmodelmenu, in two ways:

    - if we notice a GtkMenuShell from a gtkmodelmenu then don't try to
      monitor its toplevel

    - when searching for menubars inside of toplevels, ignore ones created
      by gtkmodelmenu

To post a comment you must log in.
Revision history for this message
Ted Gould (ted) :
review: Approve
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

FAILED: Autolanding.
No commit message was specified.
http://jenkins.qa.ubuntu.com/job/appmenu-gtk-autolanding/6/

review: Needs Fixing (continuous-integration)
Revision history for this message
jenkins (martin-mrazik+qa) wrote :
review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) wrote :

Copying description to commit message to trigger autolanding

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches