Title: [159002] trunk/Source/WebCore
Revision
159002
Author
[email protected]
Date
2013-11-09 13:41:52 -0800 (Sat, 09 Nov 2013)

Log Message

Modernize CanvasObserverProxy
https://bugs.webkit.org/show_bug.cgi?id=124106

Reviewed by Anders Carlsson.

* css/CSSCanvasValue.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (159001 => 159002)


--- trunk/Source/WebCore/ChangeLog	2013-11-09 20:59:28 UTC (rev 159001)
+++ trunk/Source/WebCore/ChangeLog	2013-11-09 21:41:52 UTC (rev 159002)
@@ -1,3 +1,12 @@
+2013-11-09  Sam Weinig  <[email protected]>
+
+        Modernize CanvasObserverProxy
+        https://bugs.webkit.org/show_bug.cgi?id=124106
+
+        Reviewed by Anders Carlsson.
+
+        * css/CSSCanvasValue.h:
+
 2013-11-09  Patrick Gansterer  <[email protected]>
 
         Move RunLoop from WebCore to WTF

Modified: trunk/Source/WebCore/css/CSSCanvasValue.h (159001 => 159002)


--- trunk/Source/WebCore/css/CSSCanvasValue.h	2013-11-09 20:59:28 UTC (rev 159001)
+++ trunk/Source/WebCore/css/CSSCanvasValue.h	2013-11-09 21:41:52 UTC (rev 159002)
@@ -60,29 +60,31 @@
 
     // NOTE: We put the CanvasObserver in a member instead of inheriting from it
     // to avoid adding a vptr to CSSCanvasValue.
-    class CanvasObserverProxy : public CanvasObserver {
+    class CanvasObserverProxy FINAL : public CanvasObserver {
     public:
-        CanvasObserverProxy(CSSCanvasValue& ownerValue)
+        explicit CanvasObserverProxy(CSSCanvasValue& ownerValue)
             : m_ownerValue(ownerValue)
         {
         }
+
         virtual ~CanvasObserverProxy()
         {
         }
-        virtual void canvasChanged(HTMLCanvasElement& canvas, const FloatRect& changedRect)
+
+    private:
+        virtual void canvasChanged(HTMLCanvasElement& canvas, const FloatRect& changedRect) OVERRIDE
         {
             m_ownerValue.canvasChanged(canvas, changedRect);
         }
-        virtual void canvasResized(HTMLCanvasElement& canvas)
+        virtual void canvasResized(HTMLCanvasElement& canvas) OVERRIDE
         {
             m_ownerValue.canvasResized(canvas);
         }
-        virtual void canvasDestroyed(HTMLCanvasElement& canvas)
+        virtual void canvasDestroyed(HTMLCanvasElement& canvas) OVERRIDE
         {
             m_ownerValue.canvasDestroyed(canvas);
         }
 
-    private:
         CSSCanvasValue& m_ownerValue;
     };
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to