Build fix. It looks like the code here is racy.
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jul 2017 02:38:03 +0000 (02:38 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 12 Jul 2017 02:38:03 +0000 (02:38 +0000)
* public/v3/components/chart-pane-base.js:
(ChartPaneBase.prototype.configure):
(ChartPaneBase.prototype.setOpenRepository):

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

Websites/perf.webkit.org/ChangeLog
Websites/perf.webkit.org/public/v3/components/chart-pane-base.js

index 37d9b4a..4fda608 100644 (file)
@@ -1,5 +1,13 @@
 2017-07-11  Ryosuke Niwa  <rniwa@webkit.org>
 
+        Build fix. It looks like the code here is racy.
+
+        * public/v3/components/chart-pane-base.js:
+        (ChartPaneBase.prototype.configure):
+        (ChartPaneBase.prototype.setOpenRepository):
+
+2017-07-11  Ryosuke Niwa  <rniwa@webkit.org>
+
         Show the roots built by perf try bots on results page
         https://bugs.webkit.org/show_bug.cgi?id=174305
 
index 95662c4..574f029 100644 (file)
@@ -57,6 +57,7 @@ class ChartPaneBase extends ComponentBase {
         this._revisionRange = new ChartRevisionRange(this._mainChart);
 
         this._mainChartStatus = new ChartPaneStatusView(result.metric, this._mainChart);
+        this._mainChartStatus.setCurrentRepository(this._openRepository);
         this._mainChartStatus.listenToAction('openRepository', this.openNewRepository.bind(this));
         this.renderReplace(this.content().querySelector('.chart-pane-details'), this._mainChartStatus);
 
@@ -132,7 +133,8 @@ class ChartPaneBase extends ComponentBase {
     setOpenRepository(repository)
     {
         this._openRepository = repository;
-        this._mainChartStatus.setCurrentRepository(repository);
+        if (this._mainChartStatus)
+            this._mainChartStatus.setCurrentRepository(repository);
         this._updateCommitLogViewer();
     }