Clear caught error

This is not necessary (as proven by tests) because next invokeGuardedCallback clears it anyway. But I'll keep it for consistency with other calls.

Author: gaearon

packages/react-reconciler/src/ReactFiberCommitWork.new.js

@@ -140,7 +140,7 @@ import {
 } from './ReactHookEffectTags';
 import {didWarnAboutReassigningProps} from './ReactFiberBeginWork.new';
 import {doesFiberContain} from './ReactFiberTreeReflection';
-import {invokeGuardedCallback} from 'shared/ReactErrorUtils';
+import {invokeGuardedCallback, clearCaughtError} from 'shared/ReactErrorUtils';
 
 let didWarnAboutUndefinedSnapshotBeforeUpdate: Set<mixed> | null = null;
 if (__DEV__) {
@@ -171,6 +171,7 @@ function reportUncaughtErrorInDEV(error) {
     invokeGuardedCallback(null, () => {
       throw error;
     });
+    clearCaughtError();
   }
 }
 

packages/react-reconciler/src/ReactFiberCommitWork.old.js

@@ -140,7 +140,7 @@ import {
 } from './ReactHookEffectTags';
 import {didWarnAboutReassigningProps} from './ReactFiberBeginWork.old';
 import {doesFiberContain} from './ReactFiberTreeReflection';
-import {invokeGuardedCallback} from 'shared/ReactErrorUtils';
+import {invokeGuardedCallback, clearCaughtError} from 'shared/ReactErrorUtils';
 
 let didWarnAboutUndefinedSnapshotBeforeUpdate: Set<mixed> | null = null;
 if (__DEV__) {
@@ -171,6 +171,7 @@ function reportUncaughtErrorInDEV(error) {
     invokeGuardedCallback(null, () => {
       throw error;
     });
+    clearCaughtError();
   }
 }