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

Side by Side Diff: third_party/WebKit/LayoutTests/fast/harness/results.html

Issue 2038133002: Link to the Chromium git repo, not the old Blink svn repo, in layout test results. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove logging Created 4 years, 6 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 unified diff | Download patch
« no previous file with comments | « third_party/WebKit/LayoutTests/fast/harness/resources/results-test.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
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 414 matching lines...) Expand 10 before | Expand all | Expand 10 after
425 425
426 function shouldUseTracLinks() 426 function shouldUseTracLinks()
427 { 427 {
428 return !globalState().results.layout_tests_dir || !location.toString().index Of('file://') == 0; 428 return !globalState().results.layout_tests_dir || !location.toString().index Of('file://') == 0;
429 } 429 }
430 430
431 function testLinkTarget(test) 431 function testLinkTarget(test)
432 { 432 {
433 var target; 433 var target;
434 if (shouldUseTracLinks()) { 434 if (shouldUseTracLinks()) {
435 var revision = globalState().results.revision; 435 var revision = globalState().results.chromium_revision;
436 target = 'http://src.chromium.org/viewvc/blink/trunk/LayoutTests/' + tes t;
437 if (revision) 436 if (revision)
438 target += '?pathrev=' + revision; 437 target = 'https://crrev.com/' + revision;
439 target += '#l1'; 438 else
439 target = 'https://chromium.googlesource.com/chromium/src/+/master';
440 target += '/third_party/WebKit/LayoutTests/' + test;
440 } else 441 } else
441 target = globalState().results.layout_tests_dir + '/' + test; 442 target = globalState().results.layout_tests_dir + '/' + test;
442 return target; 443 return target;
443 } 444 }
444 445
445 function testLink(test) 446 function testLink(test)
446 { 447 {
447 var target = testLinkTarget(test); 448 var target = testLinkTarget(test);
448 return '<a class=test-link href="' + target + '">' + test + '</a><span class =flag onclick="unflag(this)"> \u2691</span>'; 449 return '<a class=test-link href="' + target + '">' + test + '</a><span class =flag onclick="unflag(this)"> \u2691</span>';
449 } 450 }
(...skipping 971 matching lines...) Expand 10 before | Expand all | Expand 10 after
1421 updateTestlistCounts(); 1422 updateTestlistCounts();
1422 1423
1423 TestNavigator.reset(); 1424 TestNavigator.reset();
1424 OptionWriter.apply(); 1425 OptionWriter.apply();
1425 } 1426 }
1426 </script> 1427 </script>
1427 <!-- HACK: when json_results_test.js is included, loading this page runs the tes ts. 1428 <!-- HACK: when json_results_test.js is included, loading this page runs the tes ts.
1428 It is not copied to the layout-test-results output directory. --> 1429 It is not copied to the layout-test-results output directory. -->
1429 <script src="resources/results-test.js"></script> 1430 <script src="resources/results-test.js"></script>
1430 <body onload="generatePage()"></body> 1431 <body onload="generatePage()"></body>
OLDNEW
« no previous file with comments | « third_party/WebKit/LayoutTests/fast/harness/resources/results-test.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698