Bug 149343

Summary: Eliminate some redundant header inclusions
Product: WebKit Reporter: Emanuele Aina <emanuele.aina>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ahmad.saleem792, cgarcia, fpizlo, ggaren, gyuyoung.kim, webkit-bug-importer, zan
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch darin: review+

Description Emanuele Aina 2015-09-18 04:29:17 PDT
While working on bug #146511 and bug #147258 I've noticed some uneeded header inclusions. Patch will follow.
Comment 1 Emanuele Aina 2015-09-18 04:45:35 PDT
Created attachment 261499 [details]
Patch
Comment 2 Emanuele Aina 2015-09-18 06:31:53 PDT
Created attachment 261500 [details]
Patch
Comment 3 Ahmad Saleem 2024-02-24 15:59:49 PST
It seems this r+ patch didn't landed and we still have these headers except the change of 'LayerTreeHostGtk' where the file was removed with this commit:

https://github.com/WebKit/WebKit/commit/9a0bd86b48685a6bc5a2a7729c9d3666aee2f31c

Local build with 'build-webkit --release' work with remaining patch but I don't know if it would have on internal builds. Happy to do PR and give credits to original author.
Comment 4 EWS 2024-02-25 08:57:34 PST
Committed 275294@main (36e27eaf8bd4): <https://commits.webkit.org/275294@main>

Reviewed commits have been landed. Closing PR #25070 and removing active labels.
Comment 5 Radar WebKit Bug Importer 2024-02-25 08:58:18 PST
<rdar://problem/123599112>