Code review: pass 'true' for another visitAll()-_clearCachedData() loop

that follows the same potentially-inefficient pattern. Ian & I can't figure
out when this code would ever be needed - receiving a change event for
something that's not watched is nearly impossible, and it won't have any
cached data to clear anyway. We should probably remove this code in the
future. See #7155 (comment)
& follow-on comments for more.

Author: peterflynn

src/filesystem/FileSystem.js

@@ -720,7 +720,8 @@ define(function (require, exports, module) {
             if (!watchedRoot || !watchedRoot.filter(directory.name, directory.parentPath)) {
                 this._index.visitAll(function (entry) {
                     if (entry.fullPath.indexOf(directory.fullPath) === 0) {
-                        entry._clearCachedData();
+                        // Passing 'true' for a similar reason as in _unwatchEntry() - see #7150
+                        entry._clearCachedData(true);
                     }
                 }.bind(this));