src: remove outdated comments in contextify

Author: legendecas

src/node_contextify.cc

@@ -600,38 +600,8 @@ Intercepted ContextifyContext::PropertySetterCallback(
     return Intercepted::kNo;
   }
 
-  // V8 comment: As long as the context is not detached the contextual accesses
-  // are the same as regular accesses to `context->Global()`s data property.
-  // The only difference is that after detaching `args.Holder()` will
-  // become a new identity and will no longer be equal to `context->Global()`.
-  // TODO(Node.js): revise the code below as the "contextual"-ness of the
-  // store is not actually relevant here. Also, new variable declaration is
-  // reported by V8 via PropertyDefinerCallback.
   bool is_declared = is_declared_on_global_proxy || is_declared_on_sandbox;
 
-  /*
-    // true for x = 5
-    // false for this.x = 5
-    // false for Object.defineProperty(this, 'foo', ...)
-    // false for vmResult.x = 5 where vmResult = vm.runInContext();
-
-    bool is_contextual_store = ctx->global_proxy() != args.This();
-
-    // Indicator to not return before setting (undeclared) function declarations
-    // on the sandbox in strict mode, i.e. args.ShouldThrowOnError() = true.
-    // True for 'function f() {}', 'this.f = function() {}',
-    // 'var f = function()'.
-    // In effect only for 'function f() {}' because
-    // var f = function(), is_declared = true
-    // this.f = function() {}, is_contextual_store = false.
-    bool is_function = value->IsFunction();
-
-    bool is_declared = is_declared_on_global_proxy || is_declared_on_sandbox;
-    if (!is_declared && args.ShouldThrowOnError() && is_contextual_store &&
-        !is_function) {
-      return Intercepted::kNo;
-    }
-  */
   if (!is_declared && property->IsSymbol()) {
     return Intercepted::kNo;
   }