D13850: Remove pointless setMin/setMax size on DesktopView

[David Edmundson]

davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
davidedmundson requested review of this revision.

REVISION SUMMARY
  On X, this is entirely useless.There's a special code path in create for
  windowType == Desktop and in ::setGeometry it doesn't seem to help/hinder.
  It's not like we need WM_NORMAL_HINTS for a WM to resize us.
  
  On wayland it's downright destructive.
  On the second setGeometry call the QPA bounds the new geography to the
  current min/max (from our previous screenGeometryChanged). But updating
  the min/max doesn't update our geometry. End result is our screen is
  off.

TEST PLAN
  Resized screen on Wayland. Wallpaper now filled exactly instead of being broken
  Resized screen on Xcb. Wallpaper still filled the screen precisely

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

AFFECTED FILES
  shell/desktopview.cpp

To: davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180702/84ecced3/attachment.html>