Bug 47114

Summary: Add ENABLE(ACCELERATED_2D_CANVAS) guard for m_uploadTexture
Product: WebKit Reporter: Kwang Yul Seo <skyul>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
kling: review-
Patch none

Description Kwang Yul Seo 2010-10-04 14:32:36 PDT
PlatformContextSkia's m_uploadTexture is used only in code guarded with ENABLE(ACCELERATED_2D_CANVAS).
Comment 1 Kwang Yul Seo 2010-10-04 14:33:32 PDT
Created attachment 69683 [details]
Patch
Comment 2 Andreas Kling 2010-10-06 23:45:11 PDT
Comment on attachment 69683 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=69683&action=review

> WebCore/platform/graphics/skia/PlatformContextSkia.h:236
> +#if ENABLE(ACCELERATED_2D_CANVAS)
>      mutable RefPtr<Texture> m_uploadTexture;
> +#endif

There's an ENABLE(ACCELERATED_2D_CANVAS) guard just above, put both variables in there instead.
Comment 3 Kwang Yul Seo 2010-10-07 12:43:22 PDT
Created attachment 70131 [details]
Patch

Move m_uploadTexture to the ENABLE(ACCELERATED_2D_CANVAS) guard above.
Comment 4 Andreas Kling 2010-10-09 10:43:16 PDT
Comment on attachment 70131 [details]
Patch

LGTM.
Comment 5 WebKit Commit Bot 2010-10-09 11:11:26 PDT
Comment on attachment 70131 [details]
Patch

Clearing flags on attachment: 70131

Committed r69450: <http://trac.webkit.org/changeset/69450>
Comment 6 WebKit Commit Bot 2010-10-09 11:11:34 PDT
All reviewed patches have been landed.  Closing bug.