Bug 105968

Summary:

[V8] V8DOMWrapper.cpp has many more includes than necessary

Product:

WebKit

Reporter:

Adam Barth <abarth>

Component:

New Bugs

Assignee:

Adam Barth <abarth>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

eric, haraken, japhet, ojan, tony, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none

Description Adam Barth 2013-01-02 16:14:55 PST

[V8] V8DOMWrapper.cpp has many more includes than necessary

Comment 1 Adam Barth 2013-01-02 16:15:33 PST

Created attachment 181101 [details]
Patch

Comment 2 WebKit Review Bot 2013-01-02 17:05:50 PST

Comment on attachment 181101 [details]
Patch

Clearing flags on attachment: 181101

Committed r138682: <http://trac.webkit.org/changeset/138682>

Comment 3 WebKit Review Bot 2013-01-02 17:05:55 PST

All reviewed patches have been landed.  Closing bug.

Comment 4 Eric Seidel (no email) 2013-01-02 22:42:09 PST

I wish we had a system to automate this sort of thing.  Our build times are affected by these death-by-a-thousand cuts.

Comment 5 Adam Barth 2013-01-03 13:31:29 PST

There might be an open-source tools we can run.  With ninja, testing the build isn't too bad.