RESOLVED FIXED 226654
Replace invalidSnapOffsetIndex with std::optional<unsigned> 
https://bugs.webkit.org/show_bug.cgi?id=226654
Summary Replace invalidSnapOffsetIndex with std::optional<unsigned>
Martin Robinson
Reported 2021-06-04 11:27:13 PDT
Using std::optional<unsigned> instead of UINT_MAX to indicate when there is no selected scroll offset will more easily prevent errors in the code.
Attachments
Patch (59.02 KB, patch)
2021-06-15 04:48 PDT, Martin Robinson 		no flags
DetailsFormatted DiffDiff
Patch (65.07 KB, patch)
2021-06-15 06:31 PDT, Martin Robinson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-06-11 11:28:43 PDT
<rdar://problem/79208115>
Martin Robinson
Comment 2 2021-06-15 04:48:30 PDT
Created attachment 431423 [details] Patch
Martin Robinson
Comment 3 2021-06-15 06:31:57 PDT
Created attachment 431432 [details] Patch
EWS
Comment 4 2021-06-16 00:37:57 PDT
Committed r278923 (238854@main): <https://commits.webkit.org/238854@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 431432 [details].
Note You need to log in before you can comment on or make changes to this bug.