AppScale: Assertion hit when hovering a webkit-queue bubble

[Aakash Jain]

console.assert(this.statusPageURL === data.status_page);

Steps to reproduce:
1. Open https://build.webkit.org/dashboard
2. Open Web Inspector
3. Hover a commit queue or EWS bubble.

[Aakash Jain]

Created attachment 258950 [details]
Proposed patch

[Alexey Proskuryakov]

Comment on attachment 258950 [details]
Proposed patch

View in context: https://bugs.webkit.org/attachment.cgi?id=258950&action=review

> Tools/BuildSlaveSupport/build.webkit.org-config/public_html/dashboard/Scripts/BubbleQueue.js:136
> +            console.assert(this.statusPageURL.replace(/.*?:\/\//g, "") === data.status_page.replace(/.*?:\/\//g, ""));

This replace removes anything before a colon, which I think is incorrect. E.g. "foo://bar://baz".replace(/.*?:\/\//g, "") -> "baz".

How about something like "/^https?://" ?

[Aakash Jain]

Created attachment 258954 [details]
Proposed patch

good point. made the change.

[WebKit Commit Bot]

Comment on attachment 258954 [details]
Proposed patch

Clearing flags on attachment: 258954

Committed r188419: <http://trac.webkit.org/changeset/188419>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.