From d2c791dcbb26679166f3cbbc645e616d963753dd Mon Sep 17 00:00:00 2001 From: Iuri Matias Date: Thu, 6 Sep 2018 13:23:29 -0400 Subject: [PATCH] cleanup --- src/app/debugger/debuggerUI/ButtonNavigator.js | 5 ++--- src/app/debugger/debuggerUI/StepManager.js | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/src/app/debugger/debuggerUI/ButtonNavigator.js b/src/app/debugger/debuggerUI/ButtonNavigator.js index 59e09a5328..afd2b03560 100644 --- a/src/app/debugger/debuggerUI/ButtonNavigator.js +++ b/src/app/debugger/debuggerUI/ButtonNavigator.js @@ -36,7 +36,7 @@ var css = csjs` } ` -function ButtonNavigator (_parent, _traceManager) { +function ButtonNavigator () { this.event = new EventManager() this.intoBackDisabled = true this.overBackDisabled = true @@ -46,8 +46,6 @@ function ButtonNavigator (_parent, _traceManager) { this.jumpNextBreakpointDisabled = true this.jumpPreviousBreakpointDisabled = true - this.traceManager = _traceManager - this.currentCall = null this.revertionPoint = null this.view } @@ -131,6 +129,7 @@ ButtonNavigator.prototype.updateDisabled = function (id, disabled) { } ButtonNavigator.prototype.resetAndCheckRevertionPoint = function (revertedReason, revertionPoint) { + // TODO: since this is only used for the jump, this value should instead be outside of the view if (revertionPoint) { this.revertionPoint = revertionPoint } diff --git a/src/app/debugger/debuggerUI/StepManager.js b/src/app/debugger/debuggerUI/StepManager.js index f0d0df3738..4b09398803 100644 --- a/src/app/debugger/debuggerUI/StepManager.js +++ b/src/app/debugger/debuggerUI/StepManager.js @@ -40,7 +40,7 @@ function StepManager (_parent, _traceManager) { } }) - this.buttonNavigator = new ButtonNavigator(_parent, this.traceManager) + this.buttonNavigator = new ButtonNavigator() _parent.event.register('indexChanged', this, (index) => { // if (!this.view) return