Autopilot

Merge lp:~3v1n0/autopilot/bamf-gdk-dpy into lp:autopilot

  1. bamf-gdk-dpy
  2. Merge into trunk
Proposed by Marco Trevisan (Treviño)
Status: Merged
Approved by: Thomi Richards
Approved revision: 345
Merged at revision: 347
Proposed branch: lp:~3v1n0/autopilot/bamf-gdk-dpy
Merge into: lp:autopilot
Diff against target: 12 lines (+1/-1)
1 file modified
autopilot/process/_bamf.py (+1/-1)
To merge this branch: bzr merge lp:~3v1n0/autopilot/bamf-gdk-dpy
Reviewer Review Type Date Requested Status
Thomi Richards (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+188973@code.launchpad.net

Commit message

Bamf: pass Gdk.Display to GdkX11.X11Window.foreign_new_for_display

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:345
http://jenkins.qa.ubuntu.com/job/autopilot-ci/259/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-saucy-amd64-ci/187
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-saucy-armhf-ci/187
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-saucy-i386-ci/31

Click here to trigger a rebuild:
http://s-jenkins:8080/job/autopilot-ci/259/rebuild

review: Approve (continuous-integration)
Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote :

LGTM, thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'autopilot/process/_bamf.py'
2--- autopilot/process/_bamf.py 2013-09-10 09:35:19 +0000
3+++ autopilot/process/_bamf.py 2013-10-03 02:02:28 +0000
4@@ -488,7 +488,7 @@
5 # FIXME: We need to use the gdk window here to get the real coordinates
6 geometry = self._x_win.get_geometry()
7 origin = GdkX11.X11Window.foreign_new_for_display(
8- get_display(), self._xid).get_origin()
9+ GdkX11.X11Display(), self._xid).get_origin()
10 return (origin[0], origin[1], geometry.width, geometry.height)
11
12 @property

Subscribers

People subscribed via source and target branches