[Differential] [Request, 6 lines] D2446: Remove redundant manual focusOut handling in DesktopView

[davidedmundson (David Edmundson)]

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

REVISION SUMMARY
  It makes no sense to modify the focus of containment within it's
  FocusScope, and activeFocus will be set correctly by code in
  QQuickWindow. Especially as it doesn't set it back.
  
  The only way this would make any sense is if someone in Desktop is
  incorrectly using "focus" when they mean to use "activeFocus". This code
  was introduced to fix a bug in the toolbox (331971) where someone was
  making the mistake above.
  
  That toolbox code has been subsequently fixed properly, leaving this as
  dead code.

TEST PLAN
  Toolbox still closes appropriately.
  Everything else still seems the same

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

BRANCH
  master

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

AFFECTED FILES
  shell/desktopview.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160815/4f73ebc0/attachment.html>