When ApplyPatch step fails, the buildbot generated summary is: 'Applied patch (failure)'. This isn't good and might be confusing. We should improve this to something like: 'Patch does not apply'.
Created attachment 365335 [details] Patch
Attachment 365335 [details] did not pass style-queue: ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:138: [ApplyPatch.getResultSummary] Use of super on an old style class [pylint/E1002] [5] ERROR: Tools/BuildSlaveSupport/ews-build/steps.py:139: [ApplyPatch.getResultSummary] Instance of 'ApplyPatch' has no 'results' member [pylint/E1101] [5] Total errors found: 2 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
I'm having second thoughts about making Buildbot changes to tweak status bubble content. It's useful to have good text messages in Buildbot UI too. However, relying on Buildbot as authoritative source of status bubble UI text will become problematic once the system is in production, as we don't want to restart Buildbot often. So a less intrusive way to polish UI will need to be added.
Comment on attachment 365335 [details] Patch Clearing flags on attachment: 365335 Committed r243217: <https://trac.webkit.org/changeset/243217>
All reviewed patches have been landed. Closing bug.
<rdar://problem/49069801>