cleanup code

This commit cleans up the code by removing the unnecessary `this.shell.saveAll()` which was present in `saveDirty()` Signed-off-by: Vlad Arama <[email protected]>
eclipse-theia · Aug 22, 2023 · 4a95793 · 4a95793
1 parent feaf963
commit 4a95793
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/packages/core/src/browser/common-frontend-contribution.ts b/packages/core/src/browser/common-frontend-contribution.ts
@@ -1150,8 +1150,8 @@ export class CommonFrontendContribution implements FrontendApplicationContributi
                         const captionsToSave = this.unsavedTabsCaptions();
                         const untitledCaptionsToSave = this.unsavedUntitledTabsCaptions();
                         const result = await confirmExitWithOrWithoutSaving(captionsToSave, async () => {
-                            await this.shell.saveAll();
                             await this.saveDirty(untitledCaptionsToSave);
+                            await this.shell.saveAll();
                         });
                         return result;
                     }
@@ -1188,7 +1188,6 @@ export class CommonFrontendContribution implements FrontendApplicationContributi
                 await this.saveResourceService.save(widget);
             }
         }
-        await this.shell.saveAll();
     }
     protected toggleBreadcrumbs(): void {
         const value: boolean | undefined = this.preferenceService.get('breadcrumbs.enabled');