diff options
| author | tsepez <tsepez@chromium.org> | 2016-08-04 12:07:28 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-08-04 12:07:29 -0700 |
| commit | 0fa54b80d488571f10d8a94f6740006f0bf4957c (patch) | |
| tree | 0e4e7359dedcd3821b7a827276e67502de5a3f31 /fpdfsdk/fpdfformfill_embeddertest.cpp | |
| parent | beecc34d2b4b295dcefe4a71772983fbccb50eff (diff) | |
| download | pdfium-0fa54b80d488571f10d8a94f6740006f0bf4957c.tar.xz | |
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
Diffstat (limited to 'fpdfsdk/fpdfformfill_embeddertest.cpp')
| -rw-r--r-- | fpdfsdk/fpdfformfill_embeddertest.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
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 |