[webkit-changes] [WebKit/WebKit] 72357c: Use dynamicDowncast instead of downcast in Injecte...

Alex Christensen noreply at github.com
Tue Feb 7 15:40:09 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 72357c53b5ff55ace758892da24cc4bd86f96c5c
      https://github.com/WebKit/WebKit/commit/72357c53b5ff55ace758892da24cc4bd86f96c5c
  Author: Alex Christensen <achristensen at apple.com>
  Date:   2023-02-07 (Tue, 07 Feb 2023)

  Changed paths:
    M Source/WebKit/WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.cpp
    M Source/WebKit/WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.h
    M Tools/TestWebKitAPI/Tests/WebKitCocoa/DidResignInputElementStrongPasswordAppearance.mm

  Log Message:
  -----------
  Use dynamicDowncast instead of downcast in InjectedBundleNodeHandle::htmlIFrameElementContentFrame
https://bugs.webkit.org/show_bug.cgi?id=251882
rdar://105143592

Reviewed by Chris Dumez.

There was some unused code.  Removed.
We were downcasting to the wrong type.  Removed and replaced with dynamicDowncast.
I added the first use of htmlIFrameElementContentFrame in automated tests to prevent this in the future.

* Source/WebKit/WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.cpp:
(WebKit::InjectedBundleNodeHandle::htmlIFrameElementContentFrame):
(WebKit::InjectedBundleNodeHandle::htmlFrameElementContentFrame): Deleted.
* Source/WebKit/WebProcess/InjectedBundle/DOM/InjectedBundleNodeHandle.h:
* Tools/TestWebKitAPI/Tests/WebKitCocoa/DidResignInputElementStrongPasswordAppearance.mm:
(-[DidResignInputElementStrongPasswordAppearance webProcessPlugIn:didCreateBrowserContextController:]):

Canonical link: https://commits.webkit.org/259985@main




More information about the webkit-changes mailing list