summaryrefslogtreecommitdiff
path: root/fpdfsdk/cpdfsdk_widget.h
diff options
context:
space:
mode:
authorTom Sepez <tsepez@chromium.org>2017-09-15 15:43:11 -0700
committerChromium commit bot <commit-bot@chromium.org>2017-09-15 22:54:34 +0000
commit7b1e53c0ef09da360304c6f4397082d09a945a7e (patch)
tree0167576535814d271950c4aea24071c4e7b6afce /fpdfsdk/cpdfsdk_widget.h
parent134ac9105586407eb3b1e06001101ff893dd4a31 (diff)
downloadpdfium-7b1e53c0ef09da360304c6f4397082d09a945a7e.tar.xz
Use unsigned types for app age, value age in widgets.chromium/3218chromium/3217
Then if they roll over, it doesn't matter, since we only check for change. And then we can pull a silly check. Then remove some no-op calls where we didn't use the result. Change-Id: I35ba470b42fb8c32a6984999e0311b21729791ca Reviewed-on: https://pdfium-review.googlesource.com/14210 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'fpdfsdk/cpdfsdk_widget.h')
-rw-r--r--fpdfsdk/cpdfsdk_widget.h8
1 files changed, 4 insertions, 4 deletions
diff --git a/fpdfsdk/cpdfsdk_widget.h b/fpdfsdk/cpdfsdk_widget.h
index 5c63400443..d33413afae 100644
--- a/fpdfsdk/cpdfsdk_widget.h
+++ b/fpdfsdk/cpdfsdk_widget.h
@@ -130,8 +130,8 @@ class CPDFSDK_Widget : public CPDFSDK_BAAnnot {
void ClearAppModified();
bool IsAppModified() const;
- int32_t GetAppearanceAge() const;
- int32_t GetValueAge() const;
+ uint32_t GetAppearanceAge() const { return m_nAppearanceAge; }
+ uint32_t GetValueAge() const { return m_nValueAge; }
bool IsWidgetAppearanceValid(CPDF_Annot::AppearanceMode mode);
void DrawAppearance(CFX_RenderDevice* pDevice,
@@ -149,8 +149,8 @@ class CPDFSDK_Widget : public CPDFSDK_BAAnnot {
private:
CFX_UnownedPtr<CPDFSDK_InterForm> const m_pInterForm;
bool m_bAppModified;
- int32_t m_nAppAge;
- int32_t m_nValueAge;
+ uint32_t m_nAppearanceAge;
+ uint32_t m_nValueAge;
#ifdef PDF_ENABLE_XFA
mutable CFX_UnownedPtr<CXFA_FFWidget> m_hMixXFAWidget;