Summary: | [SKIA] Move forward decl of skia type outside namespace Webkit | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Brian Salomon <bsalomon> | ||||
Component: | New Bugs | Assignee: | Brian Salomon <bsalomon> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | senorblanco, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Brian Salomon
2011-08-25 07:48:44 PDT
Created attachment 105191 [details]
Patch
D'oh! I didn't notice that I was adding the forward decl inside of a namespace in my original patch. Comment on attachment 105191 [details]
Patch
Looks good. r=me
Did this cause a build failure? Or just that it would have in the future?
There was no build failure. If left as is it would have caused the subclass implementations of grGLInterface() to have to cast from GrGLInterface* to the bogus WebKit::GrGLInterface* type. Comment on attachment 105191 [details] Patch Clearing flags on attachment: 105191 Committed r93791: <http://trac.webkit.org/changeset/93791> All reviewed patches have been landed. Closing bug. |