fix Bug 35257

cell/model/Workbook.js

@@ -4582,19 +4582,19 @@ Woorksheet.prototype.isApplyFilterBySheet = function(){
 		return null;
 	};
 	Woorksheet.prototype.removeSparklines = function (range) {
-		for (var i = 0; i < this.aSparklineGroups.length; ++i) {
+		for (var i = this.aSparklineGroups.length - 1; i > -1 ; --i) {
 			if (this.aSparklineGroups[i].remove(range)) {
-				History.Add(this.aSparklineGroups[i], {Type: AscCH.historyitem_Sparkline_RemoveSparkline, oldPr: null, newPr: null});
-				this.aSparklineGroups.splice(i--, 1);
+				History.Add(new AscDFH.CChangesDrawingsSparklinesRemove(this.aSparklineGroups[i]));
+                this.aSparklineGroups.splice(i, 1);
 			}
 		}
 	};
 	Woorksheet.prototype.removeSparklineGroups = function (range) {
-		for (var i = 0; i < this.aSparklineGroups.length; ++i) {
+		for (var i = this.aSparklineGroups.length - 1; i > -1 ; --i) {
 			if (-1 !== this.aSparklineGroups[i].intersectionSimple(range)) {
-				History.Add(this.aSparklineGroups[i], {Type: AscCH.historyitem_Sparkline_RemoveSparkline, oldPr: null, newPr: null});
-				this.aSparklineGroups.splice(i--, 1);
-			}
+                History.Add(new AscDFH.CChangesDrawingsSparklinesRemove(this.aSparklineGroups[i]));
+                this.aSparklineGroups.splice(i, 1);
+            }
 		}
 	};
 	Woorksheet.prototype.insertSparklineGroup = function (sparklineGroup) {

cell/model/WorkbookElems.js

@@ -4733,11 +4733,7 @@ RangeDataManager.prototype = {
 				--i;
 			}
 		}
-
-		var bRemove = 0 === this.arrSparklines.length;
-		if (bRemove) {
-			History.Add(new AscDFH.CChangesDrawingsSparklinesRemove(this));
-		}
+		var bRemove = (0 === this.arrSparklines.length);
 		return bRemove;
 	};
 	sparklineGroup.prototype.getLocationRanges = function (onlySingle) {