Attachment 372548 Details for Bug 198998 – WIP

[patch] WIP

bug-198998-20190620104754.patch (text/plain), 9.33 KB, created by Zan Dobersek on 2019-06-20 01:47:56 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Zan Dobersek

Created: 2019-06-20 01:47:56 PDT

Size: 9.33 KB

patch

obsolete

>Subversion Revision: 246611
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index d683c169c7d465b5a52369dc38150d59ed767f28..1037b3a7454c9b51bd69f8e4aadf8c842febce56 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,24 @@
>+2019-06-20  Zan Dobersek  <zdobersek@igalia.com>
>+
>+        REGRESSION(r246538): [GTK] Scroll is broken
>+        https://bugs.webkit.org/show_bug.cgi?id=198998
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        No new tests (OOPS!).
>+
>+        * platform/graphics/nicosia/NicosiaPlatformLayer.h:
>+        (Nicosia::CompositionLayer::flushState):
>+        * platform/graphics/texmap/TextureMapperLayer.cpp:
>+        (WebCore::TextureMapperLayer::computeTransformsRecursive):
>+        (WebCore::TextureMapperLayer::setBoundsOrigin):
>+        * platform/graphics/texmap/TextureMapperLayer.h:
>+        * platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp:
>+        (WebCore::CoordinatedGraphicsLayer::setBoundsOrigin):
>+        (WebCore::CoordinatedGraphicsLayer::flushCompositingStateForThisLayerOnly):
>+        (WebCore::CoordinatedGraphicsLayer::computeTransformedVisibleRect):
>+        * platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h:
>+
> 2019-06-19  Jer Noble  <jer.noble@apple.com>
> 
>         iOS 12.2 Drawing portrait video to canvas is sideways
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 49e261f0208047f8ba7857b21a08c0476b664562..281be8ae3370d79734b61d559bc7bb2aad0a673a 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,13 @@
>+2019-06-20  Zan Dobersek  <zdobersek@igalia.com>
>+
>+        REGRESSION(r246538): [GTK] Scroll is broken
>+        https://bugs.webkit.org/show_bug.cgi?id=198998
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp:
>+        (WebKit::CoordinatedGraphicsScene::updateSceneState):
>+
> 2019-06-19  Sihui Liu  <sihui_liu@apple.com>
> 
>         Remove unused originsWithCredentials from WebsiteData
>diff --git a/Source/WebCore/platform/graphics/nicosia/NicosiaPlatformLayer.h b/Source/WebCore/platform/graphics/nicosia/NicosiaPlatformLayer.h
>index a3bf287eb8d752c9cc7104f47e21e2c06ff35a17..89ce59474fe0fabfbfd3c6f7426d91442965b902 100644
>--- a/Source/WebCore/platform/graphics/nicosia/NicosiaPlatformLayer.h
>+++ b/Source/WebCore/platform/graphics/nicosia/NicosiaPlatformLayer.h
>@@ -110,6 +110,7 @@ public:
>                 struct {
>                     bool positionChanged : 1;
>                     bool anchorPointChanged : 1;
>+                    bool boundsOriginChanged : 1;
>                     bool sizeChanged : 1;
>                     bool transformChanged : 1;
>                     bool childrenTransformChanged : 1;
>@@ -154,6 +155,7 @@ public:
> 
>         WebCore::FloatPoint position;
>         WebCore::FloatPoint3D anchorPoint;
>+        WebCore::FloatPoint boundsOrigin;
>         WebCore::FloatSize size;
> 
>         WebCore::TransformationMatrix transform;
>@@ -210,6 +212,8 @@ public:
>             staging.position = pending.position;
>         if (pending.delta.anchorPointChanged)
>             staging.anchorPoint = pending.anchorPoint;
>+        if (pending.delta.boundsOriginChanged)
>+            staging.boundsOrigin = pending.boundsOrigin;
>         if (pending.delta.sizeChanged)
>             staging.size = pending.size;
> 
>diff --git a/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp b/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp
>index 5e9c56b5f30f0228bb87058fbf30b209fc939bd1..fdecbdd41dc7049e699d8dd93fe31ea24bb922f0 100644
>--- a/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp
>+++ b/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp
>@@ -68,7 +68,7 @@ void TextureMapperLayer::computeTransformsRecursive()
> 
>         m_layerTransforms.combined = parentTransform;
>         m_layerTransforms.combined
>-            .translate3d(originX + m_state.pos.x(), originY + m_state.pos.y(), m_state.anchorPoint.z())
>+            .translate3d(originX + m_state.pos.x() - m_state.boundsOrigin.x(), originY + m_state.pos.y() - m_state.boundsOrigin.y(), m_state.anchorPoint.z())
>             .multiply(m_layerTransforms.localTransform);
> 
>         m_layerTransforms.combinedForChildren = m_layerTransforms.combined;
>@@ -515,6 +515,11 @@ void TextureMapperLayer::setPosition(const FloatPoint& position)
>     m_state.pos = position;
> }
> 
>+void TextureMapperLayer::setBoundsOrigin(const FloatPoint& boundsOrigin)
>+{
>+    m_state.boundsOrigin = boundsOrigin;
>+}
>+
> void TextureMapperLayer::setSize(const FloatSize& size)
> {
>     m_state.size = size;
>diff --git a/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.h b/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.h
>index ea259ba1006b256eb1af8f699aee9322b1cc5eb2..d39d6fca95692bdfaab0290536d1a10089862149 100644
>--- a/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.h
>+++ b/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.h
>@@ -56,6 +56,7 @@ public:
>     void setMaskLayer(TextureMapperLayer*);
>     void setReplicaLayer(TextureMapperLayer*);
>     void setPosition(const FloatPoint&);
>+    void setBoundsOrigin(const FloatPoint&);
>     void setSize(const FloatSize&);
>     void setAnchorPoint(const FloatPoint3D&);
>     void setPreserves3D(bool);
>@@ -135,7 +136,7 @@ private:
> 
>     inline FloatRect layerRect() const
>     {
>-        return FloatRect(FloatPoint::zero(), m_state.size);
>+        return FloatRect(m_state.boundsOrigin, m_state.size);
>     }
> 
>     Vector<TextureMapperLayer*> m_children;
>@@ -150,6 +151,7 @@ private:
>     struct State {
>         FloatPoint pos;
>         FloatPoint3D anchorPoint;
>+        FloatPoint boundsOrigin;
>         FloatSize size;
>         TransformationMatrix transform;
>         TransformationMatrix childrenTransform;
>diff --git a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
>index a95382c9d747959cb0de24bbd9b261a1c1fc0e7d..c099cae2c742ebbdd09fd881020d04e708311af3 100644
>--- a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
>+++ b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.cpp
>@@ -259,6 +259,16 @@ void CoordinatedGraphicsLayer::setSize(const FloatSize& size)
>     didChangeGeometry();
> }
> 
>+void CoordinatedGraphicsLayer::setBoundsOrigin(const FloatPoint& boundsOrigin)
>+{
>+    if (this->boundsOrigin() == boundsOrigin)
>+        return;
>+
>+    GraphicsLayer::setBoundsOrigin(boundsOrigin);
>+    m_nicosia.delta.boundsOriginChanged = true;
>+    didChangeGeometry();
>+}
>+
> void CoordinatedGraphicsLayer::setTransform(const TransformationMatrix& t)
> {
>     if (transform() == t)
>@@ -724,6 +734,8 @@ void CoordinatedGraphicsLayer::flushCompositingStateForThisLayerOnly()
>                     state.position = m_adjustedPosition;
>                 if (localDelta.anchorPointChanged)
>                     state.anchorPoint = m_adjustedAnchorPoint;
>+                if (localDelta.boundsOriginChanged)
>+                    state.boundsOrigin = m_boundsOrigin;
>                 if (localDelta.sizeChanged)
>                     state.size = m_adjustedSize;
> 
>@@ -1097,7 +1109,7 @@ void CoordinatedGraphicsLayer::computeTransformedVisibleRect()
>     m_layerTransform.setLocalTransform(currentTransform);
> 
>     m_layerTransform.setAnchorPoint(m_adjustedAnchorPoint);
>-    m_layerTransform.setPosition(m_adjustedPosition);
>+    m_layerTransform.setPosition({ m_adjustedPosition.x() - m_boundsOrigin.x(), m_adjustedPosition.y() - m_boundsOrigin.y() });
>     m_layerTransform.setSize(m_adjustedSize);
> 
>     m_layerTransform.setFlattening(!preserves3D());
>diff --git a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h
>index b5177de8cec528866a1c80a8f69b5ca72dd2b2ec..23f94394643db78a878e457147620bb84ece9603 100644
>--- a/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h
>+++ b/Source/WebCore/platform/graphics/texmap/coordinated/CoordinatedGraphicsLayer.h
>@@ -74,6 +74,7 @@ public:
>     void setPosition(const FloatPoint&) override;
>     void setAnchorPoint(const FloatPoint3D&) override;
>     void setSize(const FloatSize&) override;
>+    void setBoundsOrigin(const FloatPoint&) override;
>     void setTransform(const TransformationMatrix&) override;
>     void setChildrenTransform(const TransformationMatrix&) override;
>     void setPreserves3D(bool) override;
>diff --git a/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp b/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp
>index eb6af9806244899e86c7c5789987f4b2a59a1bc0..5616f55aa81850f686c919b7b3fae4c8b78171f6 100644
>--- a/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp
>+++ b/Source/WebKit/Shared/CoordinatedGraphics/CoordinatedGraphicsScene.cpp
>@@ -289,6 +289,8 @@ void CoordinatedGraphicsScene::updateSceneState()
>                             layer.setPosition(layerState.position);
>                         if (layerState.delta.anchorPointChanged)
>                             layer.setAnchorPoint(layerState.anchorPoint);
>+                        if (layerState.delta.boundsOriginChanged)
>+                            layer.setBoundsOrigin(layerState.boundsOrigin);
>                         if (layerState.delta.sizeChanged)
>                             layer.setSize(layerState.size);
>

Actions: View | Formatted Diff | Diff

Attachments on bug 198998: 372458 | 372478 | 372548 | 379633