D16314: [wayland] Fix Placement::placeTransient keeping screens in bound.

davidedmundson created this revision.
davidedmundson added a reviewer: KWin.
Herald added a project: KWin.
Herald added a subscriber: kwin.
davidedmundson requested review of this revision.

REVISION SUMMARY
  If a window flows off the left, we move left of the popup to the left
  edge of the screen.
  
  Currently if a window flows off the right, we move the window back by
  it's own width, leaving it floating at a random point.
  For consistency we should be setting it so the right edge of the popup is on the right
  edge of the screen.
  
  So in the auto test for the "right border" case:
  The screen is 1280 wide, and we open a 10px popup at 1279  the final X
  should be 1270.

TEST PLAN
  Unit test

REPOSITORY
  R108 KWin

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D16314

AFFECTED FILES
  autotests/integration/transient_placement.cpp
  placement.cpp

To: davidedmundson, #kwin
Cc: kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart