D26213: Exclude daemonized processes from Dolphin::attachToExistingInstance()

[Elvis Angelaccio]

elvisangelaccio created this revision.
elvisangelaccio added a reviewer: Dolphin.
Herald added a project: Dolphin.
Herald added a subscriber: kfm-devel.
elvisangelaccio requested review of this revision.

REVISION SUMMARY
  `dolphin --daemon` does not have the `/dolphin/Dolphin_1` dbus path,
  because it doesn't have any DolphinMainWindow.
  
  Instead of working around this issue (as we did in D21666 <https://phabricator.kde.org/D21666> and D25510 <https://phabricator.kde.org/D25510>),
  just exclude these processes from the list of dbus instances checked by
  `Dolphin::attachToExistingInstance()`.
  
  CCBUG: 408244

TEST PLAN
  Same test plan as in D21666 <https://phabricator.kde.org/D21666> and D25510 <https://phabricator.kde.org/D25510>

REPOSITORY
  R318 Dolphin

BRANCH
  deamon-refactor (branched from master)

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

AFFECTED FILES
  src/dbusinterface.cpp
  src/dbusinterface.h
  src/global.cpp
  src/global.h
  src/main.cpp

To: elvisangelaccio, #dolphin
Cc: kfm-devel, pberestov, iasensio, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20191224/828ba52f/attachment.htm>