Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(227)

Unified Diff: fpdfsdk/cpdfsdk_formfillenvironment.cpp

Issue 2449783008: Fix one more FX_BOOL / int issue in fpdfsdk (Closed)
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: fpdfsdk/cpdfsdk_formfillenvironment.cpp
diff --git a/fpdfsdk/cpdfsdk_formfillenvironment.cpp b/fpdfsdk/cpdfsdk_formfillenvironment.cpp
index f729f717d90b84230678935bfe8078ac78699b12..1f39de9069b1851217b3db4e115b4bbe3c1dddc4 100644
--- a/fpdfsdk/cpdfsdk_formfillenvironment.cpp
+++ b/fpdfsdk/cpdfsdk_formfillenvironment.cpp
@@ -430,9 +430,8 @@ FX_BOOL CPDFSDK_FormFillEnvironment::PopupMenu(FPDF_PAGE page,
FPDF_WIDGET hWidget,
int menuFlag,
CFX_PointF pt) {
- if (!m_pInfo || !m_pInfo->FFI_PopupMenu)
- return FALSE;
- return m_pInfo->FFI_PopupMenu(m_pInfo, page, hWidget, menuFlag, pt.x, pt.y);
+ return m_pInfo && m_pInfo->FFI_PopupMenu &&
+ m_pInfo->FFI_PopupMenu(m_pInfo, page, hWidget, menuFlag, pt.x, pt.y);
}
void CPDFSDK_FormFillEnvironment::Alert(FPDF_WIDESTRING Msg,
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698