<!--  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_context_bridge.cc
args->ThrowError(
        "Cannot bind an API on top of an existing property on the window "
        "object");
    return;
  }
  v8::Local<v8::Context> isolated_context =
      frame->WorldScriptContext(args->isolate(), WorldIDs::ISOLATED_WORLD_ID);
  {
    context_bridge::ObjectCache object_cache;
    v8::Context::Scope main_context_scope(main_context);
    v8::MaybeLocal<v8::Value> maybe_proxy = PassValueToOtherContext(
        isolated_context, main_context, api, &object_cache, false, 0);
    if (maybe_proxy.IsEmpty())
      return;
    auto proxy = maybe_proxy.ToLocalChecked();
    if (proxy->IsObject() && !proxy->IsTypedArray() &&
        !DeepFreeze(v8::Local<v8::Object>::Cast(proxy), main_context))
      return;
    global.SetReadOnlyNonConfigurable(key, proxy);
  }
}
gin_helper::Dictionary TraceKeyPath(const gin_helper::Dictionary& start,
                                    const std::vector<std::string>& key_path) {
  gin_helper::Dictionary current = start;
  for (size_t i = 0; i < key_path.size() - 1; i++) {
    CHECK(current.Get(key_path[i], &current));
  }
  return current;
}
void OverrideGlobalValueFromIsolatedWorld(