changes due to review feedback

adobe · Nov 15, 2013 · 0aba29b · 0aba29b
1 parent ef43f7a
commit 0aba29b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/src/document/DocumentCommandHandlers.js b/src/document/DocumentCommandHandlers.js
@@ -181,7 +181,7 @@ define(function (require, exports, module) {
         var result = new $.Deferred();
 
         // workaround for https://github.com/adobe/brackets/issues/6001
-        // TODO shoukld be removed once bug is closed.
+        // TODO should be removed once bug is closed.
         // if we are already displaying a file do nothing but resolve immediately.
         // this fixes timing issues in test cases.
         if (EditorManager.getCurrentlyViewedPath() === fullPath) {
@@ -206,7 +206,7 @@ define(function (require, exports, module) {
             }
             result.reject();
         }
-        function _showErrorUnlessSilentAndCleanUp(fileError, fullFilePath) {
+        function _showErrorAndCleanUp(fileError, fullFilePath) {
             if (silent) {
                 _cleanup(fullFilePath);
             } else {
@@ -234,7 +234,7 @@ define(function (require, exports, module) {
                         result.resolve();
                     } else {
                         fileError = fileError || FileSystemError.NOT_FOUND;
-                        _showErrorUnlessSilentAndCleanUp(fileError);
+                        _showErrorAndCleanUp(fileError);
                     }
                 });
 
@@ -246,7 +246,7 @@ define(function (require, exports, module) {
                         result.resolve(doc);
                     })
                     .fail(function (fileError) {
-                        _showErrorUnlessSilentAndCleanUp(fileError, fullPath);
+                        _showErrorAndCleanUp(fileError, fullPath);
                     });
             }
         }

diff --git a/test/spec/DocumentCommandHandlers-test.js b/test/spec/DocumentCommandHandlers-test.js
@@ -1091,9 +1091,6 @@ define(function (require, exports, module) {
                 var promise,
                     docChangeListener = jasmine.createSpy(),
                     activeEditorChangeListener = jasmine.createSpy();
-
-                docChangeListener.callCount = 0;
-                activeEditorChangeListener.callCount = 0;
 
                 runs(function () {
                     _$(DocumentManager).on("currentDocumentChange", docChangeListener);