diff options
| author | tsepez <tsepez@chromium.org> | 2016-05-26 17:35:54 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-05-26 17:35:54 -0700 |
| commit | 54ab2343c338f55735c37ed6751040791e1df1e2 (patch) | |
| tree | 59851961d740cb6c913dbdd42d54a8d64bfbf5de /xfa/fwl/lightwidget/cfwl_widget.cpp | |
| parent | 3bd3841291b46bd7ae31cfb8dc8035e5b4ed4e40 (diff) | |
| download | pdfium-54ab2343c338f55735c37ed6751040791e1df1e2.tar.xz | |
Add opaque "layout item" to widgets for caller's use.
This avoids another use of CFX_PrivateData. Note that in the old
code, we'd be calling through a m_pImpl onto the same underlying
object as we passed as the "key" argument when setting the value,
which explains why the get calls, happening one object lower, pass
the same argument as which they are being inovked against.
Review-Url: https://codereview.chromium.org/2010923002
Diffstat (limited to 'xfa/fwl/lightwidget/cfwl_widget.cpp')
| -rw-r--r-- | xfa/fwl/lightwidget/cfwl_widget.cpp | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/xfa/fwl/lightwidget/cfwl_widget.cpp b/xfa/fwl/lightwidget/cfwl_widget.cpp index 0258c27489..239796d03a 100644 --- a/xfa/fwl/lightwidget/cfwl_widget.cpp +++ b/xfa/fwl/lightwidget/cfwl_widget.cpp @@ -124,9 +124,7 @@ FWL_Error CFWL_Widget::ModifyStylesEx(uint32_t dwStylesExAdded, } uint32_t CFWL_Widget::GetStates() { - if (!m_pIface) - return 0; - return m_pIface->GetStates(); + return m_pIface ? m_pIface->GetStates() : 0; } void CFWL_Widget::SetStates(uint32_t dwStates, FX_BOOL bSet) { @@ -134,6 +132,15 @@ void CFWL_Widget::SetStates(uint32_t dwStates, FX_BOOL bSet) { m_pIface->SetStates(dwStates, bSet); } +void* CFWL_Widget::GetLayoutItem() const { + return m_pIface ? m_pIface->GetLayoutItem() : nullptr; +} + +void CFWL_Widget::SetLayoutItem(void* pItem) { + if (m_pIface) + m_pIface->SetLayoutItem(pItem); +} + FWL_Error CFWL_Widget::SetPrivateData(void* module_id, void* pData, PD_CALLBACK_FREEDATA callback) { |