Remove unused members from CFWL_WidgetMgr::Item |pOffscreen| is totally unused. |pOwner| is set to null and never updated. In turn, GetOwnerWidget() always returns null. Change-Id: I6553abc0836e70a4726f8caf2b545c87a57ad500 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/72770 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org>
diff --git a/xfa/fwl/cfwl_widgetmgr.cpp b/xfa/fwl/cfwl_widgetmgr.cpp index a9b61b4..bdb343a 100644 --- a/xfa/fwl/cfwl_widgetmgr.cpp +++ b/xfa/fwl/cfwl_widgetmgr.cpp
@@ -30,8 +30,7 @@ } CFWL_Widget* CFWL_WidgetMgr::GetOwnerWidget(const CFWL_Widget* pWidget) const { - Item* pItem = GetWidgetMgrItem(pWidget); - return pItem && pItem->pOwner ? pItem->pOwner->pWidget : nullptr; + return nullptr; } CFWL_Widget* CFWL_WidgetMgr::GetPriorSiblingWidget(CFWL_Widget* pWidget) const {
diff --git a/xfa/fwl/cfwl_widgetmgr.h b/xfa/fwl/cfwl_widgetmgr.h index 9da6ae5..a932946 100644 --- a/xfa/fwl/cfwl_widgetmgr.h +++ b/xfa/fwl/cfwl_widgetmgr.h
@@ -69,9 +69,7 @@ explicit Item(CFWL_Widget* widget); ~Item() final; - Item* pOwner = nullptr; CFWL_Widget* const pWidget; - std::unique_ptr<CXFA_Graphics> pOffscreen; int32_t iRedrawCounter = 0; };