From 0d3622714c3ec08344a7815df6fad808ecc1018e Mon Sep 17 00:00:00 2001 From: Ben Alpert Date: Mon, 28 Apr 2014 23:22:03 -0300 Subject: [PATCH] Don't escape slash; it's unnecessary Fixes #1444. Test Plan: grunt fasttest --- src/utils/__tests__/escapeTextForBrowser-test.js | 1 - src/utils/escapeTextForBrowser.js | 5 ++--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/utils/__tests__/escapeTextForBrowser-test.js b/src/utils/__tests__/escapeTextForBrowser-test.js index 06267212e76..6fe011af768 100644 --- 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 index 4d1e23c2ace..0eceb093a77 100644 --- a/src/utils/escapeTextForBrowser.js +++ b/src/utils/escapeTextForBrowser.js @@ -24,11 +24,10 @@ var ESCAPE_LOOKUP = { ">": ">", "<": "<", "\"": """, - "'": "'", - "/": "/" + "'": "'" }; -var ESCAPE_REGEX = /[&><"'\/]/g; +var ESCAPE_REGEX = /[&><"']/g; function escaper(match) { return ESCAPE_LOOKUP[match];