Revert "Improve performance and reduce memory allocation" (#4611)

This reverts commit 79303ec.

compat/src/forwardRef.js

@@ -1,4 +1,5 @@
 import { options } from 'preact';
+import { assign } from './util';
 
 let oldDiffHook = options._diff;
 options._diff = vnode => {
@@ -24,13 +25,9 @@ export const REACT_FORWARD_SYMBOL =
  */
 export function forwardRef(fn) {
 	function Forwarded(props) {
-		if (!('ref' in props)) return fn(props, null);
-
-		let ref = props.ref;
-		delete props.ref;
-		const result = fn(props, ref);
-		props.ref = ref;
-		return result;
+		let clone = assign({}, props);
+		delete clone.ref;
+		return fn(clone, props.ref || null);
 	}
 
 	// mobx-react checks for this being present

jsx-runtime/src/index.js

@@ -35,9 +35,15 @@ function createVNode(type, props, key, isStaticChildren, __source, __self) {
 		ref,
 		i;
 
-	if ('ref' in props) {
-		ref = props.ref;
-		delete props.ref;
+	if ('ref' in normalizedProps) {
+		normalizedProps = {};
+		for (i in props) {
+			if (i == 'ref') {
+				ref = props[i];
+			} else {
+				normalizedProps[i] = props[i];
+			}
+		}
 	}
 
 	/** @type {VNode & { __source: any; __self: any }} */

jsx-runtime/test/browser/jsx-runtime.test.js

@@ -37,8 +37,7 @@ describe('Babel jsx/jsxDEV', () => {
 		const props = { ref };
 		const vnode = jsx('div', props);
 		expect(vnode.ref).to.equal(ref);
-		expect(vnode.props).to.equal(props);
-		expect(vnode.props.ref).to.equal(undefined);
+		expect(vnode.props).to.not.equal(props);
 	});
 
 	it('should not copy props wen there is no ref in props', () => {