Deprecate SVGSVGElement.forceRedraw, suspendRedraw, unsuspendRedraw[All]

Source/core/svg/SVGSVGElement.idl

Index: Source/core/svg/SVGSVGElement.idl
diff --git a/Source/core/svg/SVGSVGElement.idl b/Source/core/svg/SVGSVGElement.idl
index 1b699145555492d7456f36076b3e3bad46d875fa..f57b4f2f763e05e290a87c1266dcef2a99eb75a3 100644
--- a/Source/core/svg/SVGSVGElement.idl
+++ b/Source/core/svg/SVGSVGElement.idl
@@ -42,10 +42,10 @@
              attribute float currentScale;
     [ImplementedAs=currentTranslateFromJavascript] readonly attribute SVGPoint currentTranslate;
 
-    [RuntimeEnabled=SVG1DOM, MeasureAs=SVG1DOM] unsigned long suspendRedraw(unsigned long maxWaitMilliseconds);
-    [RuntimeEnabled=SVG1DOM, MeasureAs=SVG1DOM] void unsuspendRedraw(unsigned long suspendHandleId);
-    [RuntimeEnabled=SVG1DOM, MeasureAs=SVG1DOM] void unsuspendRedrawAll();
-    [RuntimeEnabled=SVG1DOM, MeasureAs=SVG1DOM] void forceRedraw();
+    [RuntimeEnabled=SVG1DOM, MeasureAs=SVG1DOM, DeprecateAs=SVGSVGElementSuspendRedraw] unsigned long suspendRedraw(unsigned long maxWaitMilliseconds);

[philipj_slow, 2015/02/04 15:45:24]

Have you looked at the generated code to see if both MeasureAs and DeprecateAs
can be used together?

[Erik Dahlström (inactive), 2015/02/05 08:43:00]

I have not, but I have verified that it works through manual testing. But you're
probably right in that it's less likely to break if there's a single counter,
I've changed it to just use DeprecateAs.

+    [RuntimeEnabled=SVG1DOM, MeasureAs=SVG1DOM, DeprecateAs=SVGSVGElementUnsuspendRedraw] void unsuspendRedraw(unsigned long suspendHandleId);
+    [RuntimeEnabled=SVG1DOM, MeasureAs=SVG1DOM, DeprecateAs=SVGSVGElementUnsuspendRedrawAll] void unsuspendRedrawAll();
+    [RuntimeEnabled=SVG1DOM, MeasureAs=SVG1DOM, DeprecateAs=SVGSVGElementForceRedraw] void forceRedraw();
     void pauseAnimations();
     void unpauseAnimations();
     boolean animationsPaused();