commit 2ada1739ea6731617aa3303ab6755c52201befc5
author Tom Sepez <tsepez@chromium.org> Fri Sep 17 23:02:01 2021 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Sep 17 23:02:01 2021 +0000
tree 2c2b716e8df7f861cfe17380e24db41e238adecf
parent 9173df979ba2e9826c5a3cbef0cbaeaffda8bb3c

Remove unreachable return in CPWL_MsgControl::SetFocus().

Its only caller (below in this file) passes `this` and is hence
the argument is non-null.

Change-Id: I11bb209bcdd0bd5a81556efbb45f9613da28c38b
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/85273
Auto-Submit: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>

fpdfsdk/pwl/cpwl_wnd.cpp

diff --git a/fpdfsdk/pwl/cpwl_wnd.cpp b/fpdfsdk/pwl/cpwl_wnd.cpp
index 705a405..7859f41 100644
--- a/fpdfsdk/pwl/cpwl_wnd.cpp
+++ b/fpdfsdk/pwl/cpwl_wnd.cpp
@@ -63,9 +63,6 @@
 
   void SetFocus(CPWL_Wnd* pWnd) {
     m_KeyboardPaths.clear();
-    if (!pWnd)
-      return;
-
     m_pMainKeyboardWnd = pWnd;
     CPWL_Wnd* pParent = pWnd;
     while (pParent) {