From 0f6ff8e0b2c190c26c6598778165f4a5cbfb733b Mon Sep 17 00:00:00 2001 From: "jparent@chromium.org" Date: Fri, 12 Mar 2010 01:03:05 +0000 Subject: Flakiness dashboard: use startsWith rather than contains to match expectation to test name. Dashboard was incorrectly matching a line like BUGFOO LINUX : media = PASS FAIL TIMEOUT with any test with the substring 'media' in it, like "fast/repaint/intermediate-layout-position-clip.html". TEST=none BUG=none Review URL: http://codereview.chromium.org/848007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41377 0039d316-1c4b-4281-b951-d872f2087c98 --- webkit/tools/layout_tests/flakiness_dashboard.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/webkit/tools/layout_tests/flakiness_dashboard.html b/webkit/tools/layout_tests/flakiness_dashboard.html index 5b1962f..2214e10 100644 --- a/webkit/tools/layout_tests/flakiness_dashboard.html +++ b/webkit/tools/layout_tests/flakiness_dashboard.html @@ -894,7 +894,7 @@ addTestToAllExpectations(path, expectations); } else { for (var test in allTests) { - if (stringContains(test, path)) { + if (startsWith(test, path)) { pathMatchesAnyTest = true; addTestToAllExpectations(test, expectations); } -- cgit v1.1