[PATCH 1/4] gdi32: Implement MFDRV_SetLayout().

[Zhiyi Zhang]

Signed-off-by: Zhiyi Zhang <zzhang at codeweavers.com>
---
 dlls/gdi32/mfdrv/dc.c          | 12 ++++++
 dlls/gdi32/mfdrv/init.c        |  2 +-
 dlls/gdi32/mfdrv/metafiledrv.h |  1 +
 dlls/gdi32/tests/metafile.c    | 79 ++++++++++++++++++++++++++++++++++
 4 files changed, 93 insertions(+), 1 deletion(-)

diff --git a/dlls/gdi32/mfdrv/dc.c b/dlls/gdi32/mfdrv/dc.c
index 61417bfd462..1133102371a 100644
--- a/dlls/gdi32/mfdrv/dc.c
+++ b/dlls/gdi32/mfdrv/dc.c
@@ -85,6 +85,18 @@ INT CDECL MFDRV_OffsetClipRgn( PHYSDEV dev, INT x, INT y )
     return MFDRV_MetaParam2( dev, META_OFFSETCLIPRGN, x, y );
 }
 
+DWORD CDECL MFDRV_SetLayout( PHYSDEV dev, DWORD layout )
+{
+    PHYSDEV next = GET_NEXT_PHYSDEV( dev, pSetLayout );
+
+    if (!MFDRV_MetaParam2( dev, META_SETLAYOUT, HIWORD(layout), LOWORD(layout) ))
+        return GDI_ERROR;
+
+    next->funcs->pSetLayout( next, layout );
+    /* Always return LAYOUT_RTL according to tests */
+    return LAYOUT_RTL;
+}
+
 INT CDECL MFDRV_SetMapMode( PHYSDEV dev, INT mode )
 {
     return MFDRV_MetaParam1( dev, META_SETMAPMODE, mode );
diff --git a/dlls/gdi32/mfdrv/init.c b/dlls/gdi32/mfdrv/init.c
index 2a215b70902..fe7aae631d7 100644
--- a/dlls/gdi32/mfdrv/init.c
+++ b/dlls/gdi32/mfdrv/init.c
@@ -201,7 +201,7 @@ static const struct gdi_dc_funcs MFDRV_Funcs =
     MFDRV_SetDIBitsToDevice,         /* pSetDIBitsToDevice */
     NULL,                            /* pSetDeviceClipping */
     NULL,                            /* pSetDeviceGammaRamp */
-    NULL,                            /* pSetLayout */
+    MFDRV_SetLayout,                 /* pSetLayout */
     MFDRV_SetMapMode,                /* pSetMapMode */
     MFDRV_SetMapperFlags,            /* pSetMapperFlags */
     MFDRV_SetPixel,                  /* pSetPixel */
diff --git a/dlls/gdi32/mfdrv/metafiledrv.h b/dlls/gdi32/mfdrv/metafiledrv.h
index 7d3d1c41da5..429b95b3f40 100644
--- a/dlls/gdi32/mfdrv/metafiledrv.h
+++ b/dlls/gdi32/mfdrv/metafiledrv.h
@@ -113,6 +113,7 @@ extern COLORREF CDECL MFDRV_SetBkColor( PHYSDEV dev, COLORREF color ) DECLSPEC_H
 extern INT  CDECL MFDRV_SetBkMode( PHYSDEV dev, INT mode ) DECLSPEC_HIDDEN;
 extern COLORREF CDECL MFDRV_SetDCBrushColor( PHYSDEV dev, COLORREF color ) DECLSPEC_HIDDEN;
 extern COLORREF CDECL MFDRV_SetDCPenColor( PHYSDEV dev, COLORREF color ) DECLSPEC_HIDDEN;
+extern DWORD    CDECL MFDRV_SetLayout( PHYSDEV dev, DWORD layout ) DECLSPEC_HIDDEN;
 extern INT  CDECL MFDRV_SetMapMode( PHYSDEV dev, INT mode ) DECLSPEC_HIDDEN;
 extern DWORD CDECL MFDRV_SetMapperFlags( PHYSDEV dev, DWORD flags ) DECLSPEC_HIDDEN;
 extern COLORREF CDECL MFDRV_SetPixel( PHYSDEV dev, INT x, INT y, COLORREF color ) DECLSPEC_HIDDEN;
diff --git a/dlls/gdi32/tests/metafile.c b/dlls/gdi32/tests/metafile.c
index ab823dd443d..83a10264589 100644
--- a/dlls/gdi32/tests/metafile.c
+++ b/dlls/gdi32/tests/metafile.c
@@ -5072,6 +5072,84 @@ static void test_emf_text_extents(void)
     ReleaseDC(0, dc);
 }
 
+static const unsigned char MF_SETLAYOUT_BITS[] =
+{
+/*  Winedump output. Note that there is no META_SELECTOBJECT records after META_SETLAYOUT.
+ *
+ *  METAHEADER           00000012
+ *  type 1 header_size 0x12 version 0x300 size 0x54 object_count 0 max_record_size 0xa parameter_count 0
+ *     0001 0009 0300 002a 0000 0000 0005 0000
+ *     0000
+ *  META_SETLAYOUT       0000000a
+ *     0000 0000
+ *  META_SETLAYOUT       0000000a
+ *     0001 0000
+ *  META_SETLAYOUT       0000000a
+ *     0008 0000
+ *  META_SETLAYOUT       0000000a
+ *     0000 8000
+ *  META_SETLAYOUT       0000000a
+ *     0009 0000
+ *  META_SETLAYOUT       0000000a
+ *     0001 8000
+ *  META_EOF             00000006
+ */
+    0x01, 0x00, 0x09, 0x00, 0x00, 0x03, 0x2a, 0x00,
+    0x00, 0x00, 0x00, 0x00, 0x05, 0x00, 0x00, 0x00,
+    0x00, 0x00, 0x05, 0x00, 0x00, 0x00, 0x49, 0x01,
+    0x00, 0x00, 0x00, 0x00, 0x05, 0x00, 0x00, 0x00,
+    0x49, 0x01, 0x01, 0x00, 0x00, 0x00, 0x05, 0x00,
+    0x00, 0x00, 0x49, 0x01, 0x08, 0x00, 0x00, 0x00,
+    0x05, 0x00, 0x00, 0x00, 0x49, 0x01, 0x00, 0x00,
+    0x00, 0x80, 0x05, 0x00, 0x00, 0x00, 0x49, 0x01,
+    0x09, 0x00, 0x00, 0x00, 0x05, 0x00, 0x00, 0x00,
+    0x49, 0x01, 0x01, 0x00, 0x00, 0x80, 0x03, 0x00,
+    0x00, 0x00, 0x00, 0x00
+};
+
+static void test_mf_SetLayout(void)
+{
+    static const DWORD tests[] =
+    {
+        LAYOUT_LTR,
+        LAYOUT_RTL,
+        LAYOUT_BITMAPORIENTATIONPRESERVED,
+        NOMIRRORBITMAP,
+        LAYOUT_RTL | LAYOUT_BITMAPORIENTATIONPRESERVED,
+        LAYOUT_RTL | NOMIRRORBITMAP,
+    };
+    DWORD layout;
+    HMETAFILE mf;
+    int ret, i;
+    HDC mf_dc;
+
+    mf_dc = CreateMetaFileW(NULL);
+    ok(!!mf_dc, "CreateMetaFileW failed, error %d\n", GetLastError());
+
+    for (i = 0; i < ARRAY_SIZE(tests); ++i)
+    {
+        winetest_push_context("Test %d", i);
+        layout = SetLayout(mf_dc, tests[i]);
+        ok(layout == LAYOUT_RTL, "Expected %#x, got %#x\n", tests[i], layout);
+        layout = GetLayout(mf_dc);
+        todo_wine ok(layout == GDI_ERROR, "Expected %#x, got %#x\n", GDI_ERROR, layout);
+        winetest_pop_context();
+    }
+
+    mf = CloseMetaFile(mf_dc);
+    ok(!!mf, "CloseMetaFile failed, error %d\n", GetLastError());
+
+    ret = compare_mf_bits(mf, MF_SETLAYOUT_BITS, sizeof(MF_SETLAYOUT_BITS), "mf_SetLayout");
+    ok(!ret, "Bits mismatch\n");
+    if (ret)
+    {
+        dump_mf_bits(mf, "mf_SetLayout");
+        EnumMetaFile(0, mf, mf_enum_proc, 0);
+    }
+
+    DeleteMetaFile(mf);
+}
+
 START_TEST(metafile)
 {
     init_function_pointers();
@@ -5103,6 +5181,7 @@ START_TEST(metafile)
     test_mf_ExtTextOut_on_path();
     test_mf_clipping();
     test_mf_GetPath();
+    test_mf_SetLayout();
 
     /* For metafile conversions */
     test_mf_conversions();
-- 
2.30.2