From 7833e8c9093bd0e29baaf3afe6f536a8f41a7b7e Mon Sep 17 00:00:00 2001
From: Andrew Dupont <prototype@andrewdupont.net>
Date: Sun, 8 Mar 2009 18:02:09 -0500
Subject: [PATCH] Remove sniffing when branching `escapeHTML` and
 `unescapeHTML`. [#570 state:resolved] (kangax)

---
 CHANGELOG          |  2 ++
 src/lang/string.js | 22 ++++++++++++----------
 2 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/CHANGELOG b/CHANGELOG
index 28602c9..f9915da 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,3 +1,5 @@
+* Remove sniffing when branching `escapeHTML` and `unescapeHTML`. [#570 state:resolved] (kangax)
+
 * Redefine Element#down in IE 6-7 to avoid extending all descendants when no selector is given. [#452 state:resolved] (eno, Andrew Dupont)
 
 * Reverse the definitions of Event#pointer(X|Y) and Event#pointer to prevent unnecessary computation. [#403 state:resolved] (Nick Stakenburg, Andrew Dupont)
diff --git a/src/lang/string.js b/src/lang/string.js
index beaad09..90c2ea6 100644
--- a/src/lang/string.js
+++ b/src/lang/string.js
@@ -176,7 +176,7 @@ Object.extend(String.prototype, (function() {
    *  Strips tags and converts the entity forms of special HTML characters to their normal form.
   **/
   function unescapeHTML() {
-    var div = new Element('div');
+    var div = document.createElement('div');
     div.innerHTML = this.stripTags();
     return div.childNodes[0] ? (div.childNodes.length > 1 ? 
       $A(div.childNodes).inject('', function(memo, node) { return memo+node.nodeValue }) : 
@@ -437,15 +437,6 @@ Object.extend(String.prototype, (function() {
   };
 })());
 
-if (Prototype.Browser.WebKit || Prototype.Browser.IE) Object.extend(String.prototype, {
-  escapeHTML: function() {
-    return this.replace(/&/g,'&amp;').replace(/</g,'&lt;').replace(/>/g,'&gt;');
-  },
-  unescapeHTML: function() {
-    return this.stripTags().replace(/&lt;/g,'<').replace(/&gt;/g,'>').replace(/&amp;/g,'&');
-  }
-});
-
 Object.extend(String.prototype.escapeHTML, {
   div:  document.createElement('div'),
   text: document.createTextNode('')
@@ -453,3 +444,14 @@ Object.extend(String.prototype.escapeHTML, {
 
 String.prototype.escapeHTML.div.appendChild(String.prototype.escapeHTML.text);
 
+if ('<\n'.escapeHTML() !== '&lt;\n') {
+  String.prototype.escapeHTML = function() {
+    return this.replace(/&/g,'&amp;').replace(/</g,'&lt;').replace(/>/g,'&gt;');
+  }
+}
+
+if ('&lt;\n'.unescapeHTML() !== '<\n') {
+  String.prototype.unescapeHTML = function() {
+    return this.stripTags().replace(/&lt;/g,'<').replace(/&gt;/g,'>').replace(/&amp;/g,'&');
+  }
+}
\ No newline at end of file