Make derived blob auto release

wkh237 · Nov 23, 2016 · 49a0b8d · 49a0b8d
1 parent f1e0d80
commit 49a0b8d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/src/polyfill/Blob.js b/src/polyfill/Blob.js
@@ -73,6 +73,7 @@ export default class Blob extends EventTarget {
     cType = cType || {}
     this.cacheName = getBlobName()
     this.isRNFetchBlobPolyfill = true
+    this.isDerived = defer
     this.type = cType.type || 'text/plain'
     log.verbose('Blob constructor called', 'mime', this.type, 'type', typeof data, 'length', data?  data.length:0)
     this._ref = blobCacheDir + this.cacheName

diff --git a/src/polyfill/XMLHttpRequest.js b/src/polyfill/XMLHttpRequest.js
@@ -53,6 +53,7 @@ export default class XMLHttpRequest extends XMLHttpRequestEventTarget{
   _headers: any = {
     'Content-Type' : 'text/plain'
   };
+  _cleanUp : () => void = null;
   _body: any;
 
   // RNFetchBlob promise object, which has `progress`, `uploadProgress`, and
@@ -158,9 +159,16 @@ export default class XMLHttpRequest extends XMLHttpRequestEventTarget{
       log.debug('sending blob body', body._blobCreated)
       promise = new Promise((resolve, reject) => {
           body.onCreated((blob) => {
-              log.debug('body created send request')
-              body = RNFetchBlob.wrap(blob.getRNFetchBlobRef())
-              resolve()
+            // when the blob is derived (not created by RN developer), the blob
+            // will be released after XMLHttpRequest sent
+            if(blob.isDerived) {
+              this._cleanUp = () => {
+                blob.close()
+              }
+            }
+            log.debug('body created send request')
+            body = RNFetchBlob.wrap(blob.getRNFetchBlobRef())
+            resolve()
           })
         })
     }