Nikolay Sivov : gdiplus/metafile: Implement EmfPlusRecordTypeSetClipRegion playback.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: bf9305f63547fa4050f7ac8a90314ce622d1be7b
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=bf9305f63547fa4050f7ac8a90...

Author: Nikolay Sivov 
Date:   Mon Oct 23 09:47:05 2017 +0300

gdiplus/metafile: Implement EmfPlusRecordTypeSetClipRegion playback.

Signed-off-by: Nikolay Sivov 
Signed-off-by: Vincent Povirk 
Signed-off-by: Alexandre Julliard

---

dlls/gdiplus/metafile.c       | 38 ++++++++++++++++++++++++++++++++------
 dlls/gdiplus/tests/metafile.c |  2 +-
 2 files changed, 33 insertions(+), 7 deletions(-)

diff --git a/dlls/gdiplus/metafile.c b/dlls/gdiplus/metafile.c
index 7289ea5..4990a5f 100644
--- a/dlls/gdiplus/metafile.c
+++ b/dlls/gdiplus/metafile.c
@@ -2134,6 +2134,18 @@ static GpStatus METAFILE_PlaybackObject(GpMetafile *metafile, UINT flags, UINT d
     return status;
 }

+static GpStatus metafile_set_clip_region(GpMetafile *metafile, GpRegion *region, CombineMode mode)
+{
+    GpMatrix world_to_device;
+
+    get_graphics_transform(metafile->playback_graphics, CoordinateSpaceDevice, CoordinateSpaceWorld, &world_to_device);
+
+    GdipTransformRegion(region, &world_to_device);
+    GdipCombineRegionRegion(metafile->clip, region, mode);
+
+    return METAFILE_PlaybackUpdateClip(metafile);
+}
+
 GpStatus WINGDIPAPI GdipPlayMetafileRecord(GDIPCONST GpMetafile *metafile,
     EmfPlusRecordType recordType, UINT flags, UINT dataSize, GDIPCONST BYTE *data)
 {
@@ -2315,7 +2327,6 @@ GpStatus WINGDIPAPI GdipPlayMetafileRecord(GDIPCONST GpMetafile *metafile,
             EmfPlusSetClipRect *record = (EmfPlusSetClipRect*)header;
             CombineMode mode = (CombineMode)((flags >> 8) & 0xf);
             GpRegion *region;
-            GpMatrix world_to_device;

if (dataSize + sizeof(EmfPlusRecordHeader) < sizeof(*record))
                 return InvalidParameter;
@@ -2324,17 +2335,32 @@ GpStatus WINGDIPAPI GdipPlayMetafileRecord(GDIPCONST GpMetafile *metafile,

if (stat == Ok)
             {
-                get_graphics_transform(real_metafile->playback_graphics,
-                    CoordinateSpaceDevice, CoordinateSpaceWorld, &world_to_device);
+                stat = metafile_set_clip_region(real_metafile, region, mode);
+                GdipDeleteRegion(region);
+            }
+
+            return stat;
+        }
+        case EmfPlusRecordTypeSetClipRegion:
+        {
+            CombineMode mode = (flags >> 8) & 0xf;
+            BYTE regionid = flags & 0xff;
+            GpRegion *region;

-                GdipTransformRegion(region, &world_to_device);
+            if (dataSize != 0)
+                return InvalidParameter;

-                GdipCombineRegionRegion(real_metafile->clip, region, mode);
+            if (regionid >= EmfPlusObjectTableSize || real_metafile->objtable[regionid].type != ObjectTypeRegion)
+                return InvalidParameter;

+            stat = GdipCloneRegion(real_metafile->objtable[regionid].u.region, &region);
+            if (stat == Ok)
+            {
+                stat = metafile_set_clip_region(real_metafile, region, mode);
                 GdipDeleteRegion(region);
             }

-            return METAFILE_PlaybackUpdateClip(real_metafile);
+            return stat;
         }
         case EmfPlusRecordTypeSetPageTransform:
         {
diff --git a/dlls/gdiplus/tests/metafile.c b/dlls/gdiplus/tests/metafile.c
index f232210..bd4d526 100644
--- a/dlls/gdiplus/tests/metafile.c
+++ b/dlls/gdiplus/tests/metafile.c
@@ -2171,7 +2171,7 @@ static const emfplus_record clipping_records[] = {
     { EmfPlusRecordTypeSetClipRect, 0x300 },
     { EmfPlusRecordTypeFillRects, 0xc000 },
     { EmfPlusRecordTypeObject, ObjectTypeRegion << 8 },
-    { EmfPlusRecordTypeSetClipRegion, 0x100, 0, 1 },
+    { EmfPlusRecordTypeSetClipRegion, 0x100 },
     { EmfPlusRecordTypeEndOfFile },
     { EMR_EOF },
     { 0 }