RESOLVED FIXED 171779
Reduce the number of includes in WebPage.h and WebProcess.h 
https://bugs.webkit.org/show_bug.cgi?id=171779
Summary Reduce the number of includes in WebPage.h and WebProcess.h
Tim Horton
Reported 2017-05-06 16:22:51 PDT
Reduce the number of includes in WebPage.h and WebProcess.h
Attachments
Patch (15.90 KB, patch)
2017-05-06 16:23 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (15.81 KB, patch)
2017-05-06 16:29 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (16.41 KB, patch)
2017-05-06 16:34 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (16.97 KB, patch)
2017-05-06 16:37 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (17.22 KB, patch)
2017-05-06 16:40 PDT, Tim Horton 		no flags
DetailsFormatted DiffDiff
Patch (17.22 KB, patch)
2017-05-06 16:46 PDT, Tim Horton 		sam: review+
DetailsFormatted DiffDiff
Show Obsolete (5) View All Add attachment proposed patch, testcase, etc.
Tim Horton
Comment 1 2017-05-06 16:23:10 PDT
Created attachment 309302 [details] Patch
Tim Horton
Comment 2 2017-05-06 16:27:46 PDT
There's ... a lot more low hanging fruit (it's probably more valuable to look at WebCore). I'm hacking together some scripts to help identify valuable candidates.
Tim Horton
Comment 3 2017-05-06 16:29:42 PDT
Created attachment 309303 [details] Patch
Tim Horton
Comment 4 2017-05-06 16:34:25 PDT
Created attachment 309304 [details] Patch
Tim Horton
Comment 5 2017-05-06 16:37:01 PDT
Created attachment 309306 [details] Patch
Tim Horton
Comment 6 2017-05-06 16:37:27 PDT
Clearly another important part of doing this is going to be having a Gtk build set up :|
Tim Horton
Comment 7 2017-05-06 16:40:11 PDT
Created attachment 309308 [details] Patch
Tim Horton
Comment 8 2017-05-06 16:46:32 PDT
Created attachment 309309 [details] Patch
Tim Horton
Comment 9 2017-05-06 20:25:51 PDT
https://trac.webkit.org/changeset/216338/webkit
Michael Catanzaro
Comment 10 2017-05-07 12:50:56 PDT
Nice!
Note You need to log in before you can comment on or make changes to this bug.