@lwc/engine-core 8.3.0 → 8.4.0

package/dist/index.js

@@ -293,6 +293,16 @@ function toPrettyMemberNotation(parent, child) {
         return `${toString$1(parent)}[${JSON.stringify(child)}]`;
     }
 }
+function safelyCallGetter(target, key) {
+    // Arbitrary getters can throw. We don't want to throw an error just due to dev-mode-only mutation tracking
+    // (which is only used for performance debugging) so ignore errors here.
+    try {
+        return target[key];
+    }
+    catch (_err) {
+        /* ignore */
+    }
+}
 /**
  * Flush all the logs we've written so far and return the current logs.
  */
@@ -361,7 +371,7 @@ function trackTargetForMutationLogging(key, target) {
         // Deeply traverse arrays and objects to track every object within
         if (isArray$1(target)) {
             for (let i = 0; i < target.length; i++) {
-                trackTargetForMutationLogging(toPrettyMemberNotation(key, i), target[i]);
+                trackTargetForMutationLogging(toPrettyMemberNotation(key, i), safelyCallGetter(target, i));
             }
         }
         else {
@@ -370,10 +380,10 @@ function trackTargetForMutationLogging(key, target) {
             // https://github.com/salesforce/observable-membrane/blob/b85417f/src/base-handler.ts#L142-L143
             // Note this code path is very hot, hence doing two separate for-loops rather than creating a new array.
             for (const prop of getOwnPropertyNames$1(target)) {
-                trackTargetForMutationLogging(toPrettyMemberNotation(key, prop), target[prop]);
+                trackTargetForMutationLogging(toPrettyMemberNotation(key, prop), safelyCallGetter(target, prop));
             }
             for (const prop of getOwnPropertySymbols$1(target)) {
-                trackTargetForMutationLogging(toPrettyMemberNotation(key, prop), target[prop]);
+                trackTargetForMutationLogging(toPrettyMemberNotation(key, prop), safelyCallGetter(target, prop));
             }
         }
     }
@@ -666,8 +676,9 @@ for (const [propName, attrName] of entries(AriaPropNameToAttrNameMap)) {
                 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
+                // TODO [#3284]: According to the spec, IDL nullable type values
+                // (null and undefined) should remove the attribute; however, we
+                // only do so in the case of null for historical reasons.
                 // See also https://github.com/w3c/aria/issues/1858
                 if (isNull(newValue)) {
                     this.removeAttribute(attrName);
@@ -8005,7 +8016,10 @@ function validateClassAttr(vnode, elm, data, renderer) {
     let nodesAreCompatible = true;
     let readableVnodeClassname;
     const elmClassName = getAttribute(elm, 'class');
-    if (!isUndefined$1(className) && String(className) !== elmClassName) {
+    if (!isUndefined$1(className) &&
+        String(className) !== elmClassName &&
+        // No mismatch if SSR `class` attribute is missing and CSR `class` is the empty string
+        !(className === '' && isNull(elmClassName))) {
         // className is used when class is bound to an expr.
         nodesAreCompatible = false;
         // stringify for pretty-printing
@@ -8443,5 +8457,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, shouldBeFormAssociated, swapComponent, swapStyle, swapTemplate, track, unwrap, wire };
-/** version: 8.3.0 */
+/** version: 8.4.0 */
 //# sourceMappingURL=index.js.map