refactor(runtime-dom): avoid unnecessary typeof checks during props patching

packages/runtime-dom/src/modules/props.ts

@@ -24,6 +24,7 @@ export function patchDOMProp(
     el[key] = value == null ? '' : value
     return
   }
+
   if (key === 'value' && el.tagName !== 'PROGRESS') {
     // store value as _value as well since
     // non-string values will be stringified.
@@ -34,29 +35,36 @@ export function patchDOMProp(
     }
     return
   }
-  if (value === '' && typeof el[key] === 'boolean') {
+
-    // e.g. <select multiple> compiles to { multiple: '' }
+  if (value === '' || value == null) {
-    el[key] = true
+    const type = typeof el[key]
-  } else if (value == null && typeof el[key] === 'string') {
+    if (value === '' && type === 'boolean') {
-    // e.g. <div :id="null">
+      // e.g. <select multiple> compiles to { multiple: '' }
-    el[key] = ''
+      el[key] = true
-    el.removeAttribute(key)
+      return
-  } else if ((value == null || value === '') && typeof el[key] === 'number') {
+    } else if (value == null && type === 'string') {
-    // e.g. <img :width="null">
+      // e.g. <div :id="null">
-    el[key] = 0
+      el[key] = ''
-    el.removeAttribute(key)
+      el.removeAttribute(key)
-  } else {
+      return
-    // some properties perform value validation and throw
+    } else if (type === 'number') {
-    try {
+      // e.g. <img :width="null">
-      el[key] = value
+      el[key] = 0
-    } catch (e) {
+      el.removeAttribute(key)
-      if (__DEV__) {
+      return
-        warn(
+    }
-          `Failed setting prop "${key}" on <${el.tagName.toLowerCase()}>: ` +
+  }
-            `value ${value} is invalid.`,
+
-          e
+  // some properties perform value validation and throw
-        )
+  try {
-      }
+    el[key] = value
+  } catch (e) {
+    if (__DEV__) {
+      warn(
+        `Failed setting prop "${key}" on <${el.tagName.toLowerCase()}>: ` +
+          `value ${value} is invalid.`,
+        e
+      )
     }
   }
 }