compositor-worker: cc->blink mutation plumbing

third_party/WebKit/Source/web/tests/CompositorWorkerTest.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/frame/FrameView.h"
 #include "core/layout/LayoutView.h"
 #include "core/layout/compositing/CompositedLayerMapping.h"
 #include "core/layout/compositing/PaintLayerCompositor.h"
 #include "core/page/Page.h"
 #include "platform/graphics/CompositorMutableProperties.h"
+#include "platform/graphics/CompositorMutation.h"
 #include "platform/graphics/GraphicsLayer.h"
 #include "platform/testing/URLTestHelpers.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebLayer.h"
 #include "public/platform/WebLayerTreeView.h"
 #include "public/platform/WebUnitTestSupport.h"
 #include "public/web/WebSettings.h"
 #include "public/web/WebViewClient.h"
 #include "web/WebLocalFrameImpl.h"
 #include "web/WebViewImpl.h"
@@ skipping 182 matching lines @@
     Element* scrollElement = document->getElementById("proxied-scroller");
     WebLayer* scrollLayer = scrollingWebLayerFromElement(scrollElement);
     EXPECT_FALSE(!!scrollLayer->compositorMutableProperties());
     EXPECT_EQ(0UL, scrollLayer->elementId());
 
     WebLayer* rootScrollLayer = getRootScrollLayer();
     EXPECT_FALSE(!!rootScrollLayer->compositorMutableProperties());
     EXPECT_EQ(0UL, rootScrollLayer->elementId());
 }
 
+TEST_F(CompositorWorkerTest, applyingMutations)
+{
+    registerMockedHttpURLLoad("compositor-proxy-basic.html");
+    navigateTo(m_baseURL + "compositor-proxy-basic.html");
+
+    forceFullCompositingUpdate();
+
+    Document* document = frame()->document();
+
+    Element* proxiedElement = document->getElementById("proxied");
+    WebLayer* proxiedLayer = webLayerFromElement(proxiedElement);
+    EXPECT_TRUE(proxiedLayer->compositorMutableProperties() & CompositorMutablePropertyTransform);
+    EXPECT_FALSE(proxiedLayer->compositorMutableProperties() & (CompositorMutablePropertyScrollLeft | CompositorMutablePropertyScrollTop | CompositorMutablePropertyOpacity));
+    uint64_t elementId = proxiedLayer->elementId();
+    EXPECT_NE(0UL, elementId);
+
+    {
+        TransformationMatrix transformMatrix(11, 12, 13, 14, 21, 22, 23, 24, 31, 32, 33, 34, 41, 42, 43, 44);
+        OwnPtrWillBeRawPtr<CompositorMutation> mutation = adoptPtr(new CompositorMutation);
+        mutation->setTransform(TransformationMatrix::toSkMatrix44(transformMatrix));
+
+        CompositorMutations mutations;
+        mutations.map.add(elementId, mutation.release());
+
+        webViewImpl()->applyMutations(mutations);
+
+        const String& cssValue = proxiedElement->inlineStyle()->getPropertyValue(CSSPropertyTransform);
+        EXPECT_EQ(String("matrix3d(11px, 12px, 13px, 14px, 21px, 22px, 23px, 24px, 31px, 32px, 33px, 34px, 41px, 42px, 43px, 44px)"), cssValue);
+    }
+    {
+        OwnPtrWillBeRawPtr<CompositorMutation> mutation = adoptPtr(new CompositorMutation);
+        mutation->setOpacity(0.5);
+
+        CompositorMutations mutations;
+        mutations.map.add(elementId, mutation.release());
+
+        webViewImpl()->applyMutations(mutations);
+
+        const String& cssValue = proxiedElement->inlineStyle()->getPropertyValue(CSSPropertyOpacity);
+        EXPECT_EQ(String("0.5"), cssValue);

[flackr, 2016/01/20 16:50:50]

It feels strange that you're testing mutating a property which was not proxied.
Maybe if applyMutations doesn't care about whether it is proxied or not we
shouldn't be explicitly testing the compositorMutableProperties, though I
suppose when the plumbing is complete maybe this test would actually set them
through the compositor proxy interface?

[majidvp, 2016/01/20 19:23:59]

Good point, I agree that it is odd. It should be fixed now.
I like the idea of setting values through compositor proxy but it may be 
better as a separate test.

+    }
+}
+
 } // namespace blink