From 604f5430f8c988dab8d394dbaa25dde8b4741de4 Mon Sep 17 00:00:00 2001 From: Reece Dunn Date: Mon, 27 Oct 2008 09:04:48 +0000 Subject: [PATCH] uxtheme: fixed the todo blocks in the IsThemed tests when theming is inactive. --- dlls/uxtheme/system.c | 1 + dlls/uxtheme/tests/system.c | 14 ++++++-------- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/dlls/uxtheme/system.c b/dlls/uxtheme/system.c index f2c24ae..3630a6d 100644 --- a/dlls/uxtheme/system.c +++ b/dlls/uxtheme/system.c @@ -566,6 +566,7 @@ BOOL WINAPI IsAppThemed(void) BOOL WINAPI IsThemeActive(void) { TRACE("\n"); + SetLastError(ERROR_SUCCESS); return bThemeActive; } diff --git a/dlls/uxtheme/tests/system.c b/dlls/uxtheme/tests/system.c index b82472b..df20885 100644 --- a/dlls/uxtheme/tests/system.c +++ b/dlls/uxtheme/tests/system.c @@ -85,10 +85,9 @@ static void test_IsThemed(void) SetLastError(0xdeadbeef); bThemeActive = pIsThemeActive(); trace("Theming is %s\n", (bThemeActive) ? "active" : "inactive"); - todo_wine - ok( GetLastError() == ERROR_SUCCESS, - "Expected ERROR_SUCCESS, got 0x%08x\n", - GetLastError()); + ok( GetLastError() == ERROR_SUCCESS, + "Expected ERROR_SUCCESS, got 0x%08x\n", + GetLastError()); /* This test is not themed */ SetLastError(0xdeadbeef); @@ -101,10 +100,9 @@ static void test_IsThemed(void) /* Although Wine currently returns FALSE, the logic behind it is wrong. It is not a todo_wine though in the testing sense */ ok( bAppThemed == FALSE, "Expected FALSE as this test executable is not (yet) themed.\n"); - todo_wine - ok( GetLastError() == ERROR_SUCCESS, - "Expected ERROR_SUCCESS, got 0x%08x\n", - GetLastError()); + ok( GetLastError() == ERROR_SUCCESS, + "Expected ERROR_SUCCESS, got 0x%08x\n", + GetLastError()); SetLastError(0xdeadbeef); bTPDefined = pIsThemePartDefined(NULL, 0 , 0); -- 1.5.6.3