From 0fa54b80d488571f10d8a94f6740006f0bf4957c Mon Sep 17 00:00:00 2001 From: tsepez Date: Thu, 4 Aug 2016 12:07:28 -0700 Subject: Beef up timer cancellation tests Adds more questionable invocations of ClearTimeOut(). Also, checking that nothing happened is fragile. Log at least one thing to show that the code ran. Review-Url: https://codereview.chromium.org/2218473002 --- fpdfsdk/fpdfformfill_embeddertest.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'fpdfsdk') diff --git a/fpdfsdk/fpdfformfill_embeddertest.cpp b/fpdfsdk/fpdfformfill_embeddertest.cpp index e2920e041b..27b889085c 100644 --- a/fpdfsdk/fpdfformfill_embeddertest.cpp +++ b/fpdfsdk/fpdfformfill_embeddertest.cpp @@ -130,7 +130,8 @@ TEST_F(FPDFFormFillEmbeddertest, BUG_620428) { UnloadPage(page); const auto& alerts = delegate.GetAlerts(); - EXPECT_EQ(0U, alerts.size()); + ASSERT_EQ(1U, alerts.size()); + EXPECT_STREQ(L"done", alerts[0].message.c_str()); } #endif // PDF_ENABLE_V8 -- cgit v1.2.3