Propagate exceptions thrown by access check interceptors.

src/objects.cc

Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index d0e40b94859e2fe81f0ec3d05bcffadd3431581b..6e2f260bf8bb073f17db3e7a0277b46f2d497569 100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -1885,10 +1885,8 @@ Maybe<bool> JSObject::SetPropertyWithFailedAccessCheck(
       return SetPropertyWithAccessor(it, value, should_throw);
     }
   } else {
-    Maybe<bool> result = SetPropertyWithInterceptorInternal(
-        it, interceptor, should_throw, value);
-    RETURN_VALUE_IF_SCHEDULED_EXCEPTION(isolate, Nothing<bool>());
-    if (result.IsJust()) return result;
+    return SetPropertyWithInterceptorInternal(it, interceptor, should_throw,
+                                              value);
   }
 
   isolate->ReportFailedAccessCheck(checked);

[jochen (gone - plz use gerrit), 2016/12/06 10:05:22]

maybe move these lines into the if () { } block, so it doesn't look like there
was a way to reach them from the else {} block?