[release/9.0] Fix issue 12538: Infinite loop in System.Drawing.Graphics between IsVisible(Point) and IsVisible(int, int) (#12550)

* Fix issue 12538: Infinite loop in System.Drawing.Graphics between IsVisible(Point) and IsVisible(int, int)

* Add unit test

---------

Co-authored-by: Olina Zhang (Beyondsoft Corporation) <[email protected]>

Author: github-actions[bot]

src/System.Drawing.Common/src/System/Drawing/Graphics.cs

@@ -583,7 +583,7 @@ public void ExcludeClip(Region region)
 
     public void TranslateClip(int dx, int dy) => CheckStatus(PInvoke.GdipTranslateClip(NativeGraphics, dx, dy));
 
-    public bool IsVisible(int x, int y) => IsVisible(new Point(x, y));
+    public bool IsVisible(int x, int y) => IsVisible((float)x, y);
 
     public bool IsVisible(Point point) => IsVisible(point.X, point.Y);
 

src/System.Drawing.Common/tests/System/Drawing/GraphicsTests.cs

@@ -2818,6 +2818,45 @@ private static void VerifyGraphics(Graphics graphics, RectangleF expectedVisible
         Assert.Equal(expectedVisibleClipBounds, graphics.VisibleClipBounds);
     }
 
+    [Fact]
+    public void IsVisible_AllOverloads_ReturnSameResult()
+    {
+        using Bitmap bitmap = new(100, 100);
+        using Graphics graphics = Graphics.FromImage(bitmap);
+
+        // Test points
+        Point point = new(10, 10);
+        PointF pointF = new(10.5f, 10.5f);
+        int x = 10, y = 10;
+        float fx = 10.5f, fy = 10.5f;
+
+        // Test rectangles
+        Rectangle rect = new(10, 10, 50, 50);
+        RectangleF rectF = new(10.5f, 10.5f, 50.5f, 50.5f);
+        int width = 50, height = 50;
+        float fwidth = 50.5f, fheight = 50.5f;
+
+        // Verify that all overloads return the same result for points
+        bool result1 = graphics.IsVisible(x, y);
+        bool result2 = graphics.IsVisible(point);
+        bool result3 = graphics.IsVisible(fx, fy);
+        bool result4 = graphics.IsVisible(pointF);
+
+        result1.Should().Be(result2);
+        result1.Should().Be(result3);
+        result1.Should().Be(result4);
+
+        // Verify that all overloads return the same result for rectangles
+        bool result5 = graphics.IsVisible(x, y, width, height);
+        bool result6 = graphics.IsVisible(rect);
+        bool result7 = graphics.IsVisible(fx, fy, fwidth, fheight);
+        bool result8 = graphics.IsVisible(rectF);
+
+        result5.Should().Be(result6);
+        result5.Should().Be(result7);
+        result5.Should().Be(result8);
+    }
+
 #if NET8_0_OR_GREATER
     [Fact]
     public void DrawCachedBitmap_ThrowsArgumentNullException()