Remove RETURN_RESULT macro

src/messages.cc

 // Copyright 2011 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/messages.h"
 
 #include <memory>
 
 #include "src/api.h"
 #include "src/execution.h"
@@ skipping 580 matching lines @@
     builder.AppendCString("new ");
     if (IsNonEmptyString(function_name)) {
       builder.AppendString(Handle<String>::cast(function_name));
     } else {
       builder.AppendCString("<anonymous>");
     }
   } else if (IsNonEmptyString(function_name)) {
     builder.AppendString(Handle<String>::cast(function_name));
   } else {
     AppendFileLocation(isolate_, this, &builder);
-    RETURN_RESULT(isolate_, builder.Finish(), String);
+    return builder.Finish();
   }
 
   builder.AppendCString(" (");
   AppendFileLocation(isolate_, this, &builder);
   builder.AppendCString(")");
 
-  RETURN_RESULT(isolate_, builder.Finish(), String);
+  return builder.Finish();
 }
 
 int JSStackFrame::GetPosition() const { return code_->SourcePosition(offset_); }
 
 bool JSStackFrame::HasScript() const {
   return function_->shared()->script()->IsScript();
 }
 
 Handle<Script> JSStackFrame::GetScript() const {
   return handle(Script::cast(function_->shared()->script()), isolate_);
@@ skipping 108 matching lines @@
 
   if (IsNonEmptyString(function_name)) {
     builder.AppendString(Handle<String>::cast(function_name));
     builder.AppendCString(" (");
   }
 
   AppendFileLocation(isolate_, this, &builder);
 
   if (IsNonEmptyString(function_name)) builder.AppendCString(")");
 
-  RETURN_RESULT(isolate_, builder.Finish(), String);
+  return builder.Finish();
 }
 
 FrameArrayIterator::FrameArrayIterator(Isolate* isolate,
                                        Handle<FrameArray> array, int frame_ix)
     : isolate_(isolate), array_(array), next_frame_ix_(frame_ix) {}
 
 bool FrameArrayIterator::HasNext() const {
   return (next_frame_ix_ < array_->FrameCount());
 }
 
@@ skipping 188 matching lines @@
         builder.AppendCString("<error: ");
         builder.AppendString(maybe_frame_string.ToHandleChecked());
         builder.AppendCString("<error>");
       }
     } else {
       // CallSite.toString completed without throwing.
       builder.AppendString(maybe_frame_string.ToHandleChecked());
     }
   }
 
-  RETURN_RESULT(isolate, builder.Finish(), Object);
+  return builder.Finish();
 }
 
 Handle<String> MessageTemplate::FormatMessage(Isolate* isolate,
                                               int template_index,
                                               Handle<Object> arg) {
   Factory* factory = isolate->factory();
   Handle<String> result_string = Object::NoSideEffectsToString(isolate, arg);
   MaybeHandle<String> maybe_result_string = MessageTemplate::FormatMessage(
       template_index, result_string, factory->empty_string(),
       factory->empty_string());
@@ skipping 221 matching lines @@
   DCHECK(mode != SKIP_UNTIL_SEEN);
 
   Handle<Object> no_caller;
   Handle<String> msg = FormatMessage(isolate, template_index, arg0, arg1, arg2);
   return ErrorUtils::Construct(isolate, constructor, constructor, msg, mode,
                                no_caller, false);
 }
 
 }  // namespace internal
 }  // namespace v8