Title: [125069] trunk/Source/WebCore
Revision
125069
Author
toniki...@webkit.org
Date
2012-08-08 12:20:10 -0700 (Wed, 08 Aug 2012)

Log Message

[BlackBerry] Add boundsOrigin accessors to LayerOverride
https://bugs.webkit.org/show_bug.cgi?id=93479
PR #188677

Reviewed by Yong Li.
Patch by Antonio Gomes <ago...@rim.com>

This is needed in order to allow us to directly set the boundsOrigin value
to the LayerCompositingThread and avoid it to get overridden by the respective
LayerWebKitThread's boundsOrigin value in the next commit.

No new tests, since this is another preparation PR.

* platform/graphics/blackberry/LayerCompositingThread.cpp:
(WebCore::LayerCompositingThread::updateAnimations):
* platform/graphics/blackberry/LayerCompositingThread.h:
(WebCore::LayerOverride::isBoundsOriginSet):
(WebCore::LayerOverride::boundsOrigin):
(WebCore::LayerOverride::setBoundsOrigin):
(LayerOverride):
(WebCore::LayerOverride::LayerOverride):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (125068 => 125069)


--- trunk/Source/WebCore/ChangeLog	2012-08-08 19:19:18 UTC (rev 125068)
+++ trunk/Source/WebCore/ChangeLog	2012-08-08 19:20:10 UTC (rev 125069)
@@ -1,5 +1,28 @@
 2012-08-08  Antonio Gomes  <ago...@rim.com>
 
+        [BlackBerry] Add boundsOrigin accessors to LayerOverride
+        https://bugs.webkit.org/show_bug.cgi?id=93479
+        PR #188677
+
+        Reviewed by Yong Li.
+
+        This is needed in order to allow us to directly set the boundsOrigin value
+        to the LayerCompositingThread and avoid it to get overridden by the respective
+        LayerWebKitThread's boundsOrigin value in the next commit.
+
+        No new tests, since this is another preparation PR.
+
+        * platform/graphics/blackberry/LayerCompositingThread.cpp:
+        (WebCore::LayerCompositingThread::updateAnimations):
+        * platform/graphics/blackberry/LayerCompositingThread.h:
+        (WebCore::LayerOverride::isBoundsOriginSet):
+        (WebCore::LayerOverride::boundsOrigin):
+        (WebCore::LayerOverride::setBoundsOrigin):
+        (LayerOverride):
+        (WebCore::LayerOverride::LayerOverride):
+
+2012-08-08  Antonio Gomes  <ago...@rim.com>
+
         [BlackBerry] Propagate GraphicsLayer::boundsOrigin down to the platform layers
         https://bugs.webkit.org/show_bug.cgi?id=93478
         PR #188657

Modified: trunk/Source/WebCore/platform/graphics/blackberry/LayerCompositingThread.cpp (125068 => 125069)


--- trunk/Source/WebCore/platform/graphics/blackberry/LayerCompositingThread.cpp	2012-08-08 19:19:18 UTC (rev 125068)
+++ trunk/Source/WebCore/platform/graphics/blackberry/LayerCompositingThread.cpp	2012-08-08 19:20:10 UTC (rev 125069)
@@ -519,6 +519,8 @@
             m_transform = m_override->transform();
         if (m_override->isOpacitySet())
             m_opacity = m_override->opacity();
+        if (m_override->isBoundsOriginSet())
+            m_boundsOrigin = m_override->boundsOrigin();
 
         for (size_t i = 0; i < m_override->animations().size(); ++i) {
             LayerAnimation* animation = m_override->animations()[i].get();

Modified: trunk/Source/WebCore/platform/graphics/blackberry/LayerCompositingThread.h (125068 => 125069)


--- trunk/Source/WebCore/platform/graphics/blackberry/LayerCompositingThread.h	2012-08-08 19:19:18 UTC (rev 125068)
+++ trunk/Source/WebCore/platform/graphics/blackberry/LayerCompositingThread.h	2012-08-08 19:20:10 UTC (rev 125069)
@@ -83,6 +83,10 @@
     float opacity() const { return m_opacity; }
     void setOpacity(float) { m_opacity = opacity; m_opacitySet = true; }
 
+    bool isBoundsOriginSet() const { return m_boundsOriginSet; }
+    FloatPoint boundsOrigin() const { return m_boundsOrigin; }
+    void setBoundsOrigin(const FloatPoint& origin) { m_boundsOrigin = origin; m_boundsOriginSet = true; }
+
     const Vector<RefPtr<LayerAnimation> >& animations() const { return m_animations; }
     void addAnimation(PassRefPtr<LayerAnimation> animation) { m_animations.append(animation); }
     void removeAnimation(const String& name);
@@ -95,6 +99,7 @@
         , m_boundsSet(false)
         , m_transformSet(false)
         , m_opacitySet(false)
+        , m_boundsOriginSet(false)
     {
     }
 
@@ -103,6 +108,7 @@
     IntSize m_bounds;
     TransformationMatrix m_transform;
     float m_opacity;
+    FloatPoint m_boundsOrigin;
 
     Vector<RefPtr<LayerAnimation> > m_animations;
 
@@ -111,6 +117,7 @@
     unsigned m_boundsSet : 1;
     unsigned m_transformSet : 1;
     unsigned m_opacitySet : 1;
+    unsigned m_boundsOriginSet : 1;
 };
 
 class LayerFilterRendererAction;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to