Bug 259029
| Summary: | Remove unused 'containsScrollableArea' and add assert to 'addScrollableArea' in LocalFrameView.cpp | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ahmad Saleem <ahmad.saleem792> |
| Component: | Scrolling | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | simon.fraser, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ahmad Saleem
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
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
<rdar://problem/112346898>