scope is not needed

HtmlUnit · Aug 4, 2024 · 24ca458 · 24ca458
1 parent ddd9b83
commit 24ca458
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/main/java/org/htmlunit/javascript/JavaScriptEngine.java b/src/main/java/org/htmlunit/javascript/JavaScriptEngine.java
@@ -722,7 +722,7 @@ public Script compile(final HtmlPage owningPage, final Scriptable scope, final S
             LOG.trace("Javascript compile " + sourceName + newline + sourceCode + newline);
         }
 
-        final HtmlUnitContextAction action = new HtmlUnitContextAction(scope, owningPage) {
+        final HtmlUnitContextAction action = new HtmlUnitContextAction(owningPage) {
             @Override
             public Object doRun(final Context cx) {
                 return cx.compileString(sourceCode, sourceName, startLine, null);
@@ -803,7 +803,7 @@ public Object execute(final HtmlPage page, final Scriptable scope, final Script
             return null;
         }
 
-        final HtmlUnitContextAction action = new HtmlUnitContextAction(scope, page) {
+        final HtmlUnitContextAction action = new HtmlUnitContextAction(page) {
             @Override
             public Object doRun(final Context cx) {
                 return script.exec(cx, scope);
@@ -864,7 +864,7 @@ public Object callFunction(final HtmlPage page, final Function function,
             return null;
         }
 
-        final HtmlUnitContextAction action = new HtmlUnitContextAction(scope, page) {
+        final HtmlUnitContextAction action = new HtmlUnitContextAction(page) {
             @Override
             public Object doRun(final Context cx) {
                 if (ScriptRuntime.hasTopCall(cx)) {
@@ -903,11 +903,9 @@ public boolean isScriptRunning() {
      * "guarantees proper association of Context instances with the current thread and is faster".
      */
     private abstract class HtmlUnitContextAction implements ContextAction<Object> {
-        private final Scriptable scope_;
         private final HtmlPage page_;
 
-        HtmlUnitContextAction(final Scriptable scope, final HtmlPage page) {
-            scope_ = scope;
+        HtmlUnitContextAction(final HtmlPage page) {
             page_ = page;
         }