Add another status-bubble iframe to display results from new EWS
authoraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 5 Apr 2019 00:31:41 +0000 (00:31 +0000)
committeraakash_jain@apple.com <aakash_jain@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 5 Apr 2019 00:31:41 +0000 (00:31 +0000)
https://bugs.webkit.org/show_bug.cgi?id=196308

Reviewed by Lucas Forschler.

* js/status-bubble.js:
(handleStatusBubbleMessage): Added URL for new EWS.
(handleStatusBubbleLoadNewEWS): Method to post message to new EWS.
* code-review.js: Added another iframe to display results from new EWS.
* template/en/default/attachment/edit.html.tmpl: Ditto.
* template/en/default/attachment/list.html.tmpl: Ditto.
* template/en/default/attachment/reviewform.html.tmpl: Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@243921 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/code-review.js
Websites/bugs.webkit.org/js/status-bubble.js
Websites/bugs.webkit.org/template/en/default/attachment/edit.html.tmpl
Websites/bugs.webkit.org/template/en/default/attachment/list.html.tmpl
Websites/bugs.webkit.org/template/en/default/attachment/reviewform.html.tmpl

index 44c50fd..494eb23 100644 (file)
@@ -1,3 +1,18 @@
+2019-04-04  Aakash Jain  <aakash_jain@apple.com>
+
+        Add another status-bubble iframe to display results from new EWS
+        https://bugs.webkit.org/show_bug.cgi?id=196308
+
+        Reviewed by Lucas Forschler.
+
+        * js/status-bubble.js:
+        (handleStatusBubbleMessage): Added URL for new EWS.
+        (handleStatusBubbleLoadNewEWS): Method to post message to new EWS.
+        * code-review.js: Added another iframe to display results from new EWS.
+        * template/en/default/attachment/edit.html.tmpl: Ditto.
+        * template/en/default/attachment/list.html.tmpl: Ditto.
+        * template/en/default/attachment/reviewform.html.tmpl: Ditto.
+
 2019-01-18  Ling Ho  <lingho@apple.com>
 
         Update Bugzilla running on bugs.webkit.org and bugs.webkit-uat.org to version 5.0.4
index 51897f6..14ae98b 100644 (file)
@@ -1,4 +1,5 @@
 // Copyright (C) 2010 Adam Barth. All rights reserved.
+// Copyright (C) 2019 Apple Inc. All rights reserved.
 //
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are met:
@@ -573,6 +574,12 @@ var CODE_REVIEW_UNITTEST;
       statusBubble.onload = function () { handleStatusBubbleLoad(this); };
       $('.statusBubble').append(statusBubble);
 
+      var statusBubbleNewEWS = document.createElement('iframe');
+      statusBubbleNewEWS.src  = 'https://ews.webkit.org/status-bubble/' + attachment_id + '/';
+      // Can't append the HTML because we need to set the onload handler before appending the iframe to the DOM.
+      statusBubbleNewEWS.onload = function () { handleStatusBubbleLoadNewEWS(this); };
+      $('.statusBubble').append(statusBubbleNewEWS);
+
       $('#toolbar .bugLink').html('<a href="/show_bug.cgi?id=' + bug_id + '" target="_blank">Bug ' + bug_id + '</a>');
     });
   }
index 0b7588f..8e1c0ae 100644 (file)
@@ -1,5 +1,6 @@
 /*
  * Copyright (C) 2018 Sony Interactive Entertainment Inc.
+ * Copyright (C) 2019 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -24,7 +25,7 @@
  */
 
 function handleStatusBubbleMessage(event) {
-    if (event.origin !== 'https://webkit-queues.webkit.org' || !event.data.height)
+    if ((event.origin !== 'https://webkit-queues.webkit.org' && event.origin !== 'https://ews.webkit.org') || !event.data.height)
         return;
 
     for (const iframe of document.querySelectorAll('.statusBubble > iframe')) {
@@ -39,3 +40,7 @@ function handleStatusBubbleMessage(event) {
 function handleStatusBubbleLoad(iframe) {
     iframe.contentWindow.postMessage('containerMetrics', 'https://webkit-queues.webkit.org');
 }
+
+function handleStatusBubbleLoadNewEWS(iframe) {
+    iframe.contentWindow.postMessage('containerMetrics', 'https://ews.webkit.org');
+}
index eca018a..2402a0b 100644 (file)
 
         <div class="statusBubble">
           <iframe src="https://webkit-queues.webkit.org/status-bubble/[% attachment.id %]"
-                  style="width: 600px; height: 20px; border: none;" scrolling="no" onload="handleStatusBubbleLoad(this)">
+                  style="width: 0px; height: 20px; border: none;" scrolling="no" onload="handleStatusBubbleLoad(this)">
+          </iframe>
+          <iframe src="https://ews.webkit.org/status-bubble/[% attachment.id %]/"
+                  style="width: 0px; height: 20px; border: none;" scrolling="no" onload="handleStatusBubbleLoadNewEWS(this)">
           </iframe>
         </div>
         <br>
index dc6c5d7..a9bc375 100644 (file)
@@ -156,7 +156,10 @@ function toggle_display(link) {
           [% IF attachment.ispatch %]
           <div class="statusBubble">
             <iframe src="https://webkit-queues.webkit.org/status-bubble/[% attachment.id %]"
-                    style="width: 600px; height: 20px; border: none;" scrolling="no" onload="handleStatusBubbleLoad(this)">
+                    style="width: 0px; height: 20px; border: none;" scrolling="no" onload="handleStatusBubbleLoad(this)">
+            </iframe>
+            <iframe src="https://ews.webkit.org/status-bubble/[% attachment.id %]/"
+                    style="width: 0px; height: 20px; border: none;" scrolling="no" onload="handleStatusBubbleLoadNewEWS(this)">
             </iframe>
           </div>
           [% END %]
index 4de506e..9908377 100644 (file)
         <iframe src="https://webkit-queues.webkit.org/status-bubble/[% attachment.id %]"
                 style="width: 400px; height: 20px; border: none;" scrolling="no">
         </iframe>
+        <iframe src="https://ews.webkit.org/status-bubble/[% attachment.id %]/"
+                style="width: 400px; height: 20px; border: none;" scrolling="no">
+        </iframe>
       </div>
     </td>
     </tr>