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

Unified Diff: LayoutTests/paint/invalidation/spv2/resources/paint-invalidation-test.js

Issue 1340423004: Change output method of paint invalidation failure message (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 3 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/paint/invalidation/spv2/resources/paint-invalidation-test.js
diff --git a/LayoutTests/paint/invalidation/spv2/resources/paint-invalidation-test.js b/LayoutTests/paint/invalidation/spv2/resources/paint-invalidation-test.js
index efbedb7b1033476f44d1c94af8c357f83a007c80..9e4b0dfb84e3bf2f047caa17d225fd248bc8c129 100644
--- a/LayoutTests/paint/invalidation/spv2/resources/paint-invalidation-test.js
+++ b/LayoutTests/paint/invalidation/spv2/resources/paint-invalidation-test.js
@@ -46,15 +46,30 @@ function finishPaintInvalidationTest()
internals.stopTrackingPaintInvalidationObjects();
if (paintInvalidationObjects != expectedPaintInvalidationObjects) {
+ // TODO(wangxianzhu): The following works for HTML only. Should support SVG.
pdr. 2015/09/16 21:27:02 Lets support SVG: function finishPaintInvalidatio
var message = 'Paint invalidation objects mismatch.'
+ '\nExpected:\n' + expectedPaintInvalidationObjects.replace(/","/g, '",\n "')
+ '\nActual:\n' + paintInvalidationObjects.replace(/","/g, '",\n "');
testRunner.logToStderr(message);
- while (document.body.firstChild)
- document.body.removeChild(document.body.firstChild);
- var pre = document.createElement('pre');
- pre.appendChild(document.createTextNode('(To copy results as text, see stderr.)\n\n' + message));
- document.body.appendChild(pre);
+
+ var div = document.createElement('div');
+ div.style.zIndex = '2147483647';
+ div.style.visibility = 'visible';
+ div.style.position = 'fixed';
+ div.style.top = '0';
+ div.style.right = '0';
+ div.style.width = '300px';
+ div.style.height = '36px';
+ div.style.color = 'red';
+ div.style.fontSize = '14px';
+ div.style.lineHeight = '16px';
+ div.style.fontFamily = 'Arial';
+ div.style.border = '1px solid red';
+ div.style.background = 'none';
pdr. 2015/09/16 21:27:02 Lets set the backgroundColor to white so the text
+ div.appendChild(document.createTextNode('Paint invalidation objects mismatch.'));
+ div.appendChild(document.createElement('br'));
+ div.appendChild(document.createTextNode('See stderr for details.'));
+ document.body.appendChild(div);
}
testRunner.notifyDone();
});
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698