Merge lp://staging/~brandontschaefer/unity/move-pointer-barrier-to-xi-1.6.99.1 into lp://staging/unity

Proposed by Brandon Schaefer
Status: Merged
Merged at revision: 3407
Proposed branch: lp://staging/~brandontschaefer/unity/move-pointer-barrier-to-xi-1.6.99.1
Merge into: lp://staging/unity
Diff against target: 703 lines (+261/-152)
8 files modified
CMakeLists.txt (+2/-1)
debian/changelog (+4/-3)
debian/control (+1/-0)
launcher/EdgeBarrierController.cpp (+126/-3)
launcher/EdgeBarrierControllerPrivate.h (+9/-0)
launcher/PointerBarrier.cpp (+82/-94)
launcher/PointerBarrier.h (+7/-5)
tests/test_pointer_barrier.cpp (+30/-46)
To merge this branch: bzr merge lp://staging/~brandontschaefer/unity/move-pointer-barrier-to-xi-1.6.99.1
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Andrea Azzarone (community) Approve
Review via email: mp+150175@code.staging.launchpad.net

Commit message

Move unity to the new libxi changes, along with the new 1.14 xserver.

Description of the change

Some big changes. Updating pointer barrier logic new versions of xfixes and xi (1.6.99.1).

Had to move a bunch of things around, due to the fact that we didn't handle any kind of XI2 events before. Also re worked somethings so they only run once now...before we had a bunch of things being ran every time a barrier was being created that was not necessary.

So now the new logic, simplified:
Mouse hits barrier, XIBarrierEvent generated and sent off.
Nux receives XEvent and passes it back to EdgeBarrierController.cpp::HandleEventCB
In HandleEventCB we check to see of the opcode matches, if so pass it to HandleEvent
In HandleEvent we check if its a type XI_BarrierHit
If so, then find the barrier wrapper the event belongs to.
(XGetEventData works onces per event, so the logic we had before didn't work).
From here we pass it to PointerBarrier.cpp::HandleBarrierEvent,
Here we are pretty much back to where we were before the changed :).

To post a comment you must log in.
Revision history for this message
Andrea Azzarone (azzar1) :
review: Approve
Revision history for this message
Sebastien Bacher (seb128) wrote :

trying to build with the old libxfixes it fails on:

"/usr/include/X11/extensions/XInput2.h:173:22: error: conflicting declaration ‘typedef unsigned int BarrierEventID’
  typedef unsigned int BarrierEventID;

the requirement/build-depends on that one should probably be bumped as well

works fine otherwise ;-)

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:3175
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~brandontschaefer/unity/move-pointer-barrier-to-xi-1.6.99.1/+merge/150175/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity-ci/182/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-saucy-amd64-ci/70/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-saucy-armhf-ci/70/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-saucy-i386-ci/70/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-ci/182/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

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.