Should not render latest build information if there is no data points for a config.
authordewei_zhu@apple.com <dewei_zhu@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Jul 2019 16:04:23 +0000 (16:04 +0000)
committerdewei_zhu@apple.com <dewei_zhu@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Jul 2019 16:04:23 +0000 (16:04 +0000)
https://bugs.webkit.org/show_bug.cgi?id=200250

Reviewed by Ryosuke Niwa.

Fix a bug test freshness page that tooltip cannot be rendered when a cell does not have
a data point.

* public/v3/pages/test-freshness-page.js: Added a null check on commit set before rendering
latest build informaiton.
(TestFreshnessPage.prototype._renderTooltip):

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v3/pages/test-freshness-page.js

index 7045d2e..110e463 100644 (file)
@@ -1,3 +1,17 @@
+2019-07-29  Dewei Zhu  <dewei_zhu@apple.com>
+
+        Should not render latest build information if there is no data points for a config.
+        https://bugs.webkit.org/show_bug.cgi?id=200250
+
+        Reviewed by Ryosuke Niwa.
+
+        Fix a bug test freshness page that tooltip cannot be rendered when a cell does not have
+        a data point.
+
+        * public/v3/pages/test-freshness-page.js: Added a null check on commit set before rendering
+        latest build informaiton.
+        (TestFreshnessPage.prototype._renderTooltip):
+
 2019-07-25  Dewei Zhu  <dewei_zhu@apple.com>
 
         Fix a bug that cannot update builder password on builders administrative page.
index 70306c8..2175c56 100644 (file)
@@ -171,16 +171,19 @@ class TestFreshnessPage extends PageWithHeading {
         tooltipTable.style.left = rect.left + rect.width / 2 - tooltipTable.offsetWidth / 2 + containerMarginLeft + 'px';
 
         let tableContent = [element('tr', element('td', {colspan: 2}, buildSummary))];
-        if (commitSet.repositories().length)
-            tableContent.push(element('tr', element('th', {colspan: 2}, 'Latest build information')));
-
-        tableContent.push(Repository.sortByNamePreferringOnesWithURL(commitSet.repositories()).map((repository) => {
-            const commit = commitSet.commitForRepository(repository);
-            return element('tr', [
-                element('td', repository.name()),
-                element('td', commit.url() ? link(commit.label(), commit.label(), commit.url(), true) : commit.label())
-            ]);
-        }));
+
+        if (commitSet) {
+            if (commitSet.repositories().length)
+                tableContent.push(element('tr', element('th', {colspan: 2}, 'Latest build information')));
+
+            tableContent.push(Repository.sortByNamePreferringOnesWithURL(commitSet.repositories()).map((repository) => {
+                const commit = commitSet.commitForRepository(repository);
+                return element('tr', [
+                    element('td', repository.name()),
+                    element('td', commit.url() ? link(commit.label(), commit.label(), commit.url(), true) : commit.label())
+                ]);
+            }));
+        }
 
         if (build) {
             const url = build.url();