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

Side by Side Diff: webkit/tools/layout_tests/flakiness_dashboard.html

Issue 848007: Flakiness dashboard: use startsWith rather than contains to match expectatio... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 years, 9 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | 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 <html> 2 <html>
3 3
4 <head> 4 <head>
5 <title>Chromium/WebKit Test History</title> 5 <title>Chromium/WebKit Test History</title>
6 <style> 6 <style>
7 body { 7 body {
8 font-family: arial; 8 font-family: arial;
9 font-size: 13px; 9 font-size: 13px;
10 } 10 }
(...skipping 876 matching lines...) Expand 10 before | Expand all | Expand 10 after
887 for (var i = 0; i < expectationsArray.length; i++) { 887 for (var i = 0; i < expectationsArray.length; i++) {
888 var path = expectationsArray[i].path; 888 var path = expectationsArray[i].path;
889 var expectations = expectationsArray[i].expectations; 889 var expectations = expectationsArray[i].expectations;
890 890
891 var pathMatchesAnyTest = false; 891 var pathMatchesAnyTest = false;
892 if (allTests[path]) { 892 if (allTests[path]) {
893 pathMatchesAnyTest = true; 893 pathMatchesAnyTest = true;
894 addTestToAllExpectations(path, expectations); 894 addTestToAllExpectations(path, expectations);
895 } else { 895 } else {
896 for (var test in allTests) { 896 for (var test in allTests) {
897 if (stringContains(test, path)) { 897 if (startsWith(test, path)) {
898 pathMatchesAnyTest = true; 898 pathMatchesAnyTest = true;
899 addTestToAllExpectations(test, expectations); 899 addTestToAllExpectations(test, expectations);
900 } 900 }
901 } 901 }
902 } 902 }
903 903
904 if (!pathMatchesAnyTest) 904 if (!pathMatchesAnyTest)
905 addTestToAllExpectations(path, expectations); 905 addTestToAllExpectations(path, expectations);
906 } 906 }
907 } 907 }
(...skipping 1519 matching lines...) Expand 10 before | Expand all | Expand 10 after
2427 } 2427 }
2428 }, false); 2428 }, false);
2429 2429
2430 2430
2431 2431
2432 </script> 2432 </script>
2433 </head> 2433 </head>
2434 2434
2435 <body></body> 2435 <body></body>
2436 </html> 2436 </html>
OLDNEW
« 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