Introduce AddMediaTests factory method.

Introduce AddMediaTests factory method.

As titled, introduces a new AddMediaTests factory method which will
eventually become the home for all media tests.

None of the existing tests are actually working right now, so I've
decided to bite the bullet and drop them all now. We'll focus on
fixing these next quarter.

BUG=106257, 106260
TEST=Ran local buildbot.


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

[DaleCurtis, 2011-12-16 22:20:22]

Comments from imasaki and nirnimesh convinced me to switch to using
pyauto_functional --suite instead of using a custom launcher.

Unfortunately, this leads to quite a bit of yak shaving. I've verified the
cleanups and refactoring on my local Linux Buildbot, but would imagine there
might be some Windows/Mac quirks. Nirnimesh, what's your process for testing
PyAuto Buildbot changes?

[Nirnimesh, 2011-12-20 00:34:43]

http://codereview.chromium.org/8972010/diff/5001/scripts/master/factory/chrom...
File scripts/master/factory/chromium_commands.py (left):

http://codereview.chromium.org/8972010/diff/5001/scripts/master/factory/chrom...
scripts/master/factory/chromium_commands.py:651: env={'PYTHONPATH': '.'},
I think this is needed for QA continuous waterfall to work.

http://codereview.chromium.org/8972010/diff/5001/scripts/master/factory/chrom...
File scripts/master/factory/chromium_commands.py (right):

http://codereview.chromium.org/8972010/diff/5001/scripts/master/factory/chrom...
scripts/master/factory/chromium_commands.py:619: workdir: the working dir for
this step
workdir is unused? I think you should pass it along, or some setups (QA
continuous waterfall) will not work.

[DaleCurtis, 2011-12-20 22:42:42]

http://codereview.chromium.org/8972010/diff/5001/scripts/master/factory/chrom...
File scripts/master/factory/chromium_commands.py (right):

http://codereview.chromium.org/8972010/diff/5001/scripts/master/factory/chrom...
scripts/master/factory/chromium_commands.py:619: workdir: the working dir for
this step
On 2011/12/20 00:34:43, Nirnimesh wrote:
> workdir is unused? I think you should pass it along, or some setups (QA
> continuous waterfall) will not work.

Okay, I dropped it and everything worked, so thought it might be deprecated.
Ditto for the other option. I'll pipe kwargs to _AddBasicPythonTest to support
workdir and env.

[Nirnimesh, 2011-12-20 23:02:29]

LGTM

[cmp, 2011-12-20 23:22:59]

lgtm

[commit-bot: I haz the power, 2011-12-20 23:26:47]

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

[commit-bot: I haz the power, 2011-12-20 23:29:48]

Change committed as 115223

[Nirnimesh, 2011-12-28 22:28:10]

On 2011/12/20 23:29:48, I haz the power (commit-bot) wrote:
> Change committed as 115223

I'm going to have to revert this because it causes the QA waterfall to fail.

http://chrome-master2.mtv.corp.google.com:8011/builders/Chrome%20Win%20XP%20Q...

Besides, it's also not using runtest.py for win and linux.

[DaleCurtis, 2012-01-03 18:32:09]

Ugh, sorry about that. It was using runtest on my local machine. I suspect
something odd with the linux vs linux2 checks. The checks between what PyAuto
wants and WebDriver wants appear mutually exclusive, so I'll just revert my
attempt at creating a common Python test method. New CL will be up shortly.


On 2011/12/28 22:28:10, Nirnimesh wrote:
> On 2011/12/20 23:29:48, I haz the power (commit-bot) wrote:
> > Change committed as 115223
> 
> I'm going to have to revert this because it causes the QA waterfall to fail.
> 
>
http://chrome-master2.mtv.corp.google.com:8011/builders/Chrome%2520Win%2520XP...
> 
> Besides, it's also not using runtest.py for win and linux.