Bug 191791 - Remove superfluous LayoutUnit initializations
Summary: Remove superfluous LayoutUnit initializations
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ross Kirsling
URL:
Keywords: InRadar
Depends on:
Blocks: 191538
  Show dependency tree / graph
 
Reported: 2018-11-16 16:23 PST by Ross Kirsling
Modified: 2018-11-17 19:15 PST (History)
4 users (show)

See Also:

Attachments
Patch (69.65 KB, patch)
2018-11-16 16:25 PST, Ross Kirsling 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ross Kirsling 2018-11-16 16:23:30 PST 
Remove superfluous LayoutUnit initializations
Comment 1 Ross Kirsling 2018-11-16 16:25:56 PST 
Created attachment 355159 [details]
Patch
Comment 2 Ross Kirsling 2018-11-16 20:44:08 PST 
Technically this step is unnecessary if we leave LayoutUnit(int) as implicit (which ultimately, it sounds like we should) -- we can go ahead with it if it's independently desirable, otherwise scrapping it is fine too.
Comment 3 WebKit Commit Bot 2018-11-17 19:14:39 PST 
Comment on attachment 355159 [details]
Patch

Clearing flags on attachment: 355159

Committed r238359: <https://trac.webkit.org/changeset/238359>
Comment 4 WebKit Commit Bot 2018-11-17 19:14:41 PST 
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2018-11-17 19:15:27 PST 
<rdar://problem/46152898>