commit 2c226982bc225712a4cc1a8643f030d195461ea5
author Tom Sepez <tsepez@chromium.org> Thu Aug 13 19:57:16 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Aug 13 19:57:16 2020 +0000
tree fb29cb2635a330d756d3796ff958ea6fc721387c
parent f68b0e88007186d184b7a6117d7432d3f9bc0cce

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>

xfa/fwl/cfwl_widgetmgr.cpp

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 {

xfa/fwl/cfwl_widgetmgr.h

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;
   };