only send breakpoints on file changed, not on decoration changed

fixes #40620

src/vs/workbench/parts/debug/common/debug.ts

@@ -520,9 +520,9 @@ export interface IDebugService {
 	addBreakpoints(uri: uri, rawBreakpoints: IRawBreakpoint[]): TPromise<void>;
 
 	/**
-	 * Updates the breakpoints and notifies the debug adapter of breakpoint changes.
+	 * Updates the breakpoints.
 	 */
-	updateBreakpoints(uri: uri, data: { [id: string]: DebugProtocol.Breakpoint }): TPromise<void>;
+	updateBreakpoints(uri: uri, data: { [id: string]: DebugProtocol.Breakpoint }): void;
 
 	/**
 	 * Enables or disables all breakpoints. If breakpoint is passed only enables or disables the passed breakpoint.

src/vs/workbench/parts/debug/electron-browser/debugService.ts

@@ -587,9 +587,9 @@ export class DebugService implements debug.IDebugService {
 		return this.sendBreakpoints(uri);
 	}
 
-	public updateBreakpoints(uri: uri, data: { [id: string]: DebugProtocol.Breakpoint }): TPromise<void> {
+	public updateBreakpoints(uri: uri, data: { [id: string]: DebugProtocol.Breakpoint }): void {
 		this.model.updateBreakpoints(data);
-		return this.sendBreakpoints(uri);
+		this.breakpointsToSendOnResourceSaved.add(uri.toString());
 	}
 
 	public removeBreakpoints(id?: string): TPromise<any> {
@@ -1153,11 +1153,6 @@ export class DebugService implements debug.IDebugService {
 			if (!session.readyForBreakpoints) {
 				return TPromise.as(null);
 			}
-			if (this.textFileService.isDirty(modelUri)) {
-				// Only send breakpoints for a file once it is not dirty #8077
-				this.breakpointsToSendOnResourceSaved.add(modelUri.toString());
-				return TPromise.as(null);
-			}
 
 			const breakpointsToSend = this.model.getBreakpoints().filter(bp => this.model.areBreakpointsActivated() && bp.enabled && bp.uri.toString() === modelUri.toString());
 

src/vs/workbench/parts/debug/test/common/mockDebug.ts

@@ -43,9 +43,7 @@ export class MockDebugService implements debug.IDebugService {
 		return TPromise.as(null);
 	}
 
-	public updateBreakpoints(uri: uri, data: { [id: string]: DebugProtocol.Breakpoint }): TPromise<void> {
-		return TPromise.as(null);
-	}
+	public updateBreakpoints(uri: uri, data: { [id: string]: DebugProtocol.Breakpoint }): void { }
 
 	public enableOrDisableBreakpoints(enabled: boolean): TPromise<void> {
 		return TPromise.as(null);