Bug 108855

Summary: [WK2] CoordinatedGraphicsLayerState is a struct, not a class
Product: WebKit Reporter: Bruno Abinader (history only) <bruno.abinader>
Component: WebKit2Assignee: Bruno Abinader (history only) <bruno.abinader>
Severity: Normal CC: commit-queue, noam, webkit.review.bot, zeno
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
kling: review+
Patch for landing none

Description Bruno Abinader (history only) 2013-02-04 13:31:28 PST
Some forward declarations of CoordinatedLayerInfo were stating it is a class, but in fact it is a struct.
Comment 1 Bruno Abinader (history only) 2013-02-04 13:34:34 PST
Created attachment 186449 [details]

Proposed patch
Comment 2 Noam Rosenthal 2013-02-04 13:45:46 PST
Comment on attachment 186449 [details]

Since this is trivial I take the liberty to r+ it :)
Comment 3 Noam Rosenthal 2013-02-04 14:07:54 PST
Comment on attachment 186449 [details]

Clearing r+, since apparently it's not ok to r+ trivial patches anymore.
Comment 4 Andreas Kling 2013-04-18 06:09:13 PDT
Comment on attachment 186449 [details]

Comment 5 Bruno Abinader (history only) 2013-04-18 06:33:07 PDT
CoordinatedLayerInfo has changed to CoordinatedGraphicsLayerState in r144787, but the forward declaration issue is still valid. I'm rebasing the patch for landing atm.
Comment 6 Bruno Abinader (history only) 2013-04-18 06:37:39 PDT
Created attachment 198730 [details]
Patch for landing
Comment 7 WebKit Commit Bot 2013-04-18 07:06:50 PDT
Comment on attachment 198730 [details]
Patch for landing

Clearing flags on attachment: 198730

Committed r148674: <http://trac.webkit.org/changeset/148674>
Comment 8 Bruno Abinader (history only) 2013-04-25 21:42:35 PDT
All reviewed patches have been landed. Closing bug.