While a patch is waiting in queue, ews-app's status-bubble should display its position in queue.
Created attachment 367158 [details]
Comment on attachment 367158 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=367158&action=review
> + bubble["queue_position"] = queue_position
nit: inconsistent use of `"` and `'`. Seems like most of it is `'` so we can stick to that.
> + previously_sent_patches = set(Patch.objects.filter(modified__gte=from_timestamp).filter(sent_to_buildbot=True).filter(obsolete=False).filter(modified__lt=patch.modified))
nit: I think it would be more readable if the filters are above one another.
Created attachment 367386 [details]
Incorporated review comments.
Comment on attachment 367386 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=367386&action=review
> + processed_patch = set([build.patch for build in recent_builds])
should this be "processed_patches" ?
> should this be "processed_patches" ?
Committed r244284: <https://trac.webkit.org/changeset/244284>