[Webkit-unassigned] [Bug 106964] Add Chrome on Android instrumentation test types to flakiness dashboard

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=106964


Ojan Vafai <ojan at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #183229|review?                     |review-
               Flag|                            |




--- Comment #2 from Ojan Vafai <ojan at chromium.org>  2013-01-17 11:45:15 PST ---
(From update of attachment 183229)
View in context: https://bugs.webkit.org/attachment.cgi?id=183229&action=review

> Tools/ChangeLog:16
> +

Nit: extra line break.

> Tools/TestResultServer/generate_builders_json.py:108
> +        {'name': 'ChromiumFYI', 'url': 'http://build.chromium.org/p/chromium.fyi'},

As discussed offline, lets move this to a separate patch and update jsonp in that patch as well.

> Tools/TestResultServer/static-dashboards/builders.js:246
> +        // We don't have tests on ToT, but this must be included

This should be:
// FIXME: We don't have tests...

Ideally you'd also file a bug and list it in this comment. CC me and jparent at chromium.org.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.