Flakiness dashboard: use startsWith rather than contains to match expectatio...

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



Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=41377

[ojan, 2010-03-12 00:55:45]

lols. whoops. lgtm.

On Thu, Mar 11, 2010 at 4:51 PM, <jparent@chromium.org> wrote:

> Reviewers: ojan,
>
> Description:
> 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
> BUT=none
>
>
>
> Please review this at http://codereview.chromium.org/848007
>
> SVN Base: svn://chrome-svn/chrome/trunk/src/
>
> Affected files:
>  M     webkit/tools/layout_tests/flakiness_dashboard.html
>
>
> Index: webkit/tools/layout_tests/flakiness_dashboard.html
> ===================================================================
> --- webkit/tools/layout_tests/flakiness_dashboard.html  (revision 41363)
> +++ webkit/tools/layout_tests/flakiness_dashboard.html  (working copy)
> @@ -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);
>           }
>
>
>