From c5e57bd0a3e1b3b8d2d11307c6de1886656ca9fd Mon Sep 17 00:00:00 2001 From: "commit-bot@chromium.org" Date: Thu, 19 Sep 2013 22:11:38 +0000 Subject: silence the error test to be a better citizen BUG= R=caryclark@google.com, bsalomon@google.com, tfarina@chromium.org, mtklein@google.com Author: humper@google.com Review URL: https://chromiumcodereview.appspot.com/23481012 git-svn-id: http://skia.googlecode.com/svn/trunk@11409 2bbb7eff-a529-9590-31e7-b0007b416f81 --- tests/ErrorTest.cpp | 26 ++++++++++++++------------ 1 file changed, 14 insertions(+), 12 deletions(-) (limited to 'tests/ErrorTest.cpp') diff --git a/tests/ErrorTest.cpp b/tests/ErrorTest.cpp index 761f8bf66f..5b92808507 100644 --- a/tests/ErrorTest.cpp +++ b/tests/ErrorTest.cpp @@ -10,21 +10,32 @@ #include "SkPath.h" #include "SkRect.h" +typedef struct { + skiatest::Reporter *fReporter; + unsigned int *fIntPointer; +} ErrorContext; + #define CHECK(errcode) \ REPORTER_ASSERT( reporter, (err = SkGetLastError()) == errcode); \ if (err != kNoError_SkError) \ { \ - SkDebugf("Last error string: %s\n", SkGetLastErrorString()); \ SkClearLastError(); \ } static void cb(SkError err, void *context) { - int *context_ptr = static_cast(context); - SkDebugf("CB (0x%x): %s\n", *context_ptr, SkGetLastErrorString()); + ErrorContext *context_ptr = static_cast(context); + REPORTER_ASSERT( context_ptr->fReporter, (*(context_ptr->fIntPointer) == 0xdeadbeef) ); } static void ErrorTest(skiatest::Reporter* reporter) { SkError err; + + unsigned int test_value = 0xdeadbeef; + ErrorContext context; + context.fReporter = reporter; + context.fIntPointer = &test_value; + + SkSetErrorCallback(cb, &context); CHECK(kNoError_SkError); @@ -43,19 +54,10 @@ static void ErrorTest(skiatest::Reporter* reporter) { CHECK(kInvalidArgument_SkError); CHECK(kNoError_SkError); - int test_value = 0xdeadbeef; - SkSetErrorCallback(cb, &test_value); - // should trigger *our* callback. path.addRoundRect(r, -10, -10); CHECK(kInvalidArgument_SkError); CHECK(kNoError_SkError); - - // Should trigger the default one again. - SkSetErrorCallback(NULL, NULL); - path.addRoundRect(r, -10, -10); - CHECK(kInvalidArgument_SkError); - CHECK(kNoError_SkError); } #include "TestClassDef.h" -- cgit v1.2.3