Bug 131797

Summary: Remove some includes from image-related headers, and Document.h
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: New BugsAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: burg, commit-queue, dino, d-r, esprehn+autocc, fmalita, glenn, gyuyoung.kim, kangil.han, kling, kondapallykalyan, mrobinson, pdr, roger_fong, schenney, sergio, simon.fraser, thorton
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 134679    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
For EWS
none
Patch none

Simon Fraser (smfr)
Reported 2014-04-17 11:35:57 PDT
Remove some includes from image-related headers, and Document.h
Attachments
Patch (23.12 KB, patch)
2014-04-17 11:37 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
For EWS (23.11 KB, patch)
2014-04-17 12:29 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Patch (23.58 KB, patch)
2014-04-18 10:44 PDT, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2014-04-17 11:37:05 PDT
Created attachment 229564 [details] Patch
Martin Robinson
Comment 2 2014-04-17 11:44:14 PDT
Comment on attachment 229564 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=229564&action=review Looks reasonable to me. > Source/WebCore/page/Page.cpp:98 > +#include <replay/InputCursor.h> > #include "ReplayController.h" I think this technically sorts the other way, but it's not a big deal.
Simon Fraser (smfr)
Comment 3 2014-04-17 12:29:33 PDT
Created attachment 229567 [details] For EWS
Simon Fraser (smfr)
Comment 4 2014-04-18 10:44:57 PDT
Created attachment 229653 [details] Patch
WebKit Commit Bot
Comment 5 2014-04-18 11:35:48 PDT
Comment on attachment 229653 [details] Patch Clearing flags on attachment: 229653 Committed r167497: <http://trac.webkit.org/changeset/167497>
WebKit Commit Bot
Comment 6 2014-04-18 11:35:55 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.