From 7bb0ff535c828d5720f4ad321fe217e5271abec6 Mon Sep 17 00:00:00 2001 From: Tadeu Zagallo Date: Thu, 3 Sep 2015 12:02:39 -0700 Subject: [PATCH] [ReactNative][Packager] Fix source maps for minified sources Summary: The packager was ignoring minification for source map requests. --- packager/react-packager/src/Bundler/Bundle.js | 4 ++++ packager/react-packager/src/Server/index.js | 4 +++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/packager/react-packager/src/Bundler/Bundle.js b/packager/react-packager/src/Bundler/Bundle.js index b7920ebb8..88431cbf6 100644 --- a/packager/react-packager/src/Bundler/Bundle.js +++ b/packager/react-packager/src/Bundler/Bundle.js @@ -186,6 +186,10 @@ class Bundle { options = options || {}; + if (options.minify) { + return this.getMinifiedSourceAndMap().map; + } + if (this._shouldCombineSourceMaps) { return this._getCombinedSourceMaps(options); } diff --git a/packager/react-packager/src/Server/index.js b/packager/react-packager/src/Server/index.js index d727fd52a..7d65d0a2e 100644 --- a/packager/react-packager/src/Server/index.js +++ b/packager/react-packager/src/Server/index.js @@ -370,7 +370,9 @@ class Server { res.end(bundleSource); Activity.endEvent(startReqEventId); } else if (requestType === 'map') { - var sourceMap = JSON.stringify(p.getSourceMap()); + var sourceMap = JSON.stringify(p.getSourceMap({ + minify: options.minify, + })); res.setHeader('Content-Type', 'application/json'); res.end(sourceMap); Activity.endEvent(startReqEventId);