Bug 58216 - Remove safeFloatToInt() in FloatRect.cpp and replace with working version of clampToInteger()
Summary: Remove safeFloatToInt() in FloatRect.cpp and replace with working version of ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.5
: P2 Normal
Assignee: Matthew Delaney
URL:
Keywords:
Depends on:
Blocks: 61135
  Show dependency treegraph
 
Reported: 2011-04-10 23:07 PDT by Simon Fraser (smfr)
Modified: 2011-05-23 15:34 PDT (History)
2 users (show)

See Also:


Attachments
Patch (4.19 KB, patch)
2011-05-23 15:20 PDT, Matthew Delaney
simon.fraser: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2011-04-10 23:07:27 PDT
In r83422 I attempted to replace the safeFloatToInt() in FloatRect.app with a call to clampToInteger(float), but that gives the wrong result if the input is already INT_MAX.
Comment 1 Matthew Delaney 2011-05-23 15:20:31 PDT
Created attachment 94499 [details]
Patch
Comment 2 Matthew Delaney 2011-05-23 15:34:01 PDT
Committed r87103: <http://trac.webkit.org/changeset/87103>