cleaning the code

Author: aiday-mar

src/vs/workbench/browser/parts/editor/editorsObserver.ts

@@ -430,24 +430,19 @@ export class EditorsObserver extends Disposable {
 			entries: coalesce(entries.map(({ editor, groupId }) => {
 
 				// Find group for entry
-				console.log('editor: ', editor);
-				console.log('groupId : ', groupId);
 				const group = this.editorGroupsContainer.getGroup(groupId);
-				console.log('group : ', group);
 				if (!group) {
 					return undefined;
 				}
 
 				// Find serializable editors of group
 				let serializableEditorsOfGroup = mapGroupToSerializableEditorsOfGroup.get(group);
-				console.log('serializableEditorsOfGroup : ', serializableEditorsOfGroup);
 				if (!serializableEditorsOfGroup) {
 					serializableEditorsOfGroup = group.getEditors(EditorsOrder.SEQUENTIAL).filter(editor => {
 						const editorSerializer = registry.getEditorSerializer(editor);
 
 						return editorSerializer?.canSerialize(editor);
 					});
-					console.log('serializableEditorsOfGroup : ', serializableEditorsOfGroup);
 					mapGroupToSerializableEditorsOfGroup.set(group, serializableEditorsOfGroup);
 				}
 

src/vs/workbench/common/editor/editorGroupModel.ts

@@ -1032,10 +1032,9 @@ export class EditorGroupModel extends Disposable implements IEditorGroupModel {
 			let canSerializeEditor = false;
 
 			const editorSerializer = registry.getEditorSerializer(editor);
-			if (editorSerializer && editorSerializer.canSerialize(editor)) {
+			if (editorSerializer) {
 				const value = editorSerializer.serialize(editor);
 
-				console.log('value : ', value);
 				// Editor can be serialized
 				if (typeof value === 'string') {
 					canSerializeEditor = true;

src/vs/workbench/contrib/multiDiffEditor/browser/multiDiffEditorInput.ts

@@ -53,7 +53,6 @@ export class MultiDiffEditorInput extends EditorInput implements ILanguageSuppor
 	}
 
 	public static fromSerialized(data: ISerializedMultiDiffEditorInput, instantiationService: IInstantiationService): MultiDiffEditorInput {
-		console.log('inside of fromSerialized');
 		return instantiationService.createInstance(
 			MultiDiffEditorInput,
 			URI.parse(data.multiDiffSourceUri),
@@ -90,7 +89,6 @@ export class MultiDiffEditorInput extends EditorInput implements ILanguageSuppor
 		@IMultiDiffSourceResolverService private readonly _multiDiffSourceResolverService: IMultiDiffSourceResolverService,
 		@ITextFileService private readonly _textFileService: ITextFileService,
 	) {
-		console.log('isTransient from the constructor : ', isTransient);
 		super();
 
 		this._register(autorun((reader) => {
@@ -365,19 +363,18 @@ interface ISerializedMultiDiffEditorInput {
 }
 
 export class MultiDiffEditorSerializer implements IEditorSerializer {
+
+	// TODO@bpasero, @aiday-mar: following should be removed
 	canSerialize(editor: EditorInput): boolean {
-		console.log('inside of can serialize');
-		console.log('editor instanceof MultiDiffEditorInput && !editor.isTransient : ', editor instanceof MultiDiffEditorInput && !editor.isTransient);
 		return editor instanceof MultiDiffEditorInput && !editor.isTransient;
 	}
 
 	serialize(editor: MultiDiffEditorInput): string | undefined {
-		console.log('inside of serialize');
-		return JSON.stringify(editor.serialize());
+		const shouldSerialize = editor instanceof MultiDiffEditorInput && !editor.isTransient;
+		return shouldSerialize ? JSON.stringify(editor.serialize()) : undefined;
 	}
 
 	deserialize(instantiationService: IInstantiationService, serializedEditor: string): EditorInput | undefined {
-		console.log('inside of deserialize');
 		try {
 			const data = parse(serializedEditor) as ISerializedMultiDiffEditorInput;
 			return MultiDiffEditorInput.fromSerialized(data, instantiationService);