url: do not use HandleScope in ToObject

It is not needed / invalidates the returned value unlike EscapableHandleScope PR-URL: #14096 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Timothy Gu <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Jul 18, 2017 · d23c49f · d23c49f
1 parent 784102f
commit d23c49f
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/src/node_url.cc b/src/node_url.cc
@@ -2085,7 +2085,6 @@ static void DomainToUnicode(const FunctionCallbackInfo<Value>& args) {
 const Local<Value> URL::ToObject(Environment* env) const {
   Isolate* isolate = env->isolate();
   Local<Context> context = env->context();
-  HandleScope handle_scope(isolate);
   Context::Scope context_scope(context);
 
   const Local<Value> undef = Undefined(isolate);