From e6c93f9246f2801e200864a74aec145410f13803 Mon Sep 17 00:00:00 2001
From: Ben Alpert <spicyjalapeno@gmail.com>
Date: Fri, 28 Mar 2014 18:26:20 -1000
Subject: [PATCH] Add optional harmony support to browser transform

This mirrors the react-tools API; it doesn't include support (yet) for transforming `<script type="text/jsx">` blocks with the ES6 transforms, mostly because I don't have a good API in mind there.

Test Plan:
Ran

    JSXTransformer.transform("var two = () => 2;", {harmony: true}).code

in Chrome's dev console and got back some ES5 code.
---
 vendor/browser-transforms.js | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/vendor/browser-transforms.js b/vendor/browser-transforms.js
index 26c48a7f9b..00401537d1 100644
--- a/vendor/browser-transforms.js
+++ b/vendor/browser-transforms.js
@@ -22,7 +22,7 @@ var headEl;
 
 var buffer = require('buffer');
 var transform = require('jstransform').transform;
-var visitors = require('./fbtransform/visitors').transformVisitors;
+var visitors = require('./fbtransform/visitors');
 var docblock = require('jstransform/src/docblock');
 
 // The source-map library relies on Object.defineProperty, but IE8 doesn't
@@ -31,16 +31,23 @@ var docblock = require('jstransform/src/docblock');
 // the source map in that case.
 var supportsAccessors = Object.prototype.hasOwnProperty('__defineGetter__');
 
-function transformReact(source) {
-  return transform(visitors.react, source, {
+function transformReact(source, options) {
+  var visitorList;
+  if (options && options.harmony) {
+    visitorList = visitors.getAllVisitors();
+  } else {
+    visitorList = visitors.transformVisitors.react;
+  }
+
+  return transform(visitorList, source, {
     sourceMap: supportsAccessors
   });
 }
 
 exports.transform = transformReact;
 
-exports.exec = function(code) {
-  return eval(transformReact(code).code);
+exports.exec = function(code, options) {
+  return eval(transformReact(code, options).code);
 };
 
 var inlineScriptCount = 0;