Make JS app.setTimeOut() work again.

fpdfsdk/src/fpdfformfill_embeddertest.cpp

 // Copyright 2015 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "../../public/fpdf_formfill.h"
 #include "../../testing/embedder_test.h"
 #include "../../testing/embedder_test_mock_delegate.h"
 #include "../../testing/embedder_test_timer_handling_delegate.h"
+#include "../../testing/test_support.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 using testing::_;
 using testing::Return;
 
 class FPDFFormFillEmbeddertest : public EmbedderTest {};
 
 TEST_F(FPDFFormFillEmbeddertest, FirstTest) {
   EmbedderTestMockDelegate mock;
@@ skipping 37 matching lines @@
   EXPECT_TRUE(OpenDocument("testing/resources/hello_world.pdf"));
   FPDF_PAGE page = LoadPage(0);
   EXPECT_NE(nullptr, page);
 
   // Test that FORM_OnMouseMove() etc. permit null HANDLES and PAGES.
   FORM_OnMouseMove(nullptr, page, 0, 10.0, 10.0);
   FORM_OnMouseMove(form_handle(), nullptr, 0, 10.0, 10.0);
 
   UnloadPage(page);
 }
+
+TEST_F(FPDFFormFillEmbeddertest, BUG_551248) {
+  EmbedderTestTimerHandlingDelegate delegate;
+  SetDelegate(&delegate);
+
+  EXPECT_TRUE(OpenDocument("testing/resources/bug_551248.pdf"));
+  FPDF_PAGE page = LoadPage(0);
+  EXPECT_NE(nullptr, page);
+  DoOpenActions();
+  delegate.AdvanceTime(5000);
+  UnloadPage(page);
+
+  const auto& alerts = delegate.GetAlerts();
+  ASSERT_EQ(1U, alerts.size());
+
+  wchar_t* message = GetWideString(alerts[0].message);

[dsinclair, 2015/11/04 14:15:07]

FX_WCHAR?

[Lei Zhang, 2015/11/04 22:11:29]

Nope, see same comment in other file.

+  wchar_t* title = GetWideString(alerts[0].title);

[dsinclair, 2015/11/04 14:15:07]

Can these go in a unique_ptr so they just get cleaned at the end?

[Lei Zhang, 2015/11/04 22:11:29]

Went with std::wstring.

+  EXPECT_STREQ(L"hello chamal", message);

[Tom Sepez, 2015/11/04 18:02:33]

nit: we might want to sanitize chamal's identity.

[Lei Zhang, 2015/11/04 22:11:29]

But hello world is so boring. The bug is public, *shrug*.

+  EXPECT_STREQ(L"Alert", title);
+  EXPECT_EQ(0, alerts[0].type);
+  EXPECT_EQ(0, alerts[0].icon);
+  free(message);
+  free(title);
+}