summaryrefslogtreecommitdiff
path: root/xfa/fwl/core/cfwl_event.h
diff options
context:
space:
mode:
authordsinclair <dsinclair@chromium.org>2016-11-16 17:55:09 -0800
committerCommit bot <commit-bot@chromium.org>2016-11-16 17:55:09 -0800
commit038a23f297963997d631d91fefdf97598585f3ba (patch)
treeb6953d33c694e143ba006f0635f94ae6d3158b69 /xfa/fwl/core/cfwl_event.h
parent5913a6ca71c85401e3f5317758d44a9fc4a667b2 (diff)
downloadpdfium-038a23f297963997d631d91fefdf97598585f3ba.tar.xz
Remove ref counting from CFWL_Event
CFWL_Event has no Retain method, just a Release method. So, it's essentially just a delete call whenever used. This CL removes the Release method and turns the one usage into a unique_ptr. Review-Url: https://codereview.chromium.org/2509863002
Diffstat (limited to 'xfa/fwl/core/cfwl_event.h')
-rw-r--r--xfa/fwl/core/cfwl_event.h15
1 files changed, 1 insertions, 14 deletions
diff --git a/xfa/fwl/core/cfwl_event.h b/xfa/fwl/core/cfwl_event.h
index e7bc15ad58..a20abaa2a2 100644
--- a/xfa/fwl/core/cfwl_event.h
+++ b/xfa/fwl/core/cfwl_event.h
@@ -78,17 +78,12 @@ class CFWL_Event {
virtual CFWL_EventType GetClassID() const;
- uint32_t Release();
-
IFWL_Widget* m_pSrcTarget;
IFWL_Widget* m_pDstTarget;
-
- private:
- uint32_t m_dwRefCount;
};
inline CFWL_Event::CFWL_Event()
- : m_pSrcTarget(nullptr), m_pDstTarget(nullptr), m_dwRefCount(1) {}
+ : m_pSrcTarget(nullptr), m_pDstTarget(nullptr) {}
inline CFWL_Event::~CFWL_Event() {}
@@ -96,14 +91,6 @@ inline CFWL_EventType CFWL_Event::GetClassID() const {
return CFWL_EventType::None;
}
-inline uint32_t CFWL_Event::Release() {
- m_dwRefCount--;
- uint32_t dwRefCount = m_dwRefCount;
- if (!m_dwRefCount)
- delete this;
- return dwRefCount;
-}
-
#define FWL_EVENT_DEF(classname, eventType, ...) \
class classname : public CFWL_Event { \
public: \