Re: [PATCH v3 0/2] MR9287: uxtheme: Handle NULL options in DrawThemeTextEx().

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/tests/system.c:
> - hdc = GetDC(hwnd);
> - SetRect(&rect, 0, 0, 1, 1);
> -
> - hr = DrawThemeTextEx(NULL, hdc, 0, 0, L"Wine", -1, DT_CENTER, &rect, NULL);
> - ok(hr == E_HANDLE, "Got unexpected hr %#lx.\n", hr);
> -
> - hr = DrawThemeTextEx(htheme, hdc, 0, 0, L"Wine", -1, DT_CENTER, &rect, NULL);
> - ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr);
> -
> - options.dwSize = sizeof(options);
> - options.dwFlags = DTT_VALIDBITS;
> - options.iFontPropId = TMT_BODYFONT;
> - hr = DrawThemeTextEx(htheme, hdc, 0, 0, L"Wine", -1, DT_CENTER, &rect, &options);
> - ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr);
> -
> - options.crText = CLR_INVALID;

Same here. You should specify DTT_TEXTCOLOR.