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

Description Simon Fraser (smfr) 2014-04-17 11:35:57 PDT
Remove some includes from image-related headers, and Document.h
Comment 1 Simon Fraser (smfr) 2014-04-17 11:37:05 PDT
Created attachment 229564 [details]
Patch
Comment 2 Martin Robinson 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.
Comment 3 Simon Fraser (smfr) 2014-04-17 12:29:33 PDT
Created attachment 229567 [details]
For EWS
Comment 4 Simon Fraser (smfr) 2014-04-18 10:44:57 PDT
Created attachment 229653 [details]
Patch
Comment 5 WebKit Commit Bot 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>
Comment 6 WebKit Commit Bot 2014-04-18 11:35:55 PDT
All reviewed patches have been landed.  Closing bug.