@lwc/engine-core 5.0.4 → 5.0.5

package/dist/index.js

@@ -1,7 +1,7 @@
 /**
  * Copyright (C) 2023 salesforce.com, inc.
  */
-import { noop, StringToLowerCase, isNull, ArrayPush as ArrayPush$1, ArrayJoin, isFrozen, isUndefined as isUndefined$1, defineProperty, ArrayIndexOf, ArraySplice, create, seal, isArray as isArray$1, isFunction as isFunction$1, keys, hasOwnProperty as hasOwnProperty$1, globalThis as globalThis$1, entries, AriaPropNameToAttrNameMap, getPropertyDescriptor, forEach, defineProperties, getPrototypeOf as getPrototypeOf$1, setPrototypeOf, assign, isObject, freeze, KEY__SYNTHETIC_MODE, isString, assert, toString as toString$1, getOwnPropertyDescriptor as getOwnPropertyDescriptor$1, isFalse, LWC_VERSION_COMMENT_REGEX, LWC_VERSION, getOwnPropertyNames as getOwnPropertyNames$1, getOwnPropertyDescriptors, htmlPropertyToAttribute, ArraySlice, ArrayMap, KEY__SCOPED_CSS, kebabCaseToCamelCase, StringCharCodeAt, XML_NAMESPACE, XLINK_NAMESPACE, StringSlice, ArrayShift, ArrayUnshift, isTrue, SVG_NAMESPACE, KEY__SHADOW_STATIC, KEY__SHADOW_RESOLVER, ArraySome, ArrayPop, isAPIFeatureEnabled, isNumber, StringReplace, LOWEST_API_VERSION, KEY__NATIVE_GET_ELEMENT_BY_ID, KEY__NATIVE_QUERY_SELECTOR_ALL, ID_REFERENCING_ATTRIBUTES_SET, KEY__SHADOW_TOKEN, ArrayFilter, StringSplit, arrayEvery, ArrayIncludes, ArrayCopyWithin, ArrayFill, ArraySort, ArrayReverse } from '@lwc/shared';
+import { noop, StringToLowerCase, isNull, ArrayPush as ArrayPush$1, ArrayJoin, isFrozen, isUndefined as isUndefined$1, defineProperty, ArrayIndexOf, ArraySplice, create, seal, isArray as isArray$1, isFunction as isFunction$1, keys, hasOwnProperty as hasOwnProperty$1, globalThis as globalThis$1, AriaPropNameToAttrNameMap, getOwnPropertyDescriptor as getOwnPropertyDescriptor$1, forEach, getPropertyDescriptor, defineProperties, getPrototypeOf as getPrototypeOf$1, setPrototypeOf, assign, isObject, freeze, KEY__SYNTHETIC_MODE, isString, assert, toString as toString$1, isFalse, LWC_VERSION_COMMENT_REGEX, LWC_VERSION, getOwnPropertyNames as getOwnPropertyNames$1, getOwnPropertyDescriptors, htmlPropertyToAttribute, ArraySlice, ArrayMap, KEY__SCOPED_CSS, kebabCaseToCamelCase, StringCharCodeAt, XML_NAMESPACE, XLINK_NAMESPACE, StringSlice, ArrayShift, ArrayUnshift, isTrue, SVG_NAMESPACE, KEY__SHADOW_STATIC, KEY__SHADOW_RESOLVER, ArraySome, ArrayPop, isAPIFeatureEnabled, isNumber, StringReplace, LOWEST_API_VERSION, KEY__NATIVE_GET_ELEMENT_BY_ID, KEY__NATIVE_QUERY_SELECTOR_ALL, ID_REFERENCING_ATTRIBUTES_SET, KEY__SHADOW_TOKEN, ArrayFilter, StringSplit, arrayEvery, ArrayIncludes, ArrayCopyWithin, ArrayFill, ArraySort, ArrayReverse } from '@lwc/shared';
 export { setFeatureFlag, setFeatureFlagForTest } from '@lwc/features';
 
 /*
@@ -420,17 +420,6 @@ var _a, _b;
 const instrumentDef = (_a = globalThis$1.__lwc_instrument_cmp_def) !== null && _a !== void 0 ? _a : noop;
 const instrumentInstance = (_b = globalThis$1.__lwc_instrument_cmp_instance) !== null && _b !== void 0 ? _b : noop;
 
-/*
- * Copyright (c) 2018, salesforce.com, inc.
- * All rights reserved.
- * SPDX-License-Identifier: MIT
- * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/MIT
- */
-// This is a temporary workaround to get the @lwc/engine-server to evaluate in node without having
-// to inject at runtime.
-const HTMLElementConstructor = typeof HTMLElement !== 'undefined' ? HTMLElement : function () { };
-const HTMLElementPrototype = HTMLElementConstructor.prototype;
-
 /*
  * Copyright (c) 2023, salesforce.com, inc.
  * All rights reserved.
@@ -439,36 +428,48 @@ const HTMLElementPrototype = HTMLElementConstructor.prototype;
  */
 // Apply ARIA string reflection behavior to a prototype.
 // This is deliberately kept separate from @lwc/aria-reflection. @lwc/aria-reflection is a global polyfill that is
-// needed for backwards compatibility in LEX, whereas this is designed to only apply to our own
+// needed for backwards compatibility in LEX, whereas `applyAriaReflection` is designed to only apply to our own
 // LightningElement/BaseBridgeElement prototypes.
-// Note we only need to handle ARIA reflections that aren't already in Element.prototype
-const ariaReflectionPolyfillDescriptors = create(null);
-for (const [propName, attrName] of entries(AriaPropNameToAttrNameMap)) {
-    if (isUndefined$1(getPropertyDescriptor(HTMLElementPrototype, propName))) {
-        // Note that we need to call this.{get,set,has,remove}Attribute rather than dereferencing
-        // from Element.prototype, because these methods are overridden in LightningElement.
-        ariaReflectionPolyfillDescriptors[propName] = {
-            get() {
-                return this.getAttribute(attrName);
-            },
-            set(newValue) {
-                // TODO [#3284]: there is disagreement between browsers and the spec on how to treat undefined
-                // Our historical behavior is to only treat null as removing the attribute
-                // See also https://github.com/w3c/aria/issues/1858
-                if (isNull(newValue)) {
-                    this.removeAttribute(attrName);
-                }
-                else {
-                    this.setAttribute(attrName, newValue);
-                }
-            },
-            // configurable and enumerable to allow it to be overridden – this mimics Safari's/Chrome's behavior
-            configurable: true,
-            enumerable: true,
-        };
+function applyAriaReflection(prototype) {
+    for (const propName of keys(AriaPropNameToAttrNameMap)) {
+        const attrName = AriaPropNameToAttrNameMap[propName];
+        if (isUndefined$1(getOwnPropertyDescriptor$1(prototype, propName))) {
+            // Note that we need to call this.{get,set,has,remove}Attribute rather than dereferencing
+            // from Element.prototype, because these methods are overridden in LightningElement.
+            defineProperty(prototype, propName, {
+                get() {
+                    return this.getAttribute(attrName);
+                },
+                set(newValue) {
+                    // TODO [#3284]: there is disagreement between browsers and the spec on how to treat undefined
+                    // Our historical behavior is to only treat null as removing the attribute
+                    // See also https://github.com/w3c/aria/issues/1858
+                    if (isNull(newValue)) {
+                        this.removeAttribute(attrName);
+                    }
+                    else {
+                        this.setAttribute(attrName, newValue);
+                    }
+                },
+                // configurable and enumerable to allow it to be overridden – this mimics Safari's/Chrome's behavior
+                configurable: true,
+                enumerable: true,
+            });
+        }
     }
 }
 
+/*
+ * Copyright (c) 2018, salesforce.com, inc.
+ * All rights reserved.
+ * SPDX-License-Identifier: MIT
+ * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/MIT
+ */
+// This is a temporary workaround to get the @lwc/engine-server to evaluate in node without having
+// to inject at runtime.
+const HTMLElementConstructor = typeof HTMLElement !== 'undefined' ? HTMLElement : function () { };
+const HTMLElementPrototype = HTMLElementConstructor.prototype;
+
 /*
  * Copyright (c) 2018, salesforce.com, inc.
  * All rights reserved.
@@ -1841,22 +1842,11 @@ const lightningBasedDescriptors = create(null);
 for (const propName in HTMLElementOriginalDescriptors) {
     lightningBasedDescriptors[propName] = createBridgeToElementDescriptor(propName, HTMLElementOriginalDescriptors[propName]);
 }
+defineProperties(LightningElement.prototype, lightningBasedDescriptors);
 // Apply ARIA reflection to LightningElement.prototype, on both the browser and server.
 // This allows `this.aria*` property accessors to work from inside a component, and to reflect `aria-*` attrs.
 // Note this works regardless of whether the global ARIA reflection polyfill is applied or not.
-if (process.env.IS_BROWSER) {
-    // In the browser, we use createBridgeToElementDescriptor, so we can get the normal reactivity lifecycle for
-    // aria* properties
-    for (const [propName, descriptor] of entries(ariaReflectionPolyfillDescriptors)) {
-        lightningBasedDescriptors[propName] = createBridgeToElementDescriptor(propName, descriptor);
-    }
-}
-else {
-    // On the server, we cannot use createBridgeToElementDescriptor because getAttribute/setAttribute are
-    // not supported on HTMLElement. So apply the polyfill directly on top of LightningElement
-    defineProperties(LightningElement.prototype, ariaReflectionPolyfillDescriptors);
-}
-defineProperties(LightningElement.prototype, lightningBasedDescriptors);
+applyAriaReflection(LightningElement.prototype);
 defineProperty(LightningElement, 'CustomElementConstructor', {
     get() {
         // If required, a runtime-specific implementation must be defined.
@@ -2706,8 +2696,7 @@ function HTMLBridgeElementFactory(SuperClass, publicProperties, methods, observe
                 // and can break tooling that expects it to be iterable or defined, e.g. Jest:
                 // https://github.com/jestjs/jest/blob/b4c9587/packages/pretty-format/src/plugins/DOMElement.ts#L95
                 // It also doesn't make sense to override e.g. "constructor".
-                .filter((propName) => !(propName in HTMLElementPrototype) &&
-                !(propName in ariaReflectionPolyfillDescriptors)));
+                .filter((propName) => !(propName in HTMLElementPrototype)));
             for (const propName of nonPublicPropertiesToWarnOn) {
                 if (ArrayIndexOf.call(publicProperties, propName) === -1) {
                     descriptors[propName] = createAccessorThatWarns(propName);
@@ -2777,22 +2766,21 @@ function HTMLBridgeElementFactory(SuperClass, publicProperties, methods, observe
     defineProperties(HTMLBridgeElement.prototype, descriptors);
     return HTMLBridgeElement;
 }
-// We do some special handling of non-standard ARIA props like ariaLabelledBy as well as props without (as of this
-// writing) broad cross-browser support like ariaBrailleLabel. This is so the reflection works correctly and preserves
-// backwards compatibility with the previous global polyfill approach.
-//
-// The goal here is to expose `elm.aria*` property accessors to work from outside a component, and to reflect `aria-*`
-// attrs. This is especially important because the template compiler compiles aria-* attrs on components to aria* props.
-// Note this works regardless of whether the global ARIA reflection polyfill is applied or not.
-//
-// Also note this ARIA reflection only really makes sense in the browser. On the server, there is no
-// `renderedCallback()`, so you cannot do e.g. `this.template.querySelector('x-child').ariaBusy = 'true'`. So we don't
-// need to expose ARIA props outside the LightningElement
-const basePublicProperties = [
-    ...getOwnPropertyNames$1(HTMLElementOriginalDescriptors),
-    ...(process.env.IS_BROWSER ? getOwnPropertyNames$1(ariaReflectionPolyfillDescriptors) : []),
-];
-const BaseBridgeElement = HTMLBridgeElementFactory(HTMLElementConstructor, basePublicProperties, [], [], null, false);
+const BaseBridgeElement = HTMLBridgeElementFactory(HTMLElementConstructor, getOwnPropertyNames$1(HTMLElementOriginalDescriptors), [], [], null, false);
+if (process.env.IS_BROWSER) {
+    // This ARIA reflection only really makes sense in the browser. On the server, there is no `renderedCallback()`,
+    // so you cannot do e.g. `this.template.querySelector('x-child').ariaBusy = 'true'`. So we don't need to expose
+    // ARIA props outside the LightningElement
+    //
+    // Apply ARIA reflection to HTMLBridgeElement.prototype. This allows `elm.aria*` property accessors to work from
+    // outside a component, and to reflect `aria-*` attrs. This is especially important because the template compiler
+    // compiles aria-* attrs on components to aria* props.
+    // Note this works regardless of whether the global ARIA reflection polyfill is applied or not.
+    //
+    // Also note that we apply this to BaseBridgeElement.prototype to avoid excessively redefining property
+    // accessors inside the HTMLBridgeElementFactory.
+    applyAriaReflection(BaseBridgeElement.prototype);
+}
 freeze(BaseBridgeElement);
 seal(BaseBridgeElement.prototype);
 
@@ -7325,5 +7313,5 @@ function readonly(obj) {
 }
 
 export { LightningElement, profilerControl as __unstable__ProfilerControl, reportingControl as __unstable__ReportingControl, api$1 as api, computeShadowAndRenderMode, connectRootElement, createContextProviderWithRegister, createVM, disconnectRootElement, freezeTemplate, getAssociatedVMIfPresent, getComponentAPIVersion, getComponentConstructor, getComponentDef, getComponentHtmlPrototype, hydrateRoot, isComponentConstructor, parseFragment, parseSVGFragment, readonly, registerComponent, registerDecorators, registerTemplate, runFormAssociatedCallback, runFormDisabledCallback, runFormResetCallback, runFormStateRestoreCallback, sanitizeAttribute, setHooks, swapComponent, swapStyle, swapTemplate, track, unwrap, wire };
-/** version: 5.0.4 */
+/** version: 5.0.5 */
 //# sourceMappingURL=index.js.map