[PATCH 3/3] gdi32: Update metafile world transform immediately in winnt mode
Andrew Eikum
aeikum at codeweavers.com
Thu Aug 24 11:26:59 CDT 2017
The Chrome browser print function implements their own handling for
EMR_MODIFYWORLDTRANSFORM which calls ModifyWorldTransform on the HDC
directly without ever calling PlayEnhMetaFileRecord. In Wine, this
transformation would get discarded when the callback function returned,
causing the page to be printed at the wrong scale.
Tests show that the transform is updated immediately during
PlayEnhMetaFileRecord. In addition, a modified transform persists
between callbacks until PlayEnhMetaFileRecord is called on a relevant
type of callback, at which point the transform is reverted before
playing back the record.
Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>
---
dlls/gdi32/enhmetafile.c | 41 +++++++-
dlls/gdi32/tests/metafile.c | 243 ++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 279 insertions(+), 5 deletions(-)
diff --git a/dlls/gdi32/enhmetafile.c b/dlls/gdi32/enhmetafile.c
index 5153e3932d..b3a3db99c1 100644
--- a/dlls/gdi32/enhmetafile.c
+++ b/dlls/gdi32/enhmetafile.c
@@ -789,6 +789,10 @@ BOOL WINAPI PlayEnhMetaFileRecord(
break;
info->state.mode = pSetMapMode->iMode;
EMF_SetMapMode(hdc, info);
+
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
+
break;
}
case EMR_SETBKMODE:
@@ -893,6 +897,10 @@ BOOL WINAPI PlayEnhMetaFileRecord(
info->state.wndOrgY = pSetWindowOrgEx->ptlOrigin.y;
TRACE("SetWindowOrgEx: %d,%d\n", info->state.wndOrgX, info->state.wndOrgY);
+
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
+
break;
}
case EMR_SETWINDOWEXTEX:
@@ -910,6 +918,10 @@ BOOL WINAPI PlayEnhMetaFileRecord(
EMF_FixIsotropic(hdc, info);
TRACE("SetWindowExtEx: %d,%d\n",info->state.wndExtX, info->state.wndExtY);
+
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
+
break;
}
case EMR_SETVIEWPORTORGEX:
@@ -922,6 +934,10 @@ BOOL WINAPI PlayEnhMetaFileRecord(
info->state.vportOrgX = pSetViewportOrgEx->ptlOrigin.x;
info->state.vportOrgY = pSetViewportOrgEx->ptlOrigin.y;
TRACE("SetViewportOrgEx: %d,%d\n", info->state.vportOrgX, info->state.vportOrgY);
+
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
+
break;
}
case EMR_SETVIEWPORTEXTEX:
@@ -938,6 +954,10 @@ BOOL WINAPI PlayEnhMetaFileRecord(
if (info->state.mode == MM_ISOTROPIC)
EMF_FixIsotropic(hdc, info);
TRACE("SetViewportExtEx: %d,%d\n", info->state.vportExtX, info->state.vportExtY);
+
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
+
break;
}
case EMR_CREATEPEN:
@@ -1271,6 +1291,10 @@ BOOL WINAPI PlayEnhMetaFileRecord(
return FALSE;
info->state.world_transform = lpXfrm->xform;
+
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
+
break;
}
@@ -1429,6 +1453,9 @@ BOOL WINAPI PlayEnhMetaFileRecord(
lpScaleViewportExtEx->xNum,lpScaleViewportExtEx->xDenom,
lpScaleViewportExtEx->yNum,lpScaleViewportExtEx->yDenom);
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
+
break;
}
@@ -1457,6 +1484,9 @@ BOOL WINAPI PlayEnhMetaFileRecord(
lpScaleWindowExtEx->xNum,lpScaleWindowExtEx->xDenom,
lpScaleWindowExtEx->yNum,lpScaleWindowExtEx->yDenom);
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
+
break;
}
@@ -1472,14 +1502,20 @@ BOOL WINAPI PlayEnhMetaFileRecord(
info->state.world_transform.eM11 = info->state.world_transform.eM22 = 1;
info->state.world_transform.eM12 = info->state.world_transform.eM21 = 0;
info->state.world_transform.eDx = info->state.world_transform.eDy = 0;
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
break;
case MWT_LEFTMULTIPLY:
CombineTransform(&info->state.world_transform, &lpModifyWorldTrans->xform,
&info->state.world_transform);
+ if (!IS_WIN9X())
+ ModifyWorldTransform(hdc, &lpModifyWorldTrans->xform, MWT_LEFTMULTIPLY);
break;
case MWT_RIGHTMULTIPLY:
CombineTransform(&info->state.world_transform, &info->state.world_transform,
&lpModifyWorldTrans->xform);
+ if (!IS_WIN9X())
+ EMF_Update_MF_Xform(hdc, info);
break;
default:
FIXME("Unknown imode %d\n", lpModifyWorldTrans->iMode);
@@ -2457,11 +2493,6 @@ BOOL WINAPI EnumEnhMetaFile(
TRACE("Calling EnumFunc with record %s, size %d\n", get_emr_name(emr->iType), emr->nSize);
ret = (*callback)(hdc, ht, emr, emh->nHandles, (LPARAM)data);
offset += emr->nSize;
-
- /* WinNT - update the transform (win9x updates when the next graphics
- output record is played). */
- if (hdc && !IS_WIN9X())
- EMF_Update_MF_Xform(hdc, info);
}
if (hdc)
diff --git a/dlls/gdi32/tests/metafile.c b/dlls/gdi32/tests/metafile.c
index dbef7fc61a..c2a0ba95ba 100644
--- a/dlls/gdi32/tests/metafile.c
+++ b/dlls/gdi32/tests/metafile.c
@@ -3999,6 +3999,248 @@ static void test_emf_PlayEnhMetaFileRecord(void)
DeleteEnhMetaFile(hemf);
}
+struct emf_WorldTransform_test_data
+{
+ const char *name;
+
+ BOOL do_modify;
+ BOOL do_playback;
+
+ XFORM expected;
+ XFORM scale;
+ XFORM stored; /* this is the "hidden" world transform used in PlayEnhMetaFileRecord */
+};
+
+static BOOL xform_eq(const XFORM *a, const XFORM *b)
+{
+ return fabs(a->eM11 - b->eM11) < 0.001f &&
+ fabs(a->eM12 - b->eM12) < 0.001f &&
+ fabs(a->eM21 - b->eM21) < 0.001f &&
+ fabs(a->eM22 - b->eM22) < 0.001f &&
+ fabs(a->eDx - b->eDx) < 0.001f &&
+ fabs(a->eDy - b->eDy) < 0.001f;
+}
+
+static INT CALLBACK enum_emf_WorldTransform(HDC hdc, HANDLETABLE *ht,
+ const ENHMETARECORD *emr, INT nobj, LPARAM param)
+{
+ XFORM xform = {0};
+ struct emf_WorldTransform_test_data *test_data = (struct emf_WorldTransform_test_data *)param;
+ BOOL ret;
+
+ switch(emr->iType)
+ {
+ case EMR_SETWORLDTRANSFORM:
+ {
+ const EMRSETWORLDTRANSFORM *lpXfrm = (const EMRSETWORLDTRANSFORM *)emr;
+
+ /* get scale factors with an identity world transform */
+ GetWorldTransform(hdc, &test_data->scale);
+
+ /* play back record */
+ ret = PlayEnhMetaFileRecord(hdc, ht, emr, nobj);
+ ok(ret == TRUE, "%s: PlayEnhMetaFileRecord failed\n", test_data->name);
+
+ test_data->stored = lpXfrm->xform;
+ CombineTransform(&test_data->expected, &test_data->stored, &test_data->scale);
+
+ /* verify it is updated immediately */
+ ret = GetWorldTransform(hdc, &xform);
+ ok(ret == TRUE, "%s: GetWorldTransform failed\n", test_data->name);
+ ok(xform_eq(&xform, &test_data->expected),
+ "%s: After SWT playback, got wrong world transform: %f, %f; %f %f; %f %f; expected: %f, %f; %f %f; %f %f\n",
+ test_data->name,
+ xform.eM11, xform.eM12,
+ xform.eM21, xform.eM22,
+ xform.eDx, xform.eDy,
+ test_data->expected.eM11, test_data->expected.eM12,
+ test_data->expected.eM21, test_data->expected.eM22,
+ test_data->expected.eDx, test_data->expected.eDy);
+
+ break;
+ }
+
+ case EMR_MODIFYWORLDTRANSFORM:
+ {
+ const EMRMODIFYWORLDTRANSFORM *lpXfrm = (const EMRMODIFYWORLDTRANSFORM *)emr;
+
+ /* transform persists across calls */
+ ret = GetWorldTransform(hdc, &xform);
+ ok(ret == TRUE, "%s: GetWorldTransform failed\n", test_data->name);
+ ok(xform_eq(&xform, &test_data->expected),
+ "%s: On MWT entry, got wrong world transform: %f, %f; %f %f; %f %f; expected: %f, %f; %f %f; %f %f\n",
+ test_data->name,
+ xform.eM11, xform.eM12,
+ xform.eM21, xform.eM22,
+ xform.eDx, xform.eDy,
+ test_data->expected.eM11, test_data->expected.eM12,
+ test_data->expected.eM21, test_data->expected.eM22,
+ test_data->expected.eDx, test_data->expected.eDy);
+
+ if(test_data->do_playback)
+ {
+ /* play back record */
+ ret = PlayEnhMetaFileRecord(hdc, ht, emr, nobj);
+ ok(ret == TRUE, "%s: PlayEnhMetaFileRecord failed\n", test_data->name);
+
+ if(lpXfrm->iMode == MWT_LEFTMULTIPLY)
+ {
+ /* left multiply does not discard direct modifications */
+ CombineTransform(&test_data->expected, &lpXfrm->xform, &test_data->expected);
+
+ /* and updates the stored matrix separately */
+ CombineTransform(&test_data->stored, &lpXfrm->xform, &test_data->stored);
+
+ }
+ else if(lpXfrm->iMode == MWT_RIGHTMULTIPLY)
+ {
+ /* but right multiply does discard */
+ CombineTransform(&test_data->stored, &test_data->stored, &lpXfrm->xform);
+
+ CombineTransform(&test_data->expected, &test_data->stored, &test_data->scale);
+ }
+
+ /* verify it is updated immediately */
+ ret = GetWorldTransform(hdc, &xform);
+ ok(ret == TRUE, "%s: GetWorldTransform failed\n", test_data->name);
+ ok(xform_eq(&xform, &test_data->expected),
+ "%s: After MWT playback, got wrong world transform: %f, %f; %f %f; %f %f; expected: %f, %f; %f %f; %f %f\n",
+ test_data->name,
+ xform.eM11, xform.eM12,
+ xform.eM21, xform.eM22,
+ xform.eDx, xform.eDy,
+ test_data->expected.eM11, test_data->expected.eM12,
+ test_data->expected.eM21, test_data->expected.eM22,
+ test_data->expected.eDx, test_data->expected.eDy);
+ }
+
+ if(test_data->do_modify)
+ {
+ /* modify directly */
+ set_rotation_xform(&xform, M_PI / 4.f, 1, -1);
+ ret = ModifyWorldTransform(hdc, &xform, MWT_LEFTMULTIPLY);
+ ok(ret == TRUE, "%s: ModifyWorldTransform failed\n", test_data->name);
+
+ /* the modified transform persists across callback calls */
+ CombineTransform(&test_data->expected, &xform, &test_data->expected);
+
+ ret = GetWorldTransform(hdc, &xform);
+ ok(ret == TRUE, "%s: GetWorldTransform failed\n", test_data->name);
+ ok(xform_eq(&xform, &test_data->expected),
+ "%s: After ModifyWT, got wrong world transform: %f, %f; %f %f; %f %f; expected: %f, %f; %f %f; %f %f\n",
+ test_data->name,
+ xform.eM11, xform.eM12,
+ xform.eM21, xform.eM22,
+ xform.eDx, xform.eDy,
+ test_data->expected.eM11, test_data->expected.eM12,
+ test_data->expected.eM21, test_data->expected.eM22,
+ test_data->expected.eDx, test_data->expected.eDy);
+ }
+
+ break;
+ }
+
+ case EMR_LINETO:
+ ret = GetWorldTransform(hdc, &xform);
+ ok(ret == TRUE, "%s: GetWorldTransform failed\n", test_data->name);
+ ok(xform_eq(&xform, &test_data->expected),
+ "%s: Before LINETO playback, got wrong world transform: %f, %f; %f %f; %f %f; expected: %f, %f; %f %f; %f %f\n",
+ test_data->name,
+ xform.eM11, xform.eM12,
+ xform.eM21, xform.eM22,
+ xform.eDx, xform.eDy,
+ test_data->expected.eM11, test_data->expected.eM12,
+ test_data->expected.eM21, test_data->expected.eM22,
+ test_data->expected.eDx, test_data->expected.eDy);
+
+ ret = PlayEnhMetaFileRecord(hdc, ht, emr, nobj);
+ ok(ret == TRUE, "%s: PlayEnhMetaFileRecord failed\n", test_data->name);
+
+ /* transform doesn't change during LINETO playback */
+ ret = GetWorldTransform(hdc, &xform);
+ ok(ret == TRUE, "%s: GetWorldTransform failed\n", test_data->name);
+ ok(xform_eq(&xform, &test_data->expected),
+ "%s: After LINETO playback, got wrong world transform: %f, %f; %f %f; %f %f; expected: %f, %f; %f %f; %f %f\n",
+ test_data->name,
+ xform.eM11, xform.eM12,
+ xform.eM21, xform.eM22,
+ xform.eDx, xform.eDy,
+ test_data->expected.eM11, test_data->expected.eM12,
+ test_data->expected.eM21, test_data->expected.eM22,
+ test_data->expected.eDx, test_data->expected.eDy);
+
+ break;
+
+ default:
+ PlayEnhMetaFileRecord(hdc, ht, emr, nobj);
+ break;
+ }
+
+ return 1;
+}
+
+static void test_emf_WorldTransform(void)
+{
+ HDC hdcMetafile, hdc;
+ HWND hwnd;
+ HENHMETAFILE hemf;
+ XFORM xform;
+ BOOL ret;
+ RECT rect = { 0, 0, 100, 100 };
+ int i;
+ struct emf_WorldTransform_test_data test_data[] = {
+ { "normal", FALSE, TRUE },
+ { "playback and modify", TRUE, TRUE },
+ { "manual modify", TRUE, FALSE }
+ };
+
+ for(i = 0; i < sizeof(test_data) / sizeof(*test_data); ++i)
+ {
+ hdcMetafile = CreateEnhMetaFileA(GetDC(0), NULL, NULL, NULL);
+ ok(hdcMetafile != 0, "CreateEnhMetaFileA error %d\n", GetLastError());
+
+ ret = SetGraphicsMode(hdcMetafile, GM_ADVANCED);
+ ok(ret == TRUE, "SetGraphicsMode failed\n");
+
+ set_rotation_xform(&xform, M_PI / 4.f, 2, 3);
+ ret = SetWorldTransform(hdcMetafile, &xform); /* EMR_SETWORLDTRANSFORM */
+ ok(ret == TRUE, "SetWorldTransform failed\n");
+
+ set_rotation_xform(&xform, M_PI / 2.f, -2, -3);
+ ret = ModifyWorldTransform(hdcMetafile, &xform, MWT_LEFTMULTIPLY); /* EMR_MODIFYWORLDTRANSFORM */
+ ok(ret == TRUE, "ModifyWorldTransform failed\n");
+
+ set_rotation_xform(&xform, M_PI / 3.f, -2, 3);
+ ret = ModifyWorldTransform(hdcMetafile, &xform, MWT_LEFTMULTIPLY); /* EMR_MODIFYWORLDTRANSFORM */
+ ok(ret == TRUE, "ModifyWorldTransform failed\n");
+
+ set_rotation_xform(&xform, M_PI, 2, -3);
+ ret = ModifyWorldTransform(hdcMetafile, &xform, MWT_RIGHTMULTIPLY); /* EMR_MODIFYWORLDTRANSFORM */
+ ok(ret == TRUE, "ModifyWorldTransform failed\n");
+
+ ret = LineTo(hdcMetafile, 1, 1);
+ ok(ret == TRUE, "LineTo failed\n");
+
+ hemf = CloseEnhMetaFile(hdcMetafile);
+ ok(hemf != 0, "CloseEnhMetaFile error %d\n", GetLastError());
+
+ hwnd = CreateWindowExA(0, "static", NULL, WS_POPUP | WS_VISIBLE,
+ 0, 0, 200, 200, 0, 0, 0, NULL);
+ ok(hwnd != 0, "CreateWindowExA error %d\n", GetLastError());
+
+ hdc = GetDC(hwnd);
+ ok(hdc != 0, "GetDC failed\n");
+
+ ret = EnumEnhMetaFile(hdc, hemf, enum_emf_WorldTransform, &test_data[i], &rect);
+ ok(ret == TRUE, "EnumEnhMetaFile failed: %u\n", GetLastError());
+
+ ReleaseDC(hwnd, hdc);
+ DestroyWindow(hwnd);
+
+ DeleteEnhMetaFile(hemf);
+ }
+}
+
START_TEST(metafile)
{
init_function_pointers();
@@ -4016,6 +4258,7 @@ START_TEST(metafile)
test_emf_PolyPolyline();
test_emf_GradientFill();
test_emf_PlayEnhMetaFileRecord();
+ test_emf_WorldTransform();
/* For win-format metafiles (mfdrv) */
test_mf_SaveDC();
--
2.14.1
More information about the wine-patches
mailing list