Remove deprecated TODOs

DEVSIX-4591

Autoported commit.
Original commit hash: [de317ca693]

Author: nikitakovaliov92

itext.tests/itext.svg.tests/itext/svg/converter/SvgConverterUnitTest.cs

@@ -308,7 +308,6 @@ public virtual void ResourceResolverInstanceTest() {
 
         [NUnit.Framework.Test]
         public virtual void ResourceResolverInstanceCustomResolverTest() {
-            //TODO DEVSIX-3814: remove in version 7.2
             DummySvgConverterProperties properties = new DummySvgConverterProperties();
             SvgConverterUnitTest.TestSvgProcessorResult testSvgProcessorResult = new SvgConverterUnitTest.TestSvgProcessorResult
                 (this);
@@ -318,14 +317,12 @@ public virtual void ResourceResolverInstanceCustomResolverTest() {
 
         [NUnit.Framework.Test]
         public virtual void ResourceResolverInstanceCustomResolverNullPropsTest() {
-            //TODO DEVSIX-3814: remove in version 7.2
             SvgConverterUnitTest.TestSvgProcessorResult testSvgProcessorResult = new SvgConverterUnitTest.TestSvgProcessorResult
                 (this);
             ResourceResolver currentResolver = SvgConverter.GetResourceResolver(testSvgProcessorResult, null);
             NUnit.Framework.Assert.IsNotNull(currentResolver);
         }
 
-        //TODO DEVSIX-3814: remove in version 7.2
         internal class TestSvgProcessorResult : ISvgProcessorResult {
             public TestSvgProcessorResult(SvgConverterUnitTest _enclosing) {
                 this._enclosing = _enclosing;

itext/itext.io/itext/io/font/otf/ContextualSubTable.cs

@@ -48,12 +48,10 @@ protected internal ContextualSubTable(OpenTypeFontTableReader openReader, int lo
         }
 
         public override ContextualSubstRule GetMatchingContextRule(GlyphLine line) {
-            // TODO DEVSIX-3814 Remove this dummy override. It's left only for binary backwards compatibility
             return base.GetMatchingContextRule(line);
         }
 
         protected internal override int CheckIfContextMatch(GlyphLine line, ContextualSubstRule rule) {
-            // TODO DEVSIX-3814 Remove this dummy override. It's left only for binary backwards compatibility
             return base.CheckIfContextMatch(line, rule);
         }
     }

itext/itext.svg/itext/svg/converter/SvgConverter.cs

@@ -1402,14 +1402,11 @@ internal static ResourceResolver GetResourceResolver(ISvgProcessorResult process
              props) {
             ResourceResolver resourceResolver = null;
             if (processorResult is SvgProcessorResult) {
-                //TODO DEVSIX-3814 add assert after 7.2 cause now be have a null pointer on deprecated constructor
                 SvgProcessorContext context = ((SvgProcessorResult)processorResult).GetContext();
                 if (context != null) {
                     resourceResolver = context.GetResourceResolver();
                 }
             }
-            //TODO DEVSIX-3814 remove the clause when the deprecated  constructor SvgProcessorResult(Map<String, ISvgNodeRenderer>,
-            // ISvgNodeRenderer, FontProvider, FontSet) is removed
             if (resourceResolver == null) {
                 resourceResolver = iText.Svg.Converter.SvgConverter.CreateResourceResolver(props);
             }
@@ -1434,8 +1431,6 @@ private static ResourceResolver CreateResourceResolver(ISvgConverterProperties p
             if (props == null) {
                 return new ResourceResolver(null);
             }
-            // TODO DEVSIX-3814 change the clause if-else to return new ResourceResolver(props.getBaseUri(),
-            //  props.getResourceRetriever()) when the ISvgConverterProperties#getResourceRetriever() is added
             if (props is SvgConverterProperties) {
                 return new ResourceResolver(props.GetBaseUri(), ((SvgConverterProperties)props).GetResourceRetriever());
             }

itext/itext.svg/itext/svg/processors/impl/SvgProcessorContext.cs

@@ -86,8 +86,6 @@ public SvgProcessorContext(ISvgConverterProperties converterProperties) {
                 fontProvider = new BasicFontProvider();
             }
             IResourceRetriever retriever = null;
-            // TODO DEVSIX-3814 change the clause if block to retriever = new ResourceResolver(props.getBaseUri(),
-            //  converterProperties.getResourceRetriever()) when the ISvgConverterProperties#getResourceRetriever() is added
             if (converterProperties is SvgConverterProperties) {
                 retriever = ((SvgConverterProperties)converterProperties).GetResourceRetriever();
             }

itext/itext.svg/itext/svg/renderers/impl/AbstractSvgNodeRenderer.cs

@@ -205,7 +205,6 @@ protected internal virtual void DeepCopyAttributesAndStyles(ISvgNodeRenderer dee
         /// </returns>
         [Obsolete]
         protected internal virtual Rectangle GetObjectBoundingBox(SvgDrawContext context) {
-            // TODO DEVSIX-3814 move to ISvgNodeRenderer in 7.2
             return null;
         }
 

itext/itext.svg/itext/svg/renderers/impl/ISvgTextNodeHelper.cs

@@ -45,6 +45,5 @@ public interface ISvgTextNodeHelper {
         /// <see cref="iText.Svg.Utils.TextRectangle"/>
         /// </returns>
         TextRectangle GetTextRectangle(SvgDrawContext context, Point basePoint);
-        // TODO DEVSIX-3814 This method should be moved to ISvgTextNodeRenderer in 7.2 and this class should be removed
     }
 }

itext/itext.svg/itext/svg/renderers/impl/PathSvgNodeRenderer.cs

@@ -146,7 +146,6 @@ protected internal override Rectangle GetObjectBoundingBox(SvgDrawContext contex
                 if (lastPoint == null) {
                     lastPoint = item.GetEndingPoint();
                 }
-                // TODO DEVSIX-3814 - remove this check after moving method getPathShapeRectangle to IPathShape
                 if (item is AbstractPathShape) {
                     Rectangle rectangle = ((AbstractPathShape)item).GetPathShapeRectangle(lastPoint);
                     commonRectangle = Rectangle.GetCommonRectangle(commonRectangle, rectangle);

itext/itext.svg/itext/svg/renderers/impl/TextSvgTSpanBranchRenderer.cs

@@ -50,7 +50,6 @@ public TextSvgTSpanBranchRenderer() {
         }
 
         protected internal override Rectangle GetObjectBoundingBox(SvgDrawContext context) {
-            // TODO DEVSIX-3814 change to getParent().getObjectBoundingBox(context) in 7.2 update
             if (GetParent() is AbstractSvgNodeRenderer) {
                 return ((AbstractSvgNodeRenderer)GetParent()).GetObjectBoundingBox(context);
             }

itext/itext.svg/itext/svg/renderers/path/impl/AbstractPathShape.cs

@@ -91,7 +91,6 @@ public virtual Point GetEndingPoint() {
         /// <param name="lastPoint">start point for this shape</param>
         /// <returns>calculated rectangle</returns>
         public virtual Rectangle GetPathShapeRectangle(Point lastPoint) {
-            // TODO DEVSIX-3814 - add method declaration into IPathShape and remove here in iText 7.2
             return new Rectangle((float)CssUtils.ConvertPxToPts(GetEndingPoint().GetX()), (float)CssUtils.ConvertPxToPts
                 (GetEndingPoint().GetY()), 0, 0);
         }

port-hash

@@ -1 +1 @@
-1a88e77a3ad86d8caa576b20ef41863b293f6403
+de317ca693a02b51737e5317a139241186f40341