Chromium Code Reviews Chromium Code Reviews
Issue 23496003:
User interruption should still create results.html for run-webkit-tests. (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| OLD | NEW |
|---|---|
| 1 <!DOCTYPE html> | 1 <!DOCTYPE html> |
| 2 <style> | 2 <style> |
| 3 html { | 3 html { |
| 4 height: 100%; | 4 height: 100%; |
| 5 } | 5 } |
| 6 body { | 6 body { |
| 7 margin: 0; | 7 margin: 0; |
| 8 font-family: Helvetica, sans-serif; | 8 font-family: Helvetica, sans-serif; |
| 9 font-size: 11pt; | 9 font-size: 11pt; |
| 10 display: -webkit-flex; | 10 display: -webkit-flex; |
| (...skipping 1277 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1288 { | 1288 { |
| 1289 forEachTest(processGlobalStateFor); | 1289 forEachTest(processGlobalStateFor); |
| 1290 | 1290 |
| 1291 var html = '<div class=content-container><div id=toolbar>' + | 1291 var html = '<div class=content-container><div id=toolbar>' + |
| 1292 '<div class="note">Use the i, j, k and l keys to navigate, e, c to expan d and collapse, and f to flag</div>' + | 1292 '<div class="note">Use the i, j, k and l keys to navigate, e, c to expan d and collapse, and f to flag</div>' + |
| 1293 '<a href="javascript:void()" onclick="expandAllExpectations()">expand al l</a> ' + | 1293 '<a href="javascript:void()" onclick="expandAllExpectations()">expand al l</a> ' + |
| 1294 '<a href="javascript:void()" onclick="collapseAllExpectations()">collaps e all</a> ' + | 1294 '<a href="javascript:void()" onclick="collapseAllExpectations()">collaps e all</a> ' + |
| 1295 '<label><input id="toggle-images" type=checkbox checked onchange="handle ToggleImagesChange()">Toggle images</label>' + | 1295 '<label><input id="toggle-images" type=checkbox checked onchange="handle ToggleImagesChange()">Toggle images</label>' + |
| 1296 '</div>'; | 1296 '</div>'; |
| 1297 | 1297 |
| 1298 if (globalState().results.interrupted) | |
| 1299 html += "<p class='stopped-running-early-message'>Testing exited early.< /p>" | |
|
eseidel
2013/08/27 14:55:08
Why would you want to remove this?
It seems we sh
Dirk Pranke
2013/08/27 18:18:25
I agree w/ eseidel. This should stay.
| |
| 1300 | |
| 1301 if (globalState().crashTests.length) | 1298 if (globalState().crashTests.length) |
| 1302 html += testList(globalState().crashTests, 'Tests that crashed', 'crash- tests-table'); | 1299 html += testList(globalState().crashTests, 'Tests that crashed', 'crash- tests-table'); |
| 1303 | 1300 |
| 1304 html += failingTestsTable(globalState().failingTests, | 1301 html += failingTestsTable(globalState().failingTests, |
| 1305 'Tests that failed text/pixel/audio diff', 'results-table'); | 1302 'Tests that failed text/pixel/audio diff', 'results-table'); |
| 1306 | 1303 |
| 1307 html += failingTestsTable(globalState().missingResults, | 1304 html += failingTestsTable(globalState().missingResults, |
| 1308 'Tests that had no expected results (probably new)', 'missing-table'); | 1305 'Tests that had no expected results (probably new)', 'missing-table'); |
| 1309 | 1306 |
| 1310 if (globalState().timeoutTests.length) | 1307 if (globalState().timeoutTests.length) |
| (...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1347 updateTestlistCounts(); | 1344 updateTestlistCounts(); |
| 1348 | 1345 |
| 1349 TestNavigator.reset(); | 1346 TestNavigator.reset(); |
| 1350 OptionWriter.apply(); | 1347 OptionWriter.apply(); |
| 1351 } | 1348 } |
| 1352 </script> | 1349 </script> |
| 1353 <!-- HACK: when json_results_test.js is included, loading this page runs the tes ts. | 1350 <!-- HACK: when json_results_test.js is included, loading this page runs the tes ts. |
| 1354 It is not copied to the layout-test-results output directory. --> | 1351 It is not copied to the layout-test-results output directory. --> |
| 1355 <script src="resources/results-test.js"></script> | 1352 <script src="resources/results-test.js"></script> |
| 1356 <body onload="generatePage()"></body> | 1353 <body onload="generatePage()"></body> |
| OLD | NEW |
