Changeset 82715 in webkit

Timestamp:

Apr 1, 2011 1:59:35 PM (13 years ago)

Author:

mdelaney@apple.com

Message:

2011-04-01 Matthew Delaney <​mdelaney@apple.com>

Reviewed by Simon Fraser.

Behavior of isAccelerated() for a IOSurface-backed canvas should be consistent with accelerated status of its ImageBuffer
​https://bugs.webkit.org/show_bug.cgi?id=57651

No new tests. This patch does not affect outward behavior.

• html/canvas/CanvasRenderingContext2D.cpp: (WebCore::CanvasRenderingContext2D::isAccelerated):
• platform/graphics/ImageBuffer.h: (WebCore::ImageBuffer::isAccelerated):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• html/canvas/CanvasRenderingContext2D.cpp (modified) (1 diff)
• platform/graphics/ImageBuffer.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-04-01  Matthew Delaney  <mdelaney@apple.com>
+
+        Reviewed by Simon Fraser.
+
+        Behavior of isAccelerated() for a IOSurface-backed canvas should be consistent with accelerated status of its ImageBuffer
+        https://bugs.webkit.org/show_bug.cgi?id=57651
+
+        No new tests. This patch does not affect outward behavior.
+
+        * html/canvas/CanvasRenderingContext2D.cpp:
+        (WebCore::CanvasRenderingContext2D::isAccelerated):
+        * platform/graphics/ImageBuffer.h:
+        (WebCore::ImageBuffer::isAccelerated):
+
 2011-04-01  Daniel Bates  <dbates@rim.com>
 

trunk/Source/WebCore/html/canvas/CanvasRenderingContext2D.cpp

@@ -147 +147 @@
 {
 #if USE(IOSURFACE_CANVAS_BACKING_STORE)
-    return canvas()->document()->page()->settings()->canvasUsesAcceleratedDrawing();
+    return canvas()->buffer()->isAccelerated();
 #elif ENABLE(ACCELERATED_2D_CANVAS)
     return m_context3D;

trunk/Source/WebCore/platform/graphics/ImageBuffer.h

@@ -87 +87 @@
         GraphicsContext* context() const;
 
+        bool isAccelerated() const { return m_accelerateRendering; }
         bool drawsUsingCopy() const; // If the image buffer has to render using a copied image, it will return true.
         PassRefPtr<Image> copyImage() const; // Return a new image that is a copy of the buffer.