Merge pull request #8960 from le717/remove-collection-utils

ingorichter · ingorichter · commit 94000171d228 · 2014-10-21T10:57:21.000-07:00
Remove CollectionUtils and StringUtils.htmlEscape()
diff --git a/src/brackets.js b/src/brackets.js
@@ -135,8 +135,6 @@ define(function (require, exports, module) {
     require("extensibility/ExtensionManagerDialog");
     require("editor/ImageViewer");
     
-    // Deprecated modules loaded just so extensions can still use them for now
-    require("utils/CollectionUtils");
     // Compatibility shims for filesystem API migration
     require("project/FileIndexManager");
     require("file/NativeFileSystem");
diff --git a/src/utils/CollectionUtils.js b/src/utils/CollectionUtils.js
diff --git a/src/utils/StringUtils.js b/src/utils/StringUtils.js
@@ -51,14 +51,6 @@ define(function (require, exports, module) {
         });
     }
 
-    /**
-     * @deprecated Use lodash `escape()` instead.
-     */
-    function htmlEscape(str) {
-        console.warn("StringUtils.htmlEscape is deprecated. Use _.escape instead.");
-        return _.escape(str);
-    }
-
     function regexEscape(str) {
         return str.replace(/([.?*+\^$\[\]\\(){}|\-])/g, "\\$1");
     }
@@ -221,7 +213,6 @@ define(function (require, exports, module) {
 
     // Define public API
     exports.format              = format;
-    exports.htmlEscape          = htmlEscape;
     exports.regexEscape         = regexEscape;
     exports.jQueryIdEscape      = jQueryIdEscape;
     exports.getLines            = getLines;
