This old attachment has an incorrect status queue log. We should be tolerant of that.
Created attachment 239247 [details] proposed fix
Comment on attachment 239247 [details] proposed fix View in context: https://bugs.webkit.org/attachment.cgi?id=239247&action=review > Tools/QueueStatusServer/app.yaml:2 > +version: ap # Bugzilla bug ID of last major change please fix the comment. > Tools/QueueStatusServer/handlers/statusbubble.py:141 > + message_to_display = statuses[1].message if len(statuses) > 1 else statuses[0].message IIRC 'if statuses' is sufficient (an empty list evaluates to false) but this is probably clearer
Committed <http://trac.webkit.org/r174303>.
Comment on attachment 239247 [details] proposed fix View in context: https://bugs.webkit.org/attachment.cgi?id=239247&action=review >> Tools/QueueStatusServer/handlers/statusbubble.py:141 >> + message_to_display = statuses[1].message if len(statuses) > 1 else statuses[0].message > > IIRC 'if statuses' is sufficient (an empty list evaluates to false) but this is probably clearer Nevermind!
Actually committed in 174304