diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-07-30 20:27:21 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-07-30 20:27:21 +0000 |
| commit | 649e059f5fe93c0de53aaa48a74068703df46301 (patch) | |
| tree | 94db3f5d67962bda272203a8c11d79b5ffcda33f /xfa/fxfa/cxfa_ffnotify.h | |
| parent | 921d24416f310b6a899f09daa4a2628be5be16f3 (diff) | |
| download | pdfium-649e059f5fe93c0de53aaa48a74068703df46301.tar.xz | |
Return unique_ptr from CXFA_FFNotify::OnCreate{Container,Content}LayoutItem()
... and then immediately release it, but it is a step in the right
direction.
Change-Id: Ib52972c6789d8f98a576d1c69f8019541c96ac51
Reviewed-on: https://pdfium-review.googlesource.com/39152
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'xfa/fxfa/cxfa_ffnotify.h')
| -rw-r--r-- | xfa/fxfa/cxfa_ffnotify.h | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/xfa/fxfa/cxfa_ffnotify.h b/xfa/fxfa/cxfa_ffnotify.h index 6715f825ff..298a10123d 100644 --- a/xfa/fxfa/cxfa_ffnotify.h +++ b/xfa/fxfa/cxfa_ffnotify.h @@ -7,6 +7,8 @@ #ifndef XFA_FXFA_CXFA_FFNOTIFY_H_ #define XFA_FXFA_CXFA_FFNOTIFY_H_ +#include <memory> + #include "xfa/fxfa/cxfa_eventparam.h" #include "xfa/fxfa/parser/cxfa_document.h" @@ -39,8 +41,10 @@ class CXFA_FFNotify { void OnChildAdded(CXFA_Node* pSender); void OnChildRemoved(); - CXFA_ContainerLayoutItem* OnCreateContainerLayoutItem(CXFA_Node* pNode); - CXFA_ContentLayoutItem* OnCreateContentLayoutItem(CXFA_Node* pNode); + std::unique_ptr<CXFA_ContainerLayoutItem> OnCreateContainerLayoutItem( + CXFA_Node* pNode); + std::unique_ptr<CXFA_ContentLayoutItem> OnCreateContentLayoutItem( + CXFA_Node* pNode); void OnLayoutItemAdded(CXFA_LayoutProcessor* pLayout, CXFA_LayoutItem* pSender, |