D7871: Properly update the visible (icon) name when the caption changes

[Martin Flöser]

graesslin created this revision.
graesslin added reviewers: KWin, Plasma, hein.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.

REVISION SUMMARY
  It was possible that we performed an early exit when the caption changes
  and this results in the visible name being wrong. E.g if we have two
  windows called foo, the second one has the visible name "foo <2>". After
  changing that to "bar" the bug resulted in it still being the foo
  variant instead of getting cleared.
  
  BUG: 384760
  FIXED-IN: 5.11.0

TEST PLAN
  Created new test case exposing the problem

REPOSITORY
  R108 KWin

BRANCH
  fix-visible-name

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

AFFECTED FILES
  autotests/integration/x11_client_test.cpp
  client.cpp

To: graesslin, #kwin, #plasma, hein
Cc: plasma-devel, kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170918/66e48a7c/attachment.html>