Esme Povirk : gdiplus: Calculate region bounding box without generating HRGN.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 290dd95dd4acc9f96ccffad03e8f343a57529469
URL:    https://gitlab.winehq.org/wine/wine/-/commit/290dd95dd4acc9f96ccffad03e8f343...

Author: Esme Povirk <esme(a)codeweavers.com>
Date:   Mon Mar 18 20:44:31 2024 +0000

gdiplus: Calculate region bounding box without generating HRGN.

---

 dlls/gdiplus/region.c       | 34 +++++++++++++++++-----------------
 dlls/gdiplus/tests/region.c | 16 ++++++++--------
 2 files changed, 25 insertions(+), 25 deletions(-)

diff --git a/dlls/gdiplus/region.c b/dlls/gdiplus/region.c
index 3034dedae76..063ccc96d90 100644
--- a/dlls/gdiplus/region.c
+++ b/dlls/gdiplus/region.c
@@ -104,6 +104,9 @@ typedef struct packed_point
     short Y;
 } packed_point;
 
+static void get_region_bounding_box(struct region_element *element,
+    REAL *min_x, REAL *min_y, REAL *max_x, REAL *max_y, BOOL *empty, BOOL *infinite);
+
 static inline INT get_element_size(const region_element* element)
 {
     INT needed = sizeof(DWORD); /* DWORD for the type */
@@ -579,9 +582,8 @@ GpStatus WINGDIPAPI GdipDeleteRegion(GpRegion *region)
  */
 GpStatus WINGDIPAPI GdipGetRegionBounds(GpRegion *region, GpGraphics *graphics, GpRectF *rect)
 {
-    HRGN hrgn;
-    RECT r;
-    GpStatus status;
+    REAL min_x, min_y, max_x, max_y;
+    BOOL empty, infinite;
 
     TRACE("(%p, %p, %p)\n", region, graphics, rect);
 
@@ -589,31 +591,29 @@ GpStatus WINGDIPAPI GdipGetRegionBounds(GpRegion *region, GpGraphics *graphics,
         return InvalidParameter;
 
     /* Contrary to MSDN, native ignores the graphics transform. */
-    status = GdipGetRegionHRgn(region, NULL, &hrgn);
-    if(status != Ok)
-        return status;
+    get_region_bounding_box(®ion->node, &min_x, &min_y, &max_x, &max_y, &empty, &infinite);
 
     /* infinite */
-    if(!hrgn){
+    if(infinite){
         rect->X = rect->Y = -(REAL)(1 << 22);
         rect->Width = rect->Height = (REAL)(1 << 23);
         TRACE("%p => infinite\n", region);
         return Ok;
     }
 
-    if(GetRgnBox(hrgn, &r)){
-        rect->X = r.left;
-        rect->Y = r.top;
-        rect->Width  = r.right  - r.left;
-        rect->Height = r.bottom - r.top;
-        TRACE("%p => %s\n", region, debugstr_rectf(rect));
+    if(empty){
+        rect->X = rect->Y = rect->Width = rect->Height = 0.0;
+        TRACE("%p => empty\n", region);
+        return Ok;
     }
-    else
-        status = GenericError;
 
-    DeleteObject(hrgn);
+    rect->X = min_x;
+    rect->Y = min_y;
+    rect->Width  = max_x - min_x;
+    rect->Height = max_y - min_y;
+    TRACE("%p => %s\n", region, debugstr_rectf(rect));
 
-    return status;
+    return Ok;
 }
 
 /*****************************************************************************
diff --git a/dlls/gdiplus/tests/region.c b/dlls/gdiplus/tests/region.c
index 375f82f9ea7..93c6c53ebc1 100644
--- a/dlls/gdiplus/tests/region.c
+++ b/dlls/gdiplus/tests/region.c
@@ -1823,10 +1823,10 @@ static void test_getbounds(void)
     rectf.Height = rectf.Width = 0.0;
     status = GdipGetRegionBounds(region, graphics, &rectf);
     ok(status == Ok, "status %08x\n", status);
-    todo_wine ok(rectf.X == 0.125, "Expected X = 0.0, got %.2f\n", rectf.X);
-    todo_wine ok(rectf.Y == 1.125, "Expected Y = 0.0, got %.2f\n", rectf.Y);
-    todo_wine ok(rectf.Width  == 2.125, "Expected width = 0.0, got %.2f\n", rectf.Width);
-    todo_wine ok(rectf.Height == 3.125, "Expected height = 0.0, got %.2f\n", rectf.Height);
+    ok(rectf.X == 0.125, "Expected X = 0.0, got %.2f\n", rectf.X);
+    ok(rectf.Y == 1.125, "Expected Y = 0.0, got %.2f\n", rectf.Y);
+    ok(rectf.Width  == 2.125, "Expected width = 0.0, got %.2f\n", rectf.Width);
+    ok(rectf.Height == 3.125, "Expected height = 0.0, got %.2f\n", rectf.Height);
 
     /* test path */
     status = GdipCreatePath(FillModeAlternate, &path);
@@ -1841,10 +1841,10 @@ static void test_getbounds(void)
     rectf.Height = rectf.Width = 0.0;
     status = GdipGetRegionBounds(region, graphics, &rectf);
     ok(status == Ok, "status %08x\n", status);
-    todo_wine ok(rectf.X == 0.125, "Expected X = 0.0, got %.2f\n", rectf.X);
-    todo_wine ok(rectf.Y == 1.125, "Expected Y = 0.0, got %.2f\n", rectf.Y);
-    todo_wine ok(rectf.Width  == 2.125, "Expected width = 0.0, got %.2f\n", rectf.Width);
-    todo_wine ok(rectf.Height == 3.125, "Expected height = 0.0, got %.2f\n", rectf.Height);
+    ok(rectf.X == 0.125, "Expected X = 0.0, got %.2f\n", rectf.X);
+    ok(rectf.Y == 1.125, "Expected Y = 0.0, got %.2f\n", rectf.Y);
+    ok(rectf.Width  == 2.125, "Expected width = 0.0, got %.2f\n", rectf.Width);
+    ok(rectf.Height == 3.125, "Expected height = 0.0, got %.2f\n", rectf.Height);
 
     status = GdipDeleteRegion(region);
     ok(status == Ok, "status %08x\n", status);