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

Unified Diff: fpdfsdk/include/fsdk_mgr.h

Issue 594403003: Lock page in LoadFXAnnot to prevent unintended page closing (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 6 years, 3 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 | fpdfsdk/src/fpdfview.cpp » ('j') | fpdfsdk/src/fsdk_mgr.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: fpdfsdk/include/fsdk_mgr.h
diff --git a/fpdfsdk/include/fsdk_mgr.h b/fpdfsdk/include/fsdk_mgr.h
index 95d1a79352ec7f33d87bf40d868a8caddb9e3aec..7353e496800efe1853ecc5d63e6dcbc7f80b2d1a 100644
--- a/fpdfsdk/include/fsdk_mgr.h
+++ b/fpdfsdk/include/fsdk_mgr.h
@@ -460,7 +460,9 @@ private:
CPDF_OCContext * m_pOccontent;
FX_BOOL m_bChangeMask;
};
-
+#define PAGEVIEW_FLAG_VALID 0x01
+#define PAGEVIEW_FLAG_LOCK 0x02
+#define PAGEVIEW_FLAG_TAKEOVERPAGE 0x04
class CPDFSDK_PageView FX_FINAL
{
public:
@@ -528,10 +530,14 @@ private:
FX_BOOL m_bExitWidget;
FX_BOOL m_bOnWidget;
public:
- void SetValid(FX_BOOL bValid) {m_bValid = bValid;}
- FX_BOOL IsValid() {return m_bValid;}
+ void SetValid(FX_BOOL bValid) {m_Flag |= PAGEVIEW_FLAG_VALID;}
Tom Sepez 2014/09/23 23:38:54 nit: we should probably clean up the redundant pub
Bo Xu 2014/09/24 01:01:13 Done.
+ FX_BOOL IsValid() {return m_Flag & PAGEVIEW_FLAG_VALID;}
+ void TakeOverPage() {m_Flag |= PAGEVIEW_FLAG_TAKEOVERPAGE;}
+ FX_BOOL IsLocked() {return m_Flag & PAGEVIEW_FLAG_LOCK;}
+ void SetLock() {m_Flag |= PAGEVIEW_FLAG_LOCK;}
+ void ClearLock() {m_Flag &= ~PAGEVIEW_FLAG_LOCK;}
private:
- FX_BOOL m_bValid;
+ FX_BOOL m_Flag;
Tom Sepez 2014/09/23 23:38:54 How many of these objects are we going to have all
Bo Xu 2014/09/24 01:01:13 Done.
};
« no previous file with comments | « no previous file | fpdfsdk/src/fpdfview.cpp » ('j') | fpdfsdk/src/fsdk_mgr.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698