RESOLVED FIXED 51730
Remove Document.h include from some headers where it isn't necessary 
https://bugs.webkit.org/show_bug.cgi?id=51730
Summary Remove Document.h include from some headers where it isn't necessary
Tony Gentilcore
Reported 2010-12-29 16:14:18 PST
Remove Document.h include from some headers where it isn't necessary
Attachments
Patch (7.26 KB, patch)
2010-12-29 16:15 PST, Tony Gentilcore 		no flags
DetailsFormatted DiffDiff
Patch (7.85 KB, patch)
2010-12-29 17:43 PST, Tony Gentilcore 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Tony Gentilcore
Comment 1 2010-12-29 16:15:38 PST
Created attachment 77650 [details] Patch
WebKit Review Bot
Comment 2 2010-12-29 16:25:42 PST
Attachment 77650 [details] did not build on chromium: Build output: http://queues.webkit.org/results/7250267
Tony Gentilcore
Comment 3 2010-12-29 17:43:49 PST
Created attachment 77654 [details] Patch
WebKit Review Bot
Comment 4 2010-12-29 18:39:37 PST
Attachment 77650 [details] did not build on chromium: Build output: http://queues.webkit.org/results/7229223
WebKit Commit Bot
Comment 5 2010-12-30 15:40:46 PST
The commit-queue encountered the following flaky tests while processing attachment 77654 [details]: http/tests/xmlhttprequest/re-login-async.html bug 51763 (author: ap@webkit.org) The commit-queue is continuing to process your patch.
WebKit Commit Bot
Comment 6 2010-12-30 15:42:11 PST
Comment on attachment 77654 [details] Patch Clearing flags on attachment: 77654 Committed r74812: <http://trac.webkit.org/changeset/74812>
WebKit Commit Bot
Comment 7 2010-12-30 15:42:17 PST
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 8 2010-12-30 16:43:51 PST
http://trac.webkit.org/changeset/74812 might have broken Leopard Intel Debug (Tests)
Note You need to log in before you can comment on or make changes to this bug.