Bug 214543

Summary: Use std::clamp for clarity at more call sites
Product: WebKit Reporter: Darin Adler <darin>
Component: WebKit Misc.Assignee: Darin Adler <darin>
Status: RESOLVED WONTFIX    
Severity: Normal CC: benjamin, calvaris, cdumez, changseok, cmarcelo, dbarton, dino, eric.carlson, esprehn+autocc, ews-watchlist, fmalita, fred.wang, glenn, graouts, gyuyoung.kim, hta, jer.noble, jfernandez, kangil.han, keith_miller, kondapallykalyan, macpherson, mark.lam, menard, mifenton, mmaxfield, msaboff, pdr, philipj, rego, saam, sabouhallawa, sam, schenney, sergio, simon.fraser, svillar, tommyw, tzagallo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

Description Darin Adler 2020-07-19 22:54:21 PDT 
Use std::clamp for clarity at more call sites
Comment 1 Darin Adler 2020-07-19 23:20:15 PDT Comment hidden (obsolete)
Comment 2 Darin Adler 2020-07-20 10:36:45 PDT 
Created attachment 404725 [details]
Patch
Comment 3 Darin Adler 2020-07-21 13:04:45 PDT 
Created attachment 404854 [details]
Patch
Comment 4 Darin Adler 2020-07-21 15:35:57 PDT 
Unfortunately I have learned if high is less than low, then std::clamp(x, low, high) returns high.
Comment 5 Radar WebKit Bug Importer 2020-07-26 22:55:17 PDT 
<rdar://problem/66146541>
Comment 6 Darin Adler 2022-11-12 21:36:07 PST 
Might do this some day, but not going to keep a bug open.