Bug 92930

Summary: [chromium] Remove unused includes from compositor code
Product: WebKit Reporter: James Robinson <jamesr>
Component: New BugsAssignee: James Robinson <jamesr>
Status: RESOLVED FIXED    
Severity: Normal CC: cc-bugs, dglazkov, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 93024    
Attachments:
Description Flags
Patch
none
compile fix - ScrollbarLayerChromium.cpp still uses Scrollbar.h enne: review+

James Robinson
Reported 2012-08-01 18:31:36 PDT
[chromium] Remove unused includes from compositor code
Attachments
Patch (2.79 KB, patch)
2012-08-01 18:31 PDT, James Robinson 		no flags
DetailsFormatted DiffDiff
compile fix - ScrollbarLayerChromium.cpp still uses Scrollbar.h (2.79 KB, patch)
2012-08-02 13:30 PDT, James Robinson 		enne: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
James Robinson
Comment 1 2012-08-01 18:31:53 PDT
Created attachment 155943 [details] Patch
WebKit Review Bot
Comment 2 2012-08-01 18:51:17 PDT
Comment on attachment 155943 [details] Patch Attachment 155943 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/13424029
James Robinson
Comment 3 2012-08-02 13:30:47 PDT
Created attachment 156151 [details] compile fix - ScrollbarLayerChromium.cpp still uses Scrollbar.h
Adrienne Walker
Comment 4 2012-08-02 13:34:43 PDT
Comment on attachment 156151 [details] compile fix - ScrollbarLayerChromium.cpp still uses Scrollbar.h R=me, assuming this compiles.
James Robinson
Comment 5 2012-08-02 13:47:11 PDT
(In reply to comment #4) > (From update of attachment 156151 [details]) > R=me, assuming this compiles. I think it does, but I'll let EWS confirm :)
James Robinson
Comment 6 2012-08-02 14:00:52 PDT
Committed r124495: <http://trac.webkit.org/changeset/124495>
Note You need to log in before you can comment on or make changes to this bug.