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

Unified Diff: LayoutTests/fast/harness/results.html

Issue 109763007: Reland r159735 - support for test harness (text-only) tests w/o expected results. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: add a single updated baseline for an example of the new output Created 7 years 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
Index: LayoutTests/fast/harness/results.html
diff --git a/LayoutTests/fast/harness/results.html b/LayoutTests/fast/harness/results.html
index 4684644ba92f16334e639e6d899d727327dbb77a..74b9baf2e2a16fecd8081ce28ac1c6f603bdc2eb 100644
--- a/LayoutTests/fast/harness/results.html
+++ b/LayoutTests/fast/harness/results.html
@@ -576,7 +576,11 @@ function tableRow(testObject)
var actual = testObject.actual;
if (actual.indexOf('TEXT') != -1) {
globalState().hasTextFailures = true;
- row += textResultLinks(testPrefix);
+ if (testObject.is_testharness_test) {
ojan 2013/12/17 01:33:08 Nit: single-line if/else shouldn't have curlies.
+ row += resultLink(testPrefix, '-actual.txt', 'actual');
+ } else {
+ row += textResultLinks(testPrefix);
+ }
}
if (actual.indexOf('AUDIO') != -1) {
« no previous file with comments | « LayoutTests/css3/device-adapt/opera/cascading-001-expected.txt ('k') | LayoutTests/resources/testharnessreport.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698