src: fix useless call in permission.cc

FromMaybe() has no side effects and the return value is ignored.
Instead, if Set() fails, then another exception is pending, so
return early.

PR-URL: #46833
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Anatoli Papirovski <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>

Author: tniessen

src/permission/permission.cc

@@ -136,22 +136,23 @@ void Permission::ThrowAccessDenied(Environment* env,
                                    const std::string_view& res) {
   Local<Value> err = ERR_ACCESS_DENIED(env->isolate());
   CHECK(err->IsObject());
-  err.As<Object>()
-      ->Set(env->context(),
-            env->permission_string(),
-            v8::String::NewFromUtf8(env->isolate(),
-                                    PermissionToString(perm),
-                                    v8::NewStringType::kNormal)
-                .ToLocalChecked())
-      .FromMaybe(false);
-  err.As<Object>()
-      ->Set(env->context(),
-            env->resource_string(),
-            v8::String::NewFromUtf8(env->isolate(),
-                                    std::string(res).c_str(),
-                                    v8::NewStringType::kNormal)
-                .ToLocalChecked())
-      .FromMaybe(false);
+  if (err.As<Object>()
+          ->Set(env->context(),
+                env->permission_string(),
+                v8::String::NewFromUtf8(env->isolate(),
+                                        PermissionToString(perm),
+                                        v8::NewStringType::kNormal)
+                    .ToLocalChecked())
+          .IsNothing() ||
+      err.As<Object>()
+          ->Set(env->context(),
+                env->resource_string(),
+                v8::String::NewFromUtf8(env->isolate(),
+                                        std::string(res).c_str(),
+                                        v8::NewStringType::kNormal)
+                    .ToLocalChecked())
+          .IsNothing())
+    return;
   env->isolate()->ThrowException(err);
 }