Bug 51730

Summary: Remove Document.h include from some headers where it isn't necessary
Product: WebKit Reporter: Tony Gentilcore <tonyg>
Component: New BugsAssignee: Tony Gentilcore <tonyg>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, commit-queue, dglazkov, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 52451    
Attachments:
Description Flags
Patch
none
Patch none

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.