Mark GlobalTimer::m_nTimerID as const.
Initialize it in the initializer list. Also initialize m_bProcessing in
the header.
Change-Id: I3316927b4ad80218c10b6786cebe2beebdc4c4d7
Reviewed-on: https://pdfium-review.googlesource.com/c/47950
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
diff --git a/fxjs/global_timer.cpp b/fxjs/global_timer.cpp
index 8027820..f4d16a5 100644
--- a/fxjs/global_timer.cpp
+++ b/fxjs/global_timer.cpp
@@ -15,16 +15,13 @@
const WideString& script,
uint32_t dwElapse,
uint32_t dwTimeOut)
- : m_nTimerID(0),
+ : m_nTimerID(pFormFillEnv->GetSysHandler()->SetTimer(dwElapse, Trigger)),
m_pEmbedApp(pObj),
- m_bProcessing(false),
m_nType(nType),
m_dwTimeOut(dwTimeOut),
m_swJScript(script),
m_pRuntime(pRuntime),
m_pFormFillEnv(pFormFillEnv) {
- CFX_SystemHandler* pHandler = m_pFormFillEnv->GetSysHandler();
- m_nTimerID = pHandler->SetTimer(dwElapse, Trigger);
if (m_nTimerID)
(*GetGlobalTimerMap())[m_nTimerID] = this;
}