Bug 196308 – Add another status-bubble iframe to display results from new EWS

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Aakash Jain]

Add another iframe for status-bubble in Bugzilla to display results from new EWS (https://ews-build.webkit.org). Initially, new EWS will only have queues which are not present in existing EWS (e.g.: EWS for API Tests, webkitperl), therefore we should display results from both EWSes.

[Aakash Jain]

Created attachment 366089 [details]
Patch

[Jonathan Bedard]

To be clear, this adds another row of bubbles, right?

Also, the intention of this is to eventually remove the current row once we've migrated everything to new EWS, right?

[Aakash Jain]

(In reply to Jonathan Bedard from comment #2)
> To be clear, this adds another row of bubbles, right?
It adds more bubbles in same row. Two iframes are aligned horizontally. e.g.: https://bugs.webkit-uat.org/show_bug.cgi?id=168329
> 
> Also, the intention of this is to eventually remove the current row once we've migrated everything to new EWS, right?
Yes.

[Lucas Forschler]

Comment on attachment 366089 [details]
Patch

is there any easy way to pull our these URLs into a constant variable and reference that in the code? If we ever change the url, it would make it much easier to update.
I think we should consider this for both https://ews.webkit.org and webkit-queues.webkit.org.

[Aakash Jain]

> is there any easy way to pull our these URLs into a constant variable and reference that in the code? If we ever change the url, it would make it much easier to update.
Don't see an easy way. Will probably attempt that in a separate patch later.

Also going to fix some UI glitches by setting the initial width of iframe to 0px instead of 600px.

[Aakash Jain]

Committed r243921: <https://trac.webkit.org/changeset/243921>

[Radar WebKit Bug Importer]

<rdar://problem/49630491>