Bug 178570 - Use auto for some variables in RenderLayerCompositor/Backing
Summary: Use auto for some variables in RenderLayerCompositor/Backing
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: Frédéric Wang (:fredw)
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-10-20 01:33 PDT by Frédéric Wang (:fredw)
Modified: 2017-11-15 12:59 PST (History)
3 users (show)

See Also:


Attachments
Patch (71.79 KB, patch)
2017-10-20 01:34 PDT, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff
Patch for landing (71.76 KB, patch)
2017-10-27 06:08 PDT, Frédéric Wang (:fredw)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Frédéric Wang (:fredw) 2017-10-20 01:33:54 PDT
Use auto for some variables in RenderLayerCompositor/Backing
Comment 1 Frédéric Wang (:fredw) 2017-10-20 01:34:56 PDT
Created attachment 324374 [details]
Patch
Comment 2 Darin Adler 2017-10-26 10:03:56 PDT
Comment on attachment 324374 [details]
Patch

I like this, but I am not always this bold. Some people like auto less than me so I try not to do any wholesale conversion. However, I do approve of doing it.
Comment 3 Frédéric Wang (:fredw) 2017-10-27 06:08:09 PDT
Created attachment 325154 [details]
Patch for landing
Comment 4 Frédéric Wang (:fredw) 2017-10-27 07:37:36 PDT
Comment on attachment 325154 [details]
Patch for landing

commit queue seems to be broken
Comment 5 Frédéric Wang (:fredw) 2017-10-27 07:38:48 PDT
Committed r224110: <https://trac.webkit.org/changeset/224110>
Comment 6 Radar WebKit Bug Importer 2017-11-15 12:59:36 PST
<rdar://problem/35568559>