Warn for forwardRef(memo(...))

packages/react/src/__tests__/forwardRef-test.js

@@ -353,4 +353,21 @@ describe('forwardRef', () => {
     expect(ref.current).toBe(null);
     expect(differentRef.current.type).toBe('div');
   });
+
+  it('warns on forwardRef(memo(...))', () => {
+    expect(() => {
+      React.forwardRef(
+        React.memo((props, ref) => {
+          return null;
+        }),
+      );
+    }).toWarnDev(
+      [
+        'Warning: forwardRef requires a render function but received a `memo` ' +
+          'component. Instead of forwardRef(memo(...)), use ' +
+          'memo(forwardRef(...)).',
+      ],
+      {withoutStack: true},
+    );
+  });
 });

packages/react/src/forwardRef.js

@@ -5,15 +5,22 @@
  * LICENSE file in the root directory of this source tree.
  */
 
-import {REACT_FORWARD_REF_TYPE} from 'shared/ReactSymbols';
+import {REACT_FORWARD_REF_TYPE, REACT_MEMO_TYPE} from 'shared/ReactSymbols';
 
 import warningWithoutStack from 'shared/warningWithoutStack';
 
 export default function forwardRef<Props, ElementType: React$ElementType>(
   render: (props: Props, ref: React$Ref<ElementType>) => React$Node,
 ) {
   if (__DEV__) {
-    if (typeof render !== 'function') {
+    if (render != null && render.$$typeof === REACT_MEMO_TYPE) {
+      warningWithoutStack(
+        false,
+        'forwardRef requires a render function but received a `memo` ' +
+          'component. Instead of forwardRef(memo(...)), use ' +
+          'memo(forwardRef(...)).',
+      );
+    } else if (typeof render !== 'function') {
       warningWithoutStack(
         false,
         'forwardRef requires a render function but was given %s.',