<!--  As an open source project with a dedicated but small maintainer team, it can sometimes take a long time for issues to be addressed so please be patient and we will get back to you as soon as we can.
-->

### Preflight Checklist
<!-- Please ensure you've completed the following steps by replacing [ ] with [x]-->

* [x] I have read the [Contributing Guidelines](https://github.com/electron/electron/blob/master/CONTRIBUTING.md) for this project.
* [x] I agree to follow the [Code of Conduct](https://github.com/electron/electron/blob/master/CODE_OF_CONDUCT.md) that this project adheres to.
* [x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

### Issue Details

* **Electron Version:**
  * 10.1.3
* **Operating System:**
  * win7 sp1 x64

- without worldSafeExecuteJavaScript, `(webFrame|webContents).executeJavaScript('new Promise(resolve => setTimeout(() => resolve(2), 3000));')` works as expected; the executeJavaScript promise waits for the eval promise.
- in the case of webFrame the evaled promise is returned raw
- in the case of webContents the promise is marshaled
- with worldSafeExecuteJavaScript, both error with: "An object could not be cloned"
- webContents should continue marshaling promises, otherwise its an unnecessary breaking change
- webFrame should marshal promises for consistency with webContents
- the clone restriction would be on the promise resolve value

### To Reproduce
`(webFrame|webContents).executeJavaScript('new Promise(resolve => setTimeout(() => resolve(2), 3000));')`


shell/renderer/api/electron_api_web_frame.cc
success = gin::ConvertFromV8(isolate, result, &ret);
      if (try_catch.HasCaught()) {
        auto message = try_catch.Message();
        if (message.IsEmpty() ||
            !gin::ConvertFromV8(isolate, message->Get(), &error_message)) {
          error_message =
              "An unknown exception occurred while getting the result of "
              "the script";
        }
      }
    }
    if (!success) {
      
      if (callback_)
        std::move(callback_).Run(
            v8::Undefined(isolate),
            v8::Exception::Error(
                v8::String::NewFromUtf8(isolate, error_message.c_str())
                    .ToLocalChecked()));
      promise_.RejectWithErrorMessage(error_message);
    } else {
      v8::Local<v8::Context> context = promise_.GetContext();
      v8::Context::Scope context_scope(context);
      v8::Local<v8::Value> cloned_value = gin::ConvertToV8(isolate, ret);
      if (callback_)
        std::move(callback_).Run(cloned_value, v8::Undefined(isolate));
      promise_.Resolve(cloned_value);
    }
  }
  void Completed(