From d9a053d2872dedc95c37af7a018e1ad3947f9ced Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Paul=20O=E2=80=99Shannessy?= Date: Fri, 15 Aug 2014 17:51:41 -0700 Subject: [PATCH] Don't force sourcemaps on for JSXTransformer APIs In case somebody doesn't want sourcemaps (eg, react-rails) or r.js --- vendor/browser-transforms.js | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/vendor/browser-transforms.js b/vendor/browser-transforms.js index 74225efe6706b..c3783d085f871 100644 --- a/vendor/browser-transforms.js +++ b/vendor/browser-transforms.js @@ -42,15 +42,16 @@ var supportsAccessors = Object.prototype.hasOwnProperty('__defineGetter__'); */ function transformReact(source, options) { // TODO: just use react-tools + options = options || {}; var visitorList; - if (options && options.harmony) { + if (options.harmony) { visitorList = visitors.getAllVisitors(); } else { visitorList = visitors.transformVisitors.react; } return transform(visitorList, source, { - sourceMap: supportsAccessors + sourceMap: supportsAccessors && options.sourceMap }); } @@ -250,11 +251,11 @@ function loadScripts(scripts) { } scripts.forEach(function(script, i) { - var options; + var options = { + sourceMap: true + }; if (/;harmony=true(;|$)/.test(script.type)) { - options = { - harmony: true - }; + options.harmony = true } // script.async is always true for non-javascript script tags