commit 44b29d13bffa30733ca0e780c71ca867a3e9a033
author Ryan Harrison <rharrison@chromium.org> Fri Jun 01 15:40:13 2018 +0000
committer Chromium commit bot <commit-bot@chromium.org> Fri Jun 01 15:40:13 2018 +0000
tree 0be335b4a2e8feae65c6c6f023d9fd1db0c9ee02
parent d60d7cb936ea50a0dfab26372f3a8958f84250e7

Remove unneeded call to GetNextSiblingWidget

The value is stored in a local scope variable at the end of a loop, so
is never used. The method is const, so is sideffect free, thus this
line has no overall effect.

Issue found with Clang Static Analyzer.

Change-Id: Ib20c5c121c8464433c6ad6d8e1439b929033d6d6
Reviewed-on: https://pdfium-review.googlesource.com/33530
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>

xfa/fwl/cfwl_widgetmgr.cpp

diff --git a/xfa/fwl/cfwl_widgetmgr.cpp b/xfa/fwl/cfwl_widgetmgr.cpp
index 7bae450..f19479e 100644
--- a/xfa/fwl/cfwl_widgetmgr.cpp
+++ b/xfa/fwl/cfwl_widgetmgr.cpp
@@ -429,7 +429,6 @@
       pDelegate->OnDrawWidget(pGraphics, widgetMatrix);
 
     DrawChild(child, clipBounds, pGraphics, &widgetMatrix);
-    child = GetNextSiblingWidget(child);
   }
 }