Related task may not have a metric or platform.
authordewei_zhu@apple.com <dewei_zhu@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Jun 2018 06:02:47 +0000 (06:02 +0000)
committerdewei_zhu@apple.com <dewei_zhu@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Jun 2018 06:02:47 +0000 (06:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=186426

Reviewed by Ryosuke Niwa.

Related task in the related task list can be a custom analysis task which
may not have platform or metric.

* public/v3/pages/analysis-task-page.js: Added null checks for platform and metric.
(AnalysisTaskPage.prototype._renderRelatedTasks):

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v3/pages/analysis-task-page.js

index 1308a05..0a3c0e0 100644 (file)
@@ -1,5 +1,18 @@
 2018-06-07  Dewei Zhu  <dewei_zhu@apple.com>
 
+        Related task may not have a metric or platform.
+        https://bugs.webkit.org/show_bug.cgi?id=186426
+
+        Reviewed by Ryosuke Niwa.
+
+        Related task in the related task list can be a custom analysis task which
+        may not have platform or metric.
+
+        * public/v3/pages/analysis-task-page.js: Added null checks for platform and metric.
+        (AnalysisTaskPage.prototype._renderRelatedTasks):
+
+2018-06-07  Dewei Zhu  <dewei_zhu@apple.com>
+
         Fix browser test failed assertions and a bug in 'common-component-base'
         https://bugs.webkit.org/show_bug.cgi?id=186423
 
index 14e3a72..0f7089f 100644 (file)
@@ -704,9 +704,9 @@ class AnalysisTaskPage extends PageWithHeading {
         this.renderReplace(this.content('related-tasks-list'), (task && relatedTasks ? relatedTasks : []).map((otherTask) => {
                 let suffix = '';
                 const taskLabel = otherTask.label();
-                if (otherTask.metric() != task.metric() && taskLabel.indexOf(otherTask.metric().label()) < 0)
+                if (otherTask.metric() && otherTask.metric() != task.metric() && taskLabel.indexOf(otherTask.metric().label()) < 0)
                     suffix += ` with "${otherTask.metric().label()}"`;
-                if (otherTask.platform() != task.platform() && taskLabel.indexOf(otherTask.platform().label()) < 0)
+                if (otherTask.platform() && otherTask.platform() != task.platform() && taskLabel.indexOf(otherTask.platform().label()) < 0)
                     suffix += ` on ${otherTask.platform().label()}`;
                 return element('li', [link(taskLabel, this.router().url(`analysis/task/${otherTask.id()}`)), suffix]);
             }));