Per issue 125757, added more test cases to be monitored in the layout test

Per issue 125757, added more test cases to be monitored in the layout test anazlyzer. Also, changed CSV format to use double quotations for seperators instead of commas.

BUG=125757
TEST=make sure layout analzyer runs with new test cases

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=135246

[imasaki1, 2012-05-01 20:01:50]

This change will enable monitoring layout test cases you asked for. 

I used to declare test cases (not to use recursion as you mentioned). I cannot
use recursion since SVN recursion is quite expensive.

[imasaki1, 2012-05-01 21:26:45]

After discussion with ddorwin and scherkus, I have decided to enable recursion.

[ddorwin, 2012-05-01 22:32:37]

Thanks for making this recursive. A few small comments and added scherkus to
make sure we didn't miss any.

https://chromiumcodereview.appspot.com/10261034/diff/1/media/tools/layout_tes...
File media/tools/layout_tests/testname/media.csv (right):

https://chromiumcodereview.appspot.com/10261034/diff/1/media/tools/layout_tes...
media/tools/layout_tests/testname/media.csv:26: "media/encrypted-media/\S+.html"
I think we should make all these alphabetical.

https://chromiumcodereview.appspot.com/10261034/diff/1/media/tools/layout_tes...
media/tools/layout_tests/testname/media.csv:28:
"media/W3C/audio/canPlayType/\S+.html"
This is going to be problematic to maintain since these are imported from W3C
tests. :(

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
File media/tools/layout_tests/testname/media.csv (right):

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/testname/media.csv:3:
"http/tests/security/\S+video\S+.html"
Do we need the S+.html?

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/testname/media.csv:24:
"platform/chromium/media/\S+.html"
Do we need S+.html?

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/testname/media.csv:25:
"fast/events/constructors/media-key-event-constructor.html"
Please alphabetize these and some of those above.

[scherkus (not reviewing), 2012-05-02 00:50:54]

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
File media/tools/layout_tests/layouttest_analyzer.py (right):

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/layouttest_analyzer.py:146: recursion = True
didn't you mention there were performance issues?

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
File media/tools/layout_tests/testname/media.csv (right):

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/testname/media.csv:1: "http/tests/media/"
why are " needed now?

it causes for quite a messy diff :(

[imasaki1, 2012-05-03 23:31:14]

PTAL? thanks.

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
File media/tools/layout_tests/layouttest_analyzer.py (right):

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/layouttest_analyzer.py:146: recursion = True
On 2012/05/02 00:50:55, scherkus wrote:
> didn't you mention there were performance issues?

There is performance issue (the execution becomes much slower. But, currently
layout analyzer will run every hour. So, as long as the execution finishes
within hour. It is OK.

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
File media/tools/layout_tests/testname/media.csv (right):

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/testname/media.csv:1: "http/tests/media/"
On 2012/05/02 00:50:55, scherkus wrote:
> why are " needed now?
> 
> it causes for quite a messy diff :(

Done.

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/testname/media.csv:3:
"http/tests/security/\S+video\S+.html"
On 2012/05/01 22:32:37, ddorwin wrote:
> Do we need the S+.html?

Yes. Currently, we cannot mix regular expression and recursion.

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/testname/media.csv:24:
"platform/chromium/media/\S+.html"
On 2012/05/01 22:32:37, ddorwin wrote:
> Do we need S+.html?

Removed.

https://chromiumcodereview.appspot.com/10261034/diff/2001/media/tools/layout_...
media/tools/layout_tests/testname/media.csv:25:
"fast/events/constructors/media-key-event-constructor.html"
On 2012/05/01 22:32:37, ddorwin wrote:
> Please alphabetize these and some of those above.

Done.

[scherkus (not reviewing), 2012-05-03 23:54:55]

lgtm

https://chromiumcodereview.appspot.com/10261034/diff/9001/media/tools/layout_...
File media/tools/layout_tests/testname/media.csv (right):

https://chromiumcodereview.appspot.com/10261034/diff/9001/media/tools/layout_...
media/tools/layout_tests/testname/media.csv:26: platform/chromium/media/,
nit: this isn't something we really need to worry about now, but this file isn't
really a CSV but rather a newline delimited file

it seems odd to require trailing commas