Cleanup deprecated PyAuto media tests.

Cleanup deprecated PyAuto tests.

Now that all the new tests have landed, the old versions can be
removed. Also removes data files which are Chrome OS only, we can
add it back to ChromeOS specific tests when we have those.

BUG=110779, 106263
TEST=none

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

[DaleCurtis, 2012-03-10 02:54:59]

Cleanup bonanza!

[imasaki1, 2012-03-10 03:53:24]

Did you make sure those files are not used?

For example, the following files might be used.
chrome/test/data/media/youtube.html

Please double check with the author of the each file.

I am OK with deleting all the files that I am the author.

Thanks.

[Ami GONE FROM CHROMIUM, 2012-03-10 18:02:55]

What's with chrome/test/functional/media/media_fps_perf.py ?

I don't know anything about any of this, so I'm deferring to Kenji for committer
review.

[shadi, 2012-03-12 17:53:06]

Ami, 

media_fps_perf.py is supposed to replace media_fps.py. However,
media_fps_perf.py got combined with cpu/memory perf test into
media_stat_perf.py.

I think you missed that because scherkus gave the LGTM on it. You can still take
a look and suggest changes if you want:
http://codereview.chromium.org/9464004/

On 2012/03/10 18:02:55, Ami Fischman wrote:
> What's with chrome/test/functional/media/media_fps_perf.py ?
> 
> I don't know anything about any of this, so I'm deferring to Kenji for
committer
> review.

[Ami GONE FROM CHROMIUM, 2012-03-12 17:55:57]

>
> media_fps_perf.py is supposed to replace media_fps.py. However,
> media_fps_perf.py got combined with cpu/memory perf test into
> media_stat_perf.py.
>

My point was that in Dale's
CL, chrome/test/functional/media/media_fps_perf.py shows up as a Modified
file, withbefore/after both being empty (
http://codereview.chromium.org/9666032/diff/1/chrome/test/functional/media/me...).
 I suspect a rebase is in order (and possibly some gitclobbery).

-a

[DaleCurtis, 2012-03-12 18:11:42]

Thanks for the pointer Kenji, all of the root files in data/media are used by a
media UI test here:

http://git.chromium.org/gitweb/?p=chromium.git;a=blob;f=chrome/browser/media_...

media_stat_perf looks a little funny since I've chmod'd it 750 from 640.
Everything should be rebased now.

PTAL.

[imasaki1, 2012-03-13 20:30:11]

lgtm

[commit-bot: I haz the power, 2012-03-13 20:33:00]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[DaleCurtis, 2012-03-13 20:37:12]

+dbj can you lgtm this?

[dennis_jeffrey, 2012-03-13 22:30:49]

LGTM

[commit-bot: I haz the power, 2012-03-13 22:48:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/9666032/5001

[commit-bot: I haz the power, 2012-03-13 22:52:46]

Try job failure for 9666032-5001 on mac_rel for step "update".
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

Step "update" is always a major failure.
Look at the try server FAQ for more details.