Move invalidate typings cache to UpdateGraphWorker

jramsay · jramsay · commit 456227bbeeab · 2016-08-29T16:43:34.000-07:00
diff --git a/src/server/editorServices.ts b/src/server/editorServices.ts
@@ -301,7 +301,6 @@ namespace ts.server {
             let shouldRefreshInferredProjects = false;
             for (const p of projects) {
                 if (!p.updateGraph()) {
-                    this.typingsCache.invalidateCachedTypingsForProject(p);
                     shouldRefreshInferredProjects = true;
                 }
             }
diff --git a/src/server/project.ts b/src/server/project.ts
@@ -312,6 +312,7 @@ namespace ts.server {
             // - newProgram is different from the old program and structure of the old program was not reused.
             if (!oldProgram || (this.program !== oldProgram && !oldProgram.structureIsReused)) {
                 hasChanges = true;
+                this.projectService.typingsCache.invalidateCachedTypingsForProject(this);
                 if (oldProgram) {
                     for (const f of oldProgram.getSourceFiles()) {
                         if (this.program.getSourceFileByPath(f.path)) {
@@ -414,9 +415,6 @@ namespace ts.server {
                         removed.push(id);
                     }
                 }
-                if (added.length > 0 || removed.length > 0) {
-                    this.projectService.typingsCache.invalidateCachedTypingsForProject(this);
-                }
                 this.lastReportedFileNames = currentFiles;
 
                 this.lastReportedFileNames = currentFiles;

Original file line number	Diff line number	Diff line change
`@@ -301,7 +301,6 @@ namespace ts.server {`
`301`	`301`	`let shouldRefreshInferredProjects = false;`
`302`	`302`	`for (const p of projects) {`
`303`	`303`	`if (!p.updateGraph()) {`
`304`		`- this.typingsCache.invalidateCachedTypingsForProject(p);`
`305`	`304`	`shouldRefreshInferredProjects = true;`
`306`	`305`	`}`
`307`	`306`	`}`