Merge pull request facebook#1461 from spicyj/no-esc-slash

Don't escape slash; it's unnecessary
syzer · Jul 3, 2014 · e8e0812 · e8e0812
2 parents 676aac1 + 0d36227
commit e8e0812
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/utils/__tests__/escapeTextForBrowser-test.js b/src/utils/__tests__/escapeTextForBrowser-test.js
@@ -46,7 +46,6 @@ describe('escapeTextForBrowser', function() {
     expect(escaped).not.toContain('<');
     expect(escaped).not.toContain('>');
     expect(escaped).not.toContain('\'');
-    expect(escaped).not.toContain('/');
     expect(escaped).not.toContain('\"');
 
     escaped = escapeTextForBrowser('&');

diff --git a/src/utils/escapeTextForBrowser.js b/src/utils/escapeTextForBrowser.js
@@ -24,11 +24,10 @@ var ESCAPE_LOOKUP = {
   ">": "&gt;",
   "<": "&lt;",
   "\"": "&quot;",
-  "'": "&#x27;",
-  "/": "&#x2f;"
+  "'": "&#x27;"
 };
 
-var ESCAPE_REGEX = /[&><"'\/]/g;
+var ESCAPE_REGEX = /[&><"']/g;
 
 function escaper(match) {
   return ESCAPE_LOOKUP[match];