Don't wrap roots in Handle just to dereference immediately.

src/builtins/builtins-error.cc

 // Copyright 2016 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/builtins/builtins.h"
 #include "src/builtins/builtins-utils.h"
 
 #include "src/messages.h"
 #include "src/property-descriptor.h"
 #include "src/string-builder.h"
@@ skipping 37 matching lines @@
 
   // Collect the stack trace.
 
   RETURN_FAILURE_ON_EXCEPTION(isolate,
                               isolate->CaptureAndSetDetailedStackTrace(object));
 
   // Eagerly format the stack trace and set the stack property.
 
   Handle<Object> stack_trace =
       isolate->CaptureSimpleStackTrace(object, mode, caller);
-  if (!stack_trace->IsJSArray()) return *isolate->factory()->undefined_value();
+  if (!stack_trace->IsJSArray()) return isolate->heap()->undefined_value();
 
   Handle<Object> formatted_stack_trace;
   ASSIGN_RETURN_FAILURE_ON_EXCEPTION(
       isolate, formatted_stack_trace,
       ErrorUtils::FormatStackTrace(isolate, object, stack_trace));
 
   PropertyDescriptor desc;
   desc.set_configurable(true);
   desc.set_writable(true);
   desc.set_value(formatted_stack_trace);
@@ skipping 57 matching lines @@
   Handle<Object> undefined = isolate->factory()->undefined_value();
   const int template_index = MessageTemplate::kURIMalformed;
   RETURN_RESULT_OR_FAILURE(
       isolate,
       ErrorUtils::MakeGenericError(isolate, constructor, template_index,
                                    undefined, undefined, undefined, SKIP_NONE));
 }
 
 }  // namespace internal
 }  // namespace v8