You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2016/12/02 21:58:29 UTC

mesos git commit: Fixed WebUI error on Firefox 45.

Repository: mesos
Updated Branches:
  refs/heads/master 71182bf9a -> a6bab9015


Fixed WebUI error on Firefox 45.

Adjust the order of function `matchTask` to fix the WebUI error on
Firefox 45. See MESOS-6624 for details.

Review: https://reviews.apache.org/r/54036/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/a6bab901
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/a6bab901
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/a6bab901

Branch: refs/heads/master
Commit: a6bab9015cd63121081495b8291635f386b95a92
Parents: 71182bf
Author: haosdent huang <ha...@gmail.com>
Authored: Fri Dec 2 13:58:10 2016 -0800
Committer: Benjamin Mahler <bm...@apache.org>
Committed: Fri Dec 2 13:58:10 2016 -0800

----------------------------------------------------------------------
 src/webui/master/static/js/controllers.js | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/a6bab901/src/webui/master/static/js/controllers.js
----------------------------------------------------------------------
diff --git a/src/webui/master/static/js/controllers.js b/src/webui/master/static/js/controllers.js
index dd0cc81..5dcfbdb 100644
--- a/src/webui/master/static/js/controllers.js
+++ b/src/webui/master/static/js/controllers.js
@@ -837,15 +837,15 @@
 
         var sandboxDirectory = executor.directory;
 
+        function matchTask(task) {
+          return $routeParams.task_id === task.id;
+        }
+
         // Continue to navigate to the task's sandbox if the task id is
         // specified in route parameters.
         if ($routeParams.task_id) {
           setTaskSandbox(executor);
 
-          function matchTask(task) {
-            return $routeParams.task_id === task.id;
-          }
-
           var task =
             _.find(executor.tasks, matchTask) ||
             _.find(executor.queued_tasks, matchTask) ||