{"push_id":3582235965,"size":6,"distinct_size":6,"ref":"refs/heads/master","head":"2becfabb7d8ea2a47276d531e2d82924e510f438","before":"69b48df7608af4efc24198c9bc999d43d93dce6c","commits":[{"sha":"c5c79e5d718ad0777e014dd8c1e8d398d8172f3d","author":{"name":"Janic Duplessis","email":"f87e5dba9accef1a9a04e8942bedbac1246035be@gmail.com"},"message":"Release underlying resources when JS instance in GC'ed on iOS (#24405)\n\nSummary:\nOur Blob implementation was very problematic because it didn't release its underlying resource when the JS instance was dealocated. The main issue is that the fetch polyfill uses blobs by default if the module is available, which causes large memory leaks.\n\nThis fixes it by using the new jsi infra to attach a `jsi::HostObject` (`BlobCollector`)  to `Blob` instances. This way when the `Blob` is collected, the `BlobCollector` also gets collected. Using the `jsi::HostObject` dtor we can schedule the cleanup of native resources. This is definitely not the ideal solution but otherwise it would require rewriting the whole module using TurboModules + jsi.\n\nFixes #23801, #20352, #21092\n\n[General] [Fixed] - [Blob] Release underlying resources when JS instance in GC'ed\nPull Request resolved: https://github.com/facebook/react-native/pull/24405\n\nDifferential Revision: D15237418\n\nPulled By: cpojer\n\nfbshipit-source-id: 00a94a54b0b172fbc62324364b753d192ac7016a","distinct":true,"url":"https://api.github.com/repos/Rachelmorrell/react-native/commits/c5c79e5d718ad0777e014dd8c1e8d398d8172f3d"},{"sha":"b3f7d53b87413abdf302c521114e4d77aa92e07f","author":{"name":"michalchudziak","email":"6a1690bad2250f284b25bc76679612756607d8c9@gmail.com"},"message":"Fix Animated.Value value after animation if component was re-mounted (#24571)\n\nSummary:\nFixes https://github.com/facebook/react-native/issues/23712\n\nCurrently, It seems like `__nativeAnimatedValueListener` is not listening to the correct `onAnimatedValueUpdate` events if component was re-mounted. In this PR I'm attaching a new listener if the native view tag has changed.\n\n[General] [Fixed] - Fixed Animated.Value value after animation if component was re-mounted\nPull Request resolved: https://github.com/facebook/react-native/pull/24571\n\nDifferential Revision: D15237431\n\nPulled By: cpojer\n\nfbshipit-source-id: 1fe4e290ab45dfe6d1d364d8d7384aabf18d6610","distinct":true,"url":"https://api.github.com/repos/Rachelmorrell/react-native/commits/b3f7d53b87413abdf302c521114e4d77aa92e07f"},{"sha":"d6c33f915df09dddda81edb05302bd7f2fb587f4","author":{"name":"Dmitry Dushkin","email":"4dbbb9c16d185c0554a045e8b233dd177fc5217e@yandex-team.ru"},"message":"Consistent reporting native module name on crash on native side (#24704)\n\nSummary:\nPR https://github.com/facebook/react-native/pull/24633 introduced some inconsistency in crash messaging, this PR fix it. Asked by mhorowitz\n\n[General] [Added] - Consistent reporting native module name on crash on native side\nPull Request resolved: https://github.com/facebook/react-native/pull/24704\n\nDifferential Revision: D15237424\n\nPulled By: cpojer\n\nfbshipit-source-id: ded8db45b2a2ec9998ff33fdbecef3f12c19578f","distinct":true,"url":"https://api.github.com/repos/Rachelmorrell/react-native/commits/d6c33f915df09dddda81edb05302bd7f2fb587f4"},{"sha":"fdd8fadea84f475714a16b6f0ec433f898d09558","author":{"name":"Zeyad Salloum","email":"9e683166be9fb9edf63cf0ace54fc8c603ae865d@fb.com"},"message":"Revert D15237424: [react-native][PR] Consistent reporting native module name on crash on native side\n\nDifferential Revision:\nD15237424\n\nOriginal commit changeset: ded8db45b2a2\n\nfbshipit-source-id: 99d176e1ee796dbbe9957eda08992a8a76ba109b","distinct":true,"url":"https://api.github.com/repos/Rachelmorrell/react-native/commits/fdd8fadea84f475714a16b6f0ec433f898d09558"},{"sha":"05baf6272143667694585a14fb59657fdc93c3b1","author":{"name":"Zeyad Salloum","email":"9e683166be9fb9edf63cf0ace54fc8c603ae865d@fb.com"},"message":"Revert D15237418: [react-native][PR] [Blob] Release underlying resources when JS instance in GC'ed on iOS\n\nDifferential Revision:\nD15237418\n\nOriginal commit changeset: 00a94a54b0b1\n\nfbshipit-source-id: bb6c7aa3f5b6ae7f40965b96f1e0fd8eb7512015","distinct":true,"url":"https://api.github.com/repos/Rachelmorrell/react-native/commits/05baf6272143667694585a14fb59657fdc93c3b1"},{"sha":"2becfabb7d8ea2a47276d531e2d82924e510f438","author":{"name":"Florian","email":"b858a87c07b04c4568f51b0dce655f78d73c02b3@monochrom.tv"},"message":"Update eslint-plugin-react-native dependency in eslint-config-react-native-community package (#24686)\n\nSummary:\nI need this in my life because version 3.7.0 fixes [this](https://github.com/Intellicode/eslint-plugin-react-native/pull/220) issue.\n\n[General][fixed] - Fix an issue with `no-raw-text` rule for TypeScript by updating the eslint-plugin-react-native dependency to 3.7.0\nPull Request resolved: https://github.com/facebook/react-native/pull/24686\n\nDifferential Revision: D15237597\n\nPulled By: cpojer\n\nfbshipit-source-id: 42ee13997d238025d0c50d154951549392d3abc7","distinct":true,"url":"https://api.github.com/repos/Rachelmorrell/react-native/commits/2becfabb7d8ea2a47276d531e2d82924e510f438"}]}
{"actor":{"display_login":"pull"}}