Bug 171779

Summary:

Reduce the number of includes in WebPage.h and WebProcess.h

Product:

WebKit

Reporter:

Tim Horton <thorton>

Component:

New Bugs

Assignee:

Tim Horton <thorton>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, ap, bdakin, ggaren, mcatanzaro, sam, simon.fraser

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	none
Patch	none
Patch	none
Patch	none
Patch	sam: review+

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	Details Formatted Diff Diff
Patch (15.81 KB, patch) 2017-05-06 16:29 PDT, Tim Horton	no flags	Details Formatted Diff Diff
Patch (16.41 KB, patch) 2017-05-06 16:34 PDT, Tim Horton	no flags	Details Formatted Diff Diff
Patch (16.97 KB, patch) 2017-05-06 16:37 PDT, Tim Horton	no flags	Details Formatted Diff Diff
Patch (17.22 KB, patch) 2017-05-06 16:40 PDT, Tim Horton	no flags	Details Formatted Diff Diff
Patch (17.22 KB, patch) 2017-05-06 16:46 PDT, Tim Horton	sam: review+	Details Formatted Diff Diff
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 :|