[GDIPLUS_WINETEST]
[reactos.git] / rostests / winetests / gdiplus / font.c
index 98c4114..982db39 100644 (file)
@@ -2,6 +2,7 @@
  * Unit test suite for fonts
  *
  * Copyright (C) 2007 Google (Evan Stade)
+ * Copyright (C) 2012 Dmitry Timoshkov
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
 #include "gdiplus.h"
 #include "wine/test.h"
 
-#define expect(expected, got) ok(got == expected, "Expected %.8x, got %.8x\n", expected, got)
-#define expectf(expected, got) ok(fabs(expected - got) < 0.0001, "Expected %.2f, got %.2f\n", expected, got)
+#define expect(expected, got) ok(got == expected, "Expected %d, got %d\n", expected, got)
+#define expectf(expected, got) ok(fabs(expected - got) < 0.0001, "Expected %f, got %f\n", expected, got)
 
-static const WCHAR arial[] = {'A','r','i','a','l','\0'};
 static const WCHAR nonexistent[] = {'T','h','i','s','F','o','n','t','s','h','o','u','l','d','N','o','t','E','x','i','s','t','\0'};
 static const WCHAR MSSansSerif[] = {'M','S',' ','S','a','n','s',' ','S','e','r','i','f','\0'};
 static const WCHAR MicrosoftSansSerif[] = {'M','i','c','r','o','s','o','f','t',' ','S','a','n','s',' ','S','e','r','i','f','\0'};
@@ -50,12 +50,7 @@ static void test_createfont(void)
     expect (FontFamilyNotFound, stat);
     stat = GdipDeleteFont(font);
     expect (InvalidParameter, stat);
-    stat = GdipCreateFontFamilyFromName(arial, NULL, &fontfamily);
-    if(stat == FontFamilyNotFound)
-    {
-        skip("Arial not installed\n");
-        return;
-    }
+    stat = GdipCreateFontFamilyFromName(Tahoma, NULL, &fontfamily);
     expect (Ok, stat);
     stat = GdipCreateFont(fontfamily, 12, FontStyleRegular, UnitPoint, &font);
     expect (Ok, stat);
@@ -67,7 +62,7 @@ static void test_createfont(void)
     expect(Ok, stat);
     stat = GdipGetFamilyName(fontfamily2, familyname, 0);
     expect(Ok, stat);
-    ok (lstrcmpiW(arial, familyname) == 0, "Expected arial, got %s\n",
+    ok (lstrcmpiW(Tahoma, familyname) == 0, "Expected Tahoma, got %s\n",
             wine_dbgstr_w(familyname));
     stat = GdipDeleteFontFamily(fontfamily2);
     expect(Ok, stat);
@@ -96,12 +91,17 @@ static void test_logfont(void)
 {
     LOGFONTA lfa, lfa2;
     GpFont *font;
+    GpFontFamily *family;
     GpStatus stat;
     GpGraphics *graphics;
     HDC hdc = GetDC(0);
     INT style;
+    REAL rval;
+    UINT16 em_height, line_spacing;
+    Unit unit;
 
     GdipCreateFromHDC(hdc, &graphics);
+
     memset(&lfa, 0, sizeof(LOGFONTA));
     memset(&lfa2, 0xff, sizeof(LOGFONTA));
 
@@ -110,14 +110,9 @@ static void test_logfont(void)
     stat = GdipCreateFontFromLogfontA(hdc, &lfa, &font);
     expect(NotTrueTypeFont, stat);
 
-    lstrcpyA(lfa.lfFaceName, "Arial");
+    lstrcpyA(lfa.lfFaceName, "Tahoma");
 
     stat = GdipCreateFontFromLogfontA(hdc, &lfa, &font);
-    if (stat == FileNotFound)
-    {
-        skip("Arial not installed.\n");
-        return;
-    }
     expect(Ok, stat);
     stat = GdipGetLogFontA(font, graphics, &lfa2);
     expect(Ok, stat);
@@ -146,7 +141,7 @@ static void test_logfont(void)
     lfa.lfItalic = lfa.lfUnderline = lfa.lfStrikeOut = TRUE;
 
     memset(&lfa2, 0xff, sizeof(LOGFONTA));
-    lstrcpyA(lfa.lfFaceName, "Arial");
+    lstrcpyA(lfa.lfFaceName, "Tahoma");
 
     stat = GdipCreateFontFromLogfontA(hdc, &lfa, &font);
     expect(Ok, stat);
@@ -173,6 +168,60 @@ static void test_logfont(void)
     ok (style == (FontStyleItalic | FontStyleUnderline | FontStyleStrikeout),
             "Expected , got %d\n", style);
 
+    stat = GdipGetFontUnit(font, &unit);
+    expect(Ok, stat);
+    expect(UnitWorld, unit);
+
+    stat = GdipGetFontHeight(font, graphics, &rval);
+    expect(Ok, stat);
+    expectf(25.347656, rval);
+    stat = GdipGetFontSize(font, &rval);
+    expect(Ok, stat);
+    expectf(21.0, rval);
+
+    stat = GdipGetFamily(font, &family);
+    expect(Ok, stat);
+    stat = GdipGetEmHeight(family, FontStyleRegular, &em_height);
+    expect(Ok, stat);
+    expect(2048, em_height);
+    stat = GdipGetLineSpacing(family, FontStyleRegular, &line_spacing);
+    expect(Ok, stat);
+    expect(2472, line_spacing);
+    GdipDeleteFontFamily(family);
+
+    GdipDeleteFont(font);
+
+    memset(&lfa, 0, sizeof(lfa));
+    lfa.lfHeight = -25;
+    lstrcpyA(lfa.lfFaceName, "Tahoma");
+    stat = GdipCreateFontFromLogfontA(hdc, &lfa, &font);
+    expect(Ok, stat);
+    memset(&lfa2, 0xff, sizeof(lfa2));
+    stat = GdipGetLogFontA(font, graphics, &lfa2);
+    expect(Ok, stat);
+    expect(lfa.lfHeight, lfa2.lfHeight);
+
+    stat = GdipGetFontUnit(font, &unit);
+    expect(Ok, stat);
+    expect(UnitWorld, unit);
+
+    stat = GdipGetFontHeight(font, graphics, &rval);
+    expect(Ok, stat);
+    expectf(30.175781, rval);
+    stat = GdipGetFontSize(font, &rval);
+    expect(Ok, stat);
+    expectf(25.0, rval);
+
+    stat = GdipGetFamily(font, &family);
+    expect(Ok, stat);
+    stat = GdipGetEmHeight(family, FontStyleRegular, &em_height);
+    expect(Ok, stat);
+    expect(2048, em_height);
+    stat = GdipGetLineSpacing(family, FontStyleRegular, &line_spacing);
+    expect(Ok, stat);
+    expect(2472, line_spacing);
+    GdipDeleteFontFamily(family);
+
     GdipDeleteFont(font);
 
     GdipDeleteGraphics(graphics);
@@ -186,7 +235,7 @@ static void test_fontfamily (void)
     GpStatus stat;
 
     /* FontFamily cannot be NULL */
-    stat = GdipCreateFontFamilyFromName (arial , NULL, NULL);
+    stat = GdipCreateFontFamilyFromName (Tahoma , NULL, NULL);
     expect (InvalidParameter, stat);
 
     /* FontFamily must be able to actually find the family.
@@ -200,17 +249,12 @@ static void test_fontfamily (void)
     expect (FontFamilyNotFound, stat);
     if(stat == Ok) GdipDeleteFontFamily(family);
 
-    stat = GdipCreateFontFamilyFromName (arial, NULL, &family);
-    if(stat == FontFamilyNotFound)
-    {
-        skip("Arial not installed\n");
-        return;
-    }
+    stat = GdipCreateFontFamilyFromName (Tahoma, NULL, &family);
     expect (Ok, stat);
 
     stat = GdipGetFamilyName (family, itsName, LANG_NEUTRAL);
     expect (Ok, stat);
-    expect (0, lstrcmpiW(itsName, arial));
+    expect (0, lstrcmpiW(itsName, Tahoma));
 
     if (0)
     {
@@ -226,7 +270,7 @@ static void test_fontfamily (void)
     GdipDeleteFontFamily(family);
     stat = GdipGetFamilyName(clonedFontFamily, itsName, LANG_NEUTRAL);
     expect(Ok, stat);
-    expect(0, lstrcmpiW(itsName, arial));
+    expect(0, lstrcmpiW(itsName, Tahoma));
 
     GdipDeleteFontFamily(clonedFontFamily);
 }
@@ -237,28 +281,25 @@ static void test_fontfamily_properties (void)
     GpStatus stat;
     UINT16 result = 0;
 
-    stat = GdipCreateFontFamilyFromName(arial, NULL, &FontFamily);
-    if(stat == FontFamilyNotFound)
-        skip("Arial not installed\n");
-    else
-    {
-        stat = GdipGetLineSpacing(FontFamily, FontStyleRegular, &result);
-        expect(Ok, stat);
-        ok (result == 2355, "Expected 2355, got %d\n", result);
-        result = 0;
-        stat = GdipGetEmHeight(FontFamily, FontStyleRegular, &result);
-        expect(Ok, stat);
-        ok(result == 2048, "Expected 2048, got %d\n", result);
-        result = 0;
-        stat = GdipGetCellAscent(FontFamily, FontStyleRegular, &result);
-        expect(Ok, stat);
-        ok(result == 1854, "Expected 1854, got %d\n", result);
-        result = 0;
-        stat = GdipGetCellDescent(FontFamily, FontStyleRegular, &result);
-        expect(Ok, stat);
-        ok(result == 434, "Expected 434, got %d\n", result);
-        GdipDeleteFontFamily(FontFamily);
-    }
+    stat = GdipCreateFontFamilyFromName(Tahoma, NULL, &FontFamily);
+    expect(Ok, stat);
+
+    stat = GdipGetLineSpacing(FontFamily, FontStyleRegular, &result);
+    expect(Ok, stat);
+    ok (result == 2472, "Expected 2472, got %d\n", result);
+    result = 0;
+    stat = GdipGetEmHeight(FontFamily, FontStyleRegular, &result);
+    expect(Ok, stat);
+    ok(result == 2048, "Expected 2048, got %d\n", result);
+    result = 0;
+    stat = GdipGetCellAscent(FontFamily, FontStyleRegular, &result);
+    expect(Ok, stat);
+    ok(result == 2049, "Expected 2049, got %d\n", result);
+    result = 0;
+    stat = GdipGetCellDescent(FontFamily, FontStyleRegular, &result);
+    expect(Ok, stat);
+    ok(result == 423, "Expected 423, got %d\n", result);
+    GdipDeleteFontFamily(FontFamily);
 
     stat = GdipCreateFontFamilyFromName(TimesNewRoman, NULL, &FontFamily);
     if(stat == FontFamilyNotFound)
@@ -362,13 +403,9 @@ static void test_heightgivendpi(void)
     GpFont* font = NULL;
     GpFontFamily* fontfamily = NULL;
     REAL height;
+    Unit unit;
 
-    stat = GdipCreateFontFamilyFromName(arial, NULL, &fontfamily);
-    if(stat == FontFamilyNotFound)
-    {
-        skip("Arial not installed\n");
-        return;
-    }
+    stat = GdipCreateFontFamilyFromName(Tahoma, NULL, &fontfamily);
     expect(Ok, stat);
 
     stat = GdipCreateFont(fontfamily, 30, FontStyleRegular, UnitPixel, &font);
@@ -382,15 +419,20 @@ static void test_heightgivendpi(void)
 
     stat = GdipGetFontHeightGivenDPI(font, 96, &height);
     expect(Ok, stat);
-    expectf((REAL)34.497070, height);
+    expectf(36.210938, height);
     GdipDeleteFont(font);
 
     height = 12345;
     stat = GdipCreateFont(fontfamily, 30, FontStyleRegular, UnitWorld, &font);
     expect(Ok, stat);
+
+    stat = GdipGetFontUnit(font, &unit);
+    expect(Ok, stat);
+    expect(UnitWorld, unit);
+
     stat = GdipGetFontHeightGivenDPI(font, 96, &height);
     expect(Ok, stat);
-    expectf((REAL)34.497070, height);
+    expectf(36.210938, height);
     GdipDeleteFont(font);
 
     height = 12345;
@@ -398,36 +440,288 @@ static void test_heightgivendpi(void)
     expect(Ok, stat);
     stat = GdipGetFontHeightGivenDPI(font, 96, &height);
     expect(Ok, stat);
-    expectf((REAL)45.996094, height);
+    expectf(48.281250, height);
     GdipDeleteFont(font);
 
     height = 12345;
     stat = GdipCreateFont(fontfamily, 30, FontStyleRegular, UnitInch, &font);
     expect(Ok, stat);
+
+    stat = GdipGetFontUnit(font, &unit);
+    expect(Ok, stat);
+    expect(UnitInch, unit);
+
     stat = GdipGetFontHeightGivenDPI(font, 96, &height);
     expect(Ok, stat);
-    expectf((REAL)3311.718750, height);
+    expectf(3476.250000, height);
     GdipDeleteFont(font);
 
     height = 12345;
     stat = GdipCreateFont(fontfamily, 30, FontStyleRegular, UnitDocument, &font);
     expect(Ok, stat);
+
+    stat = GdipGetFontUnit(font, &unit);
+    expect(Ok, stat);
+    expect(UnitDocument, unit);
+
     stat = GdipGetFontHeightGivenDPI(font, 96, &height);
     expect(Ok, stat);
-    expectf((REAL)11.039062, height);
+    expectf(11.587500, height);
     GdipDeleteFont(font);
 
     height = 12345;
     stat = GdipCreateFont(fontfamily, 30, FontStyleRegular, UnitMillimeter, &font);
     expect(Ok, stat);
+
+    stat = GdipGetFontUnit(font, &unit);
+    expect(Ok, stat);
+    expect(UnitMillimeter, unit);
+
     stat = GdipGetFontHeightGivenDPI(font, 96, &height);
     expect(Ok, stat);
-    expectf((REAL)130.382614, height);
+    expectf(136.860245, height);
     GdipDeleteFont(font);
 
     GdipDeleteFontFamily(fontfamily);
 }
 
+static int CALLBACK font_enum_proc(const LOGFONTW *lfe, const TEXTMETRICW *ntme,
+                                   DWORD type, LPARAM lparam)
+{
+    NEWTEXTMETRICW *ntm = (NEWTEXTMETRICW *)lparam;
+
+    if (type != TRUETYPE_FONTTYPE) return 1;
+
+    *ntm = *(NEWTEXTMETRICW *)ntme;
+    return 0;
+}
+
+struct font_metrics
+{
+    UINT16 em_height, line_spacing, ascent, descent;
+    REAL font_height, font_size;
+    INT lfHeight;
+};
+
+static void gdi_get_font_metrics(LOGFONTW *lf, struct font_metrics *fm)
+{
+    HDC hdc;
+    HFONT hfont;
+    NEWTEXTMETRICW ntm;
+    OUTLINETEXTMETRICW otm;
+    int ret;
+
+    hdc = CreateCompatibleDC(0);
+
+    /* it's the only way to get extended NEWTEXTMETRIC fields */
+    ret = EnumFontFamiliesExW(hdc, lf, font_enum_proc, (LPARAM)&ntm, 0);
+    ok(!ret, "EnumFontFamiliesExW failed to find %s\n", wine_dbgstr_w(lf->lfFaceName));
+
+    hfont = CreateFontIndirectW(lf);
+    SelectObject(hdc, hfont);
+
+    otm.otmSize = sizeof(otm);
+    ret = GetOutlineTextMetricsW(hdc, otm.otmSize, &otm);
+    ok(ret, "GetOutlineTextMetrics failed\n");
+
+    DeleteDC(hdc);
+    DeleteObject(hfont);
+
+    fm->lfHeight = -otm.otmTextMetrics.tmAscent;
+    fm->line_spacing = ntm.ntmCellHeight;
+    fm->font_size = (REAL)otm.otmTextMetrics.tmAscent;
+    fm->font_height = (REAL)fm->line_spacing * fm->font_size / (REAL)ntm.ntmSizeEM;
+    fm->em_height = ntm.ntmSizeEM;
+    fm->ascent = ntm.ntmSizeEM;
+    fm->descent = ntm.ntmCellHeight - ntm.ntmSizeEM;
+}
+
+static void gdip_get_font_metrics(GpFont *font, struct font_metrics *fm)
+{
+    INT style;
+    GpFontFamily *family;
+    GpStatus stat;
+
+    stat = GdipGetFontStyle(font, &style);
+    expect(Ok, stat);
+
+    stat = GdipGetFontHeight(font, NULL, &fm->font_height);
+    expect(Ok, stat);
+    stat = GdipGetFontSize(font, &fm->font_size);
+    expect(Ok, stat);
+
+    fm->lfHeight = (INT)(fm->font_size * -1.0);
+
+    stat = GdipGetFamily(font, &family);
+    expect(Ok, stat);
+
+    stat = GdipGetEmHeight(family, style, &fm->em_height);
+    expect(Ok, stat);
+    stat = GdipGetLineSpacing(family, style, &fm->line_spacing);
+    expect(Ok, stat);
+    stat = GdipGetCellAscent(family, style, &fm->ascent);
+    expect(Ok, stat);
+    stat = GdipGetCellDescent(family, style, &fm->descent);
+    expect(Ok, stat);
+
+    GdipDeleteFontFamily(family);
+}
+
+static void cmp_font_metrics(struct font_metrics *fm1, struct font_metrics *fm2, int line)
+{
+    ok_(__FILE__, line)(fm1->lfHeight == fm2->lfHeight, "lfHeight %d != %d\n", fm1->lfHeight, fm2->lfHeight);
+    ok_(__FILE__, line)(fm1->em_height == fm2->em_height, "em_height %u != %u\n", fm1->em_height, fm2->em_height);
+    ok_(__FILE__, line)(fm1->line_spacing == fm2->line_spacing, "line_spacing %u != %u\n", fm1->line_spacing, fm2->line_spacing);
+    ok_(__FILE__, line)(abs(fm1->ascent - fm2->ascent) <= 1, "ascent %u != %u\n", fm1->ascent, fm2->ascent);
+    ok_(__FILE__, line)(abs(fm1->descent - fm2->descent) <= 1, "descent %u != %u\n", fm1->descent, fm2->descent);
+    ok(fm1->font_height > 0.0, "fm1->font_height should be positive, got %f\n", fm1->font_height);
+    ok(fm2->font_height > 0.0, "fm2->font_height should be positive, got %f\n", fm2->font_height);
+    ok_(__FILE__, line)(fm1->font_height == fm2->font_height, "font_height %f != %f\n", fm1->font_height, fm2->font_height);
+    ok(fm1->font_size > 0.0, "fm1->font_size should be positive, got %f\n", fm1->font_size);
+    ok(fm2->font_size > 0.0, "fm2->font_size should be positive, got %f\n", fm2->font_size);
+    ok_(__FILE__, line)(fm1->font_size == fm2->font_size, "font_size %f != %f\n", fm1->font_size, fm2->font_size);
+}
+
+static void test_font_metrics(void)
+{
+    LOGFONTW lf;
+    GpFont *font;
+    GpFontFamily *family;
+    GpGraphics *graphics;
+    GpStatus stat;
+    Unit unit;
+    struct font_metrics fm_gdi, fm_gdip;
+    HDC hdc;
+
+    hdc = CreateCompatibleDC(0);
+    stat = GdipCreateFromHDC(hdc, &graphics);
+    expect(Ok, stat);
+
+    memset(&lf, 0, sizeof(lf));
+
+    /* Tahoma,-13 */
+    lstrcpyW(lf.lfFaceName, Tahoma);
+    lf.lfHeight = -13;
+    stat = GdipCreateFontFromLogfontW(hdc, &lf, &font);
+    expect(Ok, stat);
+
+    stat = GdipGetFontUnit(font, &unit);
+    expect(Ok, stat);
+    expect(UnitWorld, unit);
+
+    gdip_get_font_metrics(font, &fm_gdip);
+    trace("gdiplus:\n");
+    trace("%s,%d: EmHeight %u, LineSpacing %u, CellAscent %u, CellDescent %u, FontHeight %f, FontSize %f\n",
+          wine_dbgstr_w(lf.lfFaceName), lf.lfHeight,
+          fm_gdip.em_height, fm_gdip.line_spacing, fm_gdip.ascent, fm_gdip.descent,
+          fm_gdip.font_height, fm_gdip.font_size);
+
+    gdi_get_font_metrics(&lf, &fm_gdi);
+    trace("gdi:\n");
+    trace("%s,%d: EmHeight %u, LineSpacing %u, CellAscent %u, CellDescent %u, FontHeight %f, FontSize %f\n",
+          wine_dbgstr_w(lf.lfFaceName), lf.lfHeight,
+          fm_gdi.em_height, fm_gdi.line_spacing, fm_gdi.ascent, fm_gdi.descent,
+          fm_gdi.font_height, fm_gdi.font_size);
+
+    cmp_font_metrics(&fm_gdip, &fm_gdi, __LINE__);
+
+    stat = GdipGetLogFontW(font, graphics, &lf);
+    expect(Ok, stat);
+    ok(lf.lfHeight < 0, "lf.lfHeight should be negative, got %d\n", lf.lfHeight);
+    gdi_get_font_metrics(&lf, &fm_gdi);
+    trace("gdi:\n");
+    trace("%s,%d: EmHeight %u, LineSpacing %u, CellAscent %u, CellDescent %u, FontHeight %f, FontSize %f\n",
+          wine_dbgstr_w(lf.lfFaceName), lf.lfHeight,
+          fm_gdi.em_height, fm_gdi.line_spacing, fm_gdi.ascent, fm_gdi.descent,
+          fm_gdi.font_height, fm_gdi.font_size);
+    ok((REAL)lf.lfHeight * -1.0 == fm_gdi.font_size, "expected %f, got %f\n", (REAL)lf.lfHeight * -1.0, fm_gdi.font_size);
+
+    cmp_font_metrics(&fm_gdip, &fm_gdi, __LINE__);
+
+    GdipDeleteFont(font);
+
+    /* Tahoma,13 */
+    lstrcpyW(lf.lfFaceName, Tahoma);
+    lf.lfHeight = 13;
+    stat = GdipCreateFontFromLogfontW(hdc, &lf, &font);
+    expect(Ok, stat);
+
+    stat = GdipGetFontUnit(font, &unit);
+    expect(Ok, stat);
+    expect(UnitWorld, unit);
+
+    gdip_get_font_metrics(font, &fm_gdip);
+    trace("gdiplus:\n");
+    trace("%s,%d: EmHeight %u, LineSpacing %u, CellAscent %u, CellDescent %u, FontHeight %f, FontSize %f\n",
+          wine_dbgstr_w(lf.lfFaceName), lf.lfHeight,
+          fm_gdip.em_height, fm_gdip.line_spacing, fm_gdip.ascent, fm_gdip.descent,
+          fm_gdip.font_height, fm_gdip.font_size);
+
+    gdi_get_font_metrics(&lf, &fm_gdi);
+    trace("gdi:\n");
+    trace("%s,%d: EmHeight %u, LineSpacing %u, CellAscent %u, CellDescent %u, FontHeight %f, FontSize %f\n",
+          wine_dbgstr_w(lf.lfFaceName), lf.lfHeight,
+          fm_gdi.em_height, fm_gdi.line_spacing, fm_gdi.ascent, fm_gdi.descent,
+          fm_gdi.font_height, fm_gdi.font_size);
+
+    cmp_font_metrics(&fm_gdip, &fm_gdi, __LINE__);
+
+    stat = GdipGetLogFontW(font, graphics, &lf);
+    expect(Ok, stat);
+    ok(lf.lfHeight < 0, "lf.lfHeight should be negative, got %d\n", lf.lfHeight);
+    gdi_get_font_metrics(&lf, &fm_gdi);
+    trace("gdi:\n");
+    trace("%s,%d: EmHeight %u, LineSpacing %u, CellAscent %u, CellDescent %u, FontHeight %f, FontSize %f\n",
+          wine_dbgstr_w(lf.lfFaceName), lf.lfHeight,
+          fm_gdi.em_height, fm_gdi.line_spacing, fm_gdi.ascent, fm_gdi.descent,
+          fm_gdi.font_height, fm_gdi.font_size);
+    ok((REAL)lf.lfHeight * -1.0 == fm_gdi.font_size, "expected %f, got %f\n", (REAL)lf.lfHeight * -1.0, fm_gdi.font_size);
+
+    cmp_font_metrics(&fm_gdip, &fm_gdi, __LINE__);
+
+    GdipDeleteFont(font);
+
+    stat = GdipCreateFontFamilyFromName(Tahoma, NULL, &family);
+    expect(Ok, stat);
+
+    /* Tahoma,13 */
+    stat = GdipCreateFont(family, 13.0, FontStyleRegular, UnitPixel, &font);
+    expect(Ok, stat);
+
+    gdip_get_font_metrics(font, &fm_gdip);
+    trace("gdiplus:\n");
+    trace("%s,%d: EmHeight %u, LineSpacing %u, CellAscent %u, CellDescent %u, FontHeight %f, FontSize %f\n",
+          wine_dbgstr_w(lf.lfFaceName), lf.lfHeight,
+          fm_gdip.em_height, fm_gdip.line_spacing, fm_gdip.ascent, fm_gdip.descent,
+          fm_gdip.font_height, fm_gdip.font_size);
+
+    stat = GdipGetLogFontW(font, graphics, &lf);
+    expect(Ok, stat);
+    ok(lf.lfHeight < 0, "lf.lfHeight should be negative, got %d\n", lf.lfHeight);
+    gdi_get_font_metrics(&lf, &fm_gdi);
+    trace("gdi:\n");
+    trace("%s,%d: EmHeight %u, LineSpacing %u, CellAscent %u, CellDescent %u, FontHeight %f, FontSize %f\n",
+          wine_dbgstr_w(lf.lfFaceName), lf.lfHeight,
+          fm_gdi.em_height, fm_gdi.line_spacing, fm_gdi.ascent, fm_gdi.descent,
+          fm_gdi.font_height, fm_gdi.font_size);
+    ok((REAL)lf.lfHeight * -1.0 == fm_gdi.font_size, "expected %f, got %f\n", (REAL)lf.lfHeight * -1.0, fm_gdi.font_size);
+
+    cmp_font_metrics(&fm_gdip, &fm_gdi, __LINE__);
+
+    stat = GdipGetLogFontW(font, NULL, &lf);
+    expect(InvalidParameter, stat);
+
+    GdipDeleteFont(font);
+
+    stat = GdipCreateFont(family, -13.0, FontStyleRegular, UnitPixel, &font);
+    expect(InvalidParameter, stat);
+
+    GdipDeleteFontFamily(family);
+
+    GdipDeleteGraphics(graphics);
+    DeleteDC(hdc);
+}
+
 START_TEST(font)
 {
     struct GdiplusStartupInput gdiplusStartupInput;
@@ -440,6 +734,7 @@ START_TEST(font)
 
     GdiplusStartup(&gdiplusToken, &gdiplusStartupInput, NULL);
 
+    test_font_metrics();
     test_createfont();
     test_logfont();
     test_fontfamily();