Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(67)

Unified Diff: LayoutTests/fast/dom/CompositorProxy/proxy-forces-layer.html

Issue 1153023002: layout-tests: Add flag for including compositingReasons in layerTreeAsText(). (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Source/platform/graphics/GraphicsLayer.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fast/dom/CompositorProxy/proxy-forces-layer.html
diff --git a/LayoutTests/fast/dom/CompositorProxy/proxy-forces-layer.html b/LayoutTests/fast/dom/CompositorProxy/proxy-forces-layer.html
index 4ecbf97fc7f542c0eeca61f1f566935de83c4aeb..274950ecd86322fa2b7c6e3c50865544be5c5860 100644
--- a/LayoutTests/fast/dom/CompositorProxy/proxy-forces-layer.html
+++ b/LayoutTests/fast/dom/CompositorProxy/proxy-forces-layer.html
@@ -18,6 +18,7 @@ async_test(function(test) {
docLayer = JSON.parse(window.internals.layerTreeAsText(document));
assert_equals(1, docLayer.children.length);
assert_equals(1, docLayer.children[0].children.length);
+ assert_not_equals(docLayer.children[0].children[0].compositingReasons.indexOf("compositorProxy"), -1);
esprehn 2015/05/25 21:05:43 assert_true(...compositingReasons.includes("compos
sadrul 2015/05/25 21:08:45 compositingReasons is actually an array, and there
esprehn 2015/05/25 21:13:13 Oh then you want to use assert_in_array then.
sadrul 2015/05/25 21:19:32 Nice! Done. Thanks! (I looked for something like a
// Detach the element from the document. The corresponding layer should be removed.
container.parentNode.removeChild(container);
@@ -30,6 +31,7 @@ async_test(function(test) {
docLayer = JSON.parse(window.internals.layerTreeAsText(document));
assert_equals(1, docLayer.children.length);
assert_equals(1, docLayer.children[0].children.length);
+ assert_not_equals(docLayer.children[0].children[0].compositingReasons.indexOf("compositorProxy"), -1);
// Disconnecting the proxy should also remove the layer.
proxy.disconnect();
« no previous file with comments | « no previous file | Source/platform/graphics/GraphicsLayer.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698