[isolate] remove redundant return

src/isolate.cc

 // Copyright 2012 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/isolate.h"
 
 #include <stdlib.h>
 
 #include <fstream>  // NOLINT(readability/streams)
 #include <sstream>
@@ skipping 1802 matching lines @@
                                                 Handle<JSPromise> promise);
 
 bool PromiseHandlerCheck(Isolate* isolate, Handle<JSReceiver> handler,
                          Handle<JSObject> deferred) {
   // Recurse to the forwarding Promise, if any. This may be due to
   //  - await reaction forwarding to the throwaway Promise, which has
   //    a dependency edge to the outer Promise.
   //  - PromiseIdResolveHandler forwarding to the output of .then
   //  - Promise.all/Promise.race forwarding to a throwaway Promise, which
   //    has a dependency edge to the generated outer Promise.
+  // Otherwise, this is a real reject handler for the Promise.
   Handle<Symbol> key = isolate->factory()->promise_forwarding_handler_symbol();
   Handle<Object> forwarding_handler = JSReceiver::GetDataProperty(handler, key);
   if (forwarding_handler->IsUndefined(isolate)) {
     return true;
   }
 
   // TODO(gsathya): Remove this once we get rid of deferred objects.
   Handle<String> promise_str = isolate->factory()->promise_string();
   Handle<Object> deferred_promise_obj =
       JSObject::GetDataProperty(deferred, promise_str);
   if (!deferred_promise_obj->IsJSPromise()) {
     return true;
   }
 
   return InternalPromiseHasUserDefinedRejectHandler(
       isolate, Handle<JSPromise>::cast(deferred_promise_obj));
-
-  // Otherwise, this is a real reject handler for the Promise
-  return true;
 }
 
 bool InternalPromiseHasUserDefinedRejectHandler(Isolate* isolate,
                                                 Handle<JSPromise> promise) {
   // If this promise was marked as being handled by a catch block
   // in an async function, then it has a user-defined reject handler.
   if (promise->handled_hint()) return true;
 
   // If this Promise is subsumed by another Promise (a Promise resolved
   // with another Promise, or an intermediate, hidden, throwaway Promise
@@ skipping 1798 matching lines @@
   // Then check whether this scope intercepts.
   if ((flag & intercept_mask_)) {
     intercepted_flags_ |= flag;
     return true;
   }
   return false;
 }
 
 }  // namespace internal
 }  // namespace v8