259029 – Remove unused 'containsScrollableArea' and add assert to 'addScrollableArea' in LocalFrameView.cpp

NEW259029

Remove unused 'containsScrollableArea' and add assert to 'addScrollableArea' in LocalFrameView.cpp

https://bugs.webkit.org/show_bug.cgi?id=259029

Summary Remove unused 'containsScrollableArea' and add assert to 'addScrollableArea' ...

Ahmad Saleem

Reported 2023-07-09 02:17:08 PDT

Hi Team, I think these are just general system hygeine to delete unused functions and avoid 'null' cases to confirm that there is 'scrollableArea'. So raising bug to do 'draft' PR. Thanks!

Attachments
Add attachment proposed patch, testcase, etc.

Ahmad Saleem

Comment 1 2023-07-09 02:18:43 PDT

Remove: https://searchfox.org/wubkat/source/Source/WebCore/page/LocalFrameView.cpp#5638 Add ASSERT: https://searchfox.org/wubkat/source/Source/WebCore/page/LocalFrameView.cpp#5616

Radar WebKit Bug Importer

Comment 2 2023-07-16 02:18:15 PDT

<rdar://problem/112346898>

Note You need to log in before you can comment on or make changes to this bug.

Status NEW

Resolution

Priority P2

Severity Normal

Classification Unclassified

Version WebKit Nightly Build

Hardware Unspecified

OS Unspecified

Product WebKit

Component Scrolling

Assignee

Nobody

Reported

2023-07-09 02:17 PDT

Modified

2023-07-16 02:18 PDT History

CC List

2 users Show

URL

Keywords InRadar

Depends on

Blocks