Title: [179647] trunk/Websites/perf.webkit.org
Revision
179647
Author
[email protected]
Date
2015-02-04 16:32:27 -0800 (Wed, 04 Feb 2015)

Log Message

New perf dashboard shows null as the aggregator name if no aggregation is done
https://bugs.webkit.org/show_bug.cgi?id=141256

Reviewed by Chris Dumez.

Don't show the aggregator name if there isn't one.

* public/v2/manifest.js:
(App.Metric.label):

Modified Paths

Diff

Modified: trunk/Websites/perf.webkit.org/ChangeLog (179646 => 179647)


--- trunk/Websites/perf.webkit.org/ChangeLog	2015-02-05 00:21:59 UTC (rev 179646)
+++ trunk/Websites/perf.webkit.org/ChangeLog	2015-02-05 00:32:27 UTC (rev 179647)
@@ -1,5 +1,17 @@
 2015-02-04  Ryosuke Niwa  <[email protected]>
 
+        New perf dashboard shows null as the aggregator name if no aggregation is done
+        https://bugs.webkit.org/show_bug.cgi?id=141256
+
+        Reviewed by Chris Dumez.
+
+        Don't show the aggregator name if there isn't one.
+
+        * public/v2/manifest.js:
+        (App.Metric.label):
+
+2015-02-04  Ryosuke Niwa  <[email protected]>
+
         Unreviewed build fix after r179611.
 
         * public/v2/interactive-chart.js:

Modified: trunk/Websites/perf.webkit.org/public/v2/manifest.js (179646 => 179647)


--- trunk/Websites/perf.webkit.org/public/v2/manifest.js	2015-02-05 00:21:59 UTC (rev 179646)
+++ trunk/Websites/perf.webkit.org/public/v2/manifest.js	2015-02-05 00:32:27 UTC (rev 179647)
@@ -20,7 +20,7 @@
     aggregator: DS.attr('string'),
     label: function ()
     {
-        return this.get('name') + ' : ' + this.get('aggregator');
+        return this.get('name') + (this.get('aggregator') ? ' : ' + this.get('aggregator') : '');
     }.property('name', 'aggregator'),
     path: function ()
     {
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to