--- a/js/src/frontend/Parser.cpp
+++ b/js/src/frontend/Parser.cpp
@@ -282,29 +282,23 @@ EvalSharedContext::EvalSharedContext(Exc
     computeThisBinding(enclosingScope);
 
     // Like all things Debugger, Debugger.Frame.eval needs special
     // handling. Since the environment chain of such evals are non-syntactic
     // (DebuggerEnvironmentProxy is not an EnvironmentObject), computing the
     // this binding with respect to enclosingScope is incorrect if the
     // Debugger.Frame is a function frame. Recompute the this binding if we
     // are such an eval.
-    if (enclosingEnv && enclosingScope->kind() == ScopeKind::NonSyntactic) {
-        // For Debugger.Frame.eval with bindings, the environment chain may
-        // have more than the DebugEnvironmentProxy.
-        JSObject* env = enclosingEnv;
+    if (enclosingEnv && enclosingEnv->is<DebugEnvironmentProxy>()) {
+        JSObject* env = &enclosingEnv->as<DebugEnvironmentProxy>().environment();
         while (env) {
-            if (env->is<DebugEnvironmentProxy>())
-                env = &env->as<DebugEnvironmentProxy>().environment();
-
             if (env->is<CallObject>()) {
                 computeThisBinding(env->as<CallObject>().callee().nonLazyScript()->bodyScope());
                 break;
             }
-
             env = env->enclosingEnvironment();
         }
     }
 }
 
 bool
 ParseContext::init()
 {

deleted file mode 100644
--- a/js/src/jit-test/tests/debug/Frame-evalWithBindings-15.js
+++ /dev/null
@@ -1,14 +0,0 @@
-var g = newGlobal();
-var dbg = new Debugger(g);
-
-dbg.onDebuggerStatement = function (frame) {
-  // The bindings object is unused but adds another environment on the
-  // environment chain. Make sure 'this' computes the right value in light of
-  // this.
-  frame.evalWithBindings(`assertEq(this, foo);`, { bar: 42 });
-};
-
-g.eval(`
-var foo = { bar: function() { debugger; } };
-foo.bar();
-`);