Bug 100907

Summary: Coordinated Graphics: Clean up headers of Coordinated Graphics.
Product: WebKit Reporter: Dongseong Hwang <dongseong.hwang>
Component: Layout and RenderingAssignee: Dongseong Hwang <dongseong.hwang>
Status: RESOLVED FIXED    
Severity: Normal CC: hnandor, noam, ossy, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 100995    
Bug Blocks: 100341, 100797, 100819    
Attachments:
Description Flags
Patch
none
Patch none

Description Dongseong Hwang 2012-10-31 17:57:27 PDT
This patch performs:
1. Remove unused method declarations.
2. Change public methods to private if needed.
3. Put virtual, OVERRIDE and explicit keywords if needed.
Comment 1 Dongseong Hwang 2012-10-31 17:59:47 PDT
Created attachment 171748 [details]
Patch
Comment 2 Dongseong Hwang 2012-10-31 23:34:26 PDT
Created attachment 171778 [details]
Patch
Comment 3 WebKit Review Bot 2012-11-01 08:56:15 PDT
Comment on attachment 171778 [details]
Patch

Clearing flags on attachment: 171778

Committed r133182: <http://trac.webkit.org/changeset/133182>
Comment 4 WebKit Review Bot 2012-11-01 08:56:19 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 Csaba Osztrogon√°c 2012-11-01 13:59:43 PDT
It broks the Qt Mac build, please fix it as soon as possible... and watch the bots next time.
Comment 6 Csaba Osztrogon√°c 2012-11-01 14:02:44 PDT
and the Qt win build too
Comment 7 Nandor Huszka 2012-11-01 15:25:04 PDT
I've created a bug for the build fix: https://bugs.webkit.org/show_bug.cgi?id=100995
Comment 8 Dongseong Hwang 2012-11-01 16:24:41 PDT
(In reply to comment #6)
> and the Qt win build too

(In reply to comment #7)
> I've created a bug for the build fix: https://bugs.webkit.org/show_bug.cgi?id=100995

Sorry to bothering you.
Thank for reporting and bug fix.