diff options
| -rw-r--r-- | dom/bindings/BindingUtils.cpp | 20 | ||||
| -rw-r--r-- | dom/bindings/BindingUtils.h | 2 | ||||
| -rw-r--r-- | dom/bindings/Codegen.py | 2 |
3 files changed, 20 insertions, 4 deletions
diff --git a/dom/bindings/BindingUtils.cpp b/dom/bindings/BindingUtils.cpp index 4d20a95f70..3c31ec0e6e 100644 --- a/dom/bindings/BindingUtils.cpp +++ b/dom/bindings/BindingUtils.cpp @@ -3431,7 +3431,7 @@ GetCustomElementReactionsStack(JS::Handle<JSObject*> aObj) // https://html.spec.whatwg.org/multipage/dom.html#htmlconstructor already_AddRefed<nsGenericHTMLElement> CreateHTMLElement(const GlobalObject& aGlobal, const JS::CallArgs& aCallArgs, - ErrorResult& aRv) + JS::Handle<JSObject*> aGivenProto, ErrorResult& aRv) { // Step 1. nsCOMPtr<nsPIDOMWindowInner> window = do_QueryInterface(aGlobal.GetAsSupports()); @@ -3566,7 +3566,23 @@ CreateHTMLElement(const GlobalObject& aGlobal, const JS::CallArgs& aCallArgs, return nullptr; } - // Step 11 is in the code output by CGClassConstructor. + // Step 11. + // Do prototype swizzling for upgrading a custom element here, for cases when + // we have a reflector already. If we don't have one yet, our caller will + // create it with the right proto (by calling DoGetOrCreateDOMReflector with + // that proto). + JS::Rooted<JSObject*> reflector(cx, element->GetWrapper()); + if (reflector) { + // reflector might be in different compartment. + JSAutoCompartment ac(cx, reflector); + JS::Rooted<JSObject*> givenProto(cx, aGivenProto); + if (!JS_WrapObject(cx, &givenProto) || + !JS_SetPrototype(cx, reflector, givenProto)) { + aRv.NoteJSContextException(cx); + return nullptr; + } + } + // Step 12 and Step 13. return element.forget(); } diff --git a/dom/bindings/BindingUtils.h b/dom/bindings/BindingUtils.h index d600acce8b..e583d0e060 100644 --- a/dom/bindings/BindingUtils.h +++ b/dom/bindings/BindingUtils.h @@ -3433,7 +3433,7 @@ GetCustomElementReactionsStack(JS::Handle<JSObject*> aObj); // that constructor function. already_AddRefed<nsGenericHTMLElement> CreateHTMLElement(const GlobalObject& aGlobal, const JS::CallArgs& aCallArgs, - ErrorResult& aRv); + JS::Handle<JSObject*> aGivenProto, ErrorResult& aRv); void SetDocumentAndPageUseCounter(JSContext* aCx, JSObject* aObject, diff --git a/dom/bindings/Codegen.py b/dom/bindings/Codegen.py index 5bbf77bb60..730465fee7 100644 --- a/dom/bindings/Codegen.py +++ b/dom/bindings/Codegen.py @@ -7542,7 +7542,7 @@ class CGPerSignatureCall(CGThing): argsPre.append("global") if isConstructor and idlNode.isHTMLConstructor(): - argsPre.append("args") + argsPre.extend(["args", "desiredProto"]) # For JS-implemented interfaces we do not want to base the # needsCx decision on the types involved, just on our extended |