<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: KWin, VDG.<br />Herald added a project: KWin.<br />Herald added a subscriber: kwin.<br />broulik requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28021">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>When KWin crashes several times in quick succession it disables compositing for safety.<br />
However, when this happened, it leaves the user without compositing without any idea what was going on, other than "my windows look ugly now".<br />
This patch stores the fact that compositing got disabled because of crashes in the config and shows an appropriate icon which opens the Compositing KCM which then also has an explanatory message.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Called <tt style="background: #ebebeb; font-size: 13px;">killall -SIGSEGV kwin_x11</tt> serveral times in a row</li>
<li class="remarkup-list-item">KWin restarted with compositing off and I got a try icon</li>
<li class="remarkup-list-item">Clicked the tray icon, acknowledged the message, re-enabled compositing, icon went away</li>
</ul>

<p><a href="https://phabricator.kde.org/F8174182" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8174182: Screenshot_20200313_121056.PNG</a></p>

<p>When I uncheck "enable compositing at start" KWin reconfigures but doesn't disable the compositor. Not sure if this is caused by this patch or not.</p>

<p>Open to suggestions on wording</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28021">https://phabricator.kde.org/D28021</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>CMakeLists.txt<br />
composite.cpp<br />
composite.h<br />
compositingdisablednotifier.cpp<br />
compositingdisablednotifier.h<br />
kcmkwin/kwincompositing/compositing.cpp<br />
kcmkwin/kwincompositing/compositing.h<br />
kcmkwin/kwincompositing/compositing.ui<br />
kcmkwin/kwincompositing/main.cpp<br />
main_x11.cpp<br />
platform.cpp<br />
platform.h<br />
plugins/platforms/x11/standalone/x11_platform.cpp<br />
plugins/platforms/x11/standalone/x11_platform.h</div></div></div><br /><div><strong>To: </strong>broulik, KWin, VDG<br /><strong>Cc: </strong>kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>