status-im/react-native
2
0
Fork 0
mirror of https://github.com/status-im/react-native.git synced 2025-01-28 02:04:55 +00:00
Code Issues Projects Releases Wiki Activity

[ReactNative] Guard agains errors during reconciliation

Browse Source 
Summary:
@public

After refactoring the MessageQueue a guard was missing on around `batchedUpdates`
call.

Test Plan: Introduce an error on `getInitialState` of `AdsManagerTabsModalView.ios.js`
This commit is contained in:
Tadeu Zagallo 2015-06-25 09:38:54 -07:00
parent f383bf2b83
commit 5e71d352a6
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
Libraries/Utilities/MessageQueue.js
View File

@ -76,15 +76,17 @@ class MessageQueue {
* Public APIs * Public APIs
*/ */
processBatch(batch) { processBatch(batch) {
ReactUpdates.batchedUpdates(() => { guard(() => {
batch.forEach((call) => { ReactUpdates.batchedUpdates(() => {
let method = call.method === 'callFunctionReturnFlushedQueue' ? batch.forEach((call) => {
'__callFunction' : '__invokeCallback'; let method = call.method === 'callFunctionReturnFlushedQueue' ?
guard(() => this[method].apply(this, call.args)); '__callFunction' : '__invokeCallback';
guard(() => this[method].apply(this, call.args));
});
BridgeProfiling.profile('ReactUpdates.batchedUpdates()');
}); });
BridgeProfiling.profile('ReactUpdates.batchedUpdates()'); BridgeProfiling.profileEnd();
}); });
BridgeProfiling.profileEnd();
return this.flushedQueue(); return this.flushedQueue();
} }