Chromium Code Reviews| Index: xfa/fwl/core/ifwl_widget.h |
| diff --git a/xfa/fwl/core/ifwl_widget.h b/xfa/fwl/core/ifwl_widget.h |
| index e9da806291fcf03edd14c92fb3818f7a910baed4..0b864fee7de9bfd92a4e60e973d870af80f01f76 100644 |
| --- a/xfa/fwl/core/ifwl_widget.h |
| +++ b/xfa/fwl/core/ifwl_widget.h |
| @@ -18,13 +18,16 @@ |
| // FWL contains three parallel inheritance hierarchies, which reference each |
| // other via pointers as follows: |
| // |
| -// m_pIface m_pImpl |
| +// m_pAssociate |
|
Tom Sepez
2016/05/26 18:13:52
Let me know if you can think of a better term that
Lei Zhang
2016/05/26 23:06:07
Nope.
|
| +// <---------------------------------- |
| // CFWL_Widget ----------> IFWL_Widget ----------> CFWL_WidgetImp |
| -// | | | |
| +// | m_pIface | m_pImpl | |
| // A A A |
| // | | | |
| // CFWL_... IFWL_... CFWL_...Imp |
| // |
| +// TODO(tsepez): Collapse these into a single hierarchy. |
| +// |
| enum class FWL_Type { |
| Unknown = 0, |
| @@ -79,10 +82,8 @@ class IFWL_Widget { |
| void SetEventKey(uint32_t key); |
| void* GetLayoutItem() const; |
| void SetLayoutItem(void* pItem); |
| - FWL_Error SetPrivateData(void* module_id, |
| - void* pData, |
| - PD_CALLBACK_FREEDATA callback); |
| - void* GetPrivateData(void* module_id); |
| + void* GetAssociateWidget() const; |
| + void SetAssociateWidget(void* pAssociate); |
| FWL_Error Update(); |
| FWL_Error LockUpdate(); |
| FWL_Error UnlockUpdate(); |