| Summary: | [ews] status-bubble should link to buildbot queue while waiting in queue | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||
| Component: | Tools / Tests | Assignee: | Aakash Jain <aakash_jain> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | aakash_jain, ap, jbedard, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | Other | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=197520 https://bugs.webkit.org/show_bug.cgi?id=209598 |
||||||
| Attachments: |
|
||||||
|
Description
Aakash Jain
2020-03-25 07:36:55 PDT
Created attachment 394493 [details]
Patch
Comment on attachment 394493 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=394493&action=review > Tools/BuildSlaveSupport/ews-app/ews/views/statusbubble.py:99 > + if self._get_parent_queue(queue): Is _get_parent_queue low-cost call? Because we're calling it twice if it returns something. Comment on attachment 394493 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=394493&action=review >> Tools/BuildSlaveSupport/ews-app/ews/views/statusbubble.py:99 >> + if self._get_parent_queue(queue): > > Is _get_parent_queue low-cost call? Because we're calling it twice if it returns something. Yeah, it's just a local dictionary lookup. Committed r258997: <https://trac.webkit.org/changeset/258997> All reviewed patches have been landed. Closing bug and clearing flags on attachment 394493 [details]. |