ppapi/tests: Enable URLLoader test out-of-process and via NaCl.

ppapi/tests: Enable URLLoader test out-of-process and via NaCl.

Depends on: http://codereview.chromium.org/8682022/

BUG=89961
TEST=out/Debug/ui_tests
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112065

[polina, 2011-11-29 09:08:13]

Everything is passing locally, but I am still confirming out if the tests pass
reliably on the bots or occasionally time-out.

[bbudge, 2011-11-29 18:04:51]

LGTM w/nit.

http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader.cc
File ppapi/tests/test_url_loader.cc (right):

http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader....
ppapi/tests/test_url_loader.cc:466: // FileIO is not yet support by ppapi/proxy.
support->supported

[dmichael (off chromium), 2011-11-29 18:11:12]

http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader.cc
File ppapi/tests/test_url_loader.cc (right):

http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader....
ppapi/tests/test_url_loader.cc:110: if (!testing_interface_->IsOutOfProcess()) {
Why not instead use the new way for enumerating all tests, and turn these off at
ppapi_uitest.cc, like most other tests?
I'll re-forward you the e-mail about it.

[polina, 2011-11-29 20:36:37]

http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader.cc
File ppapi/tests/test_url_loader.cc (right):

http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader....
ppapi/tests/test_url_loader.cc:110: if (!testing_interface_->IsOutOfProcess()) {
On 2011/11/29 18:11:13, dmichael wrote:
> Why not instead use the new way for enumerating all tests, and turn these off
at
> ppapi_uitest.cc, like most other tests?
> I'll re-forward you the e-mail about it.

These are the only two tests that are fully incompatible. All the other ones,
only have parts of the test case that's skipped. 

It is on my todo list to convert this test to your way (I have seen it). But I
was going to do this in a separate CL. I like to break up different things into
different CLs. Plus I am going on vacation tomorrow, so it would be useful to at
least get some intermediate results in.

[dmichael (off chromium), 2011-11-29 20:40:50]

lgtm

[polina, 2011-11-29 23:16:22]

http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader.cc
File ppapi/tests/test_url_loader.cc (right):

http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader....
ppapi/tests/test_url_loader.cc:466: // FileIO is not yet support by ppapi/proxy.
On 2011/11/29 18:04:51, bbudge1 wrote:
> support->supported

Done.

[polina, 2011-11-29 23:29:25]

On 2011/11/29 23:16:22, polina wrote:
>
http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader.cc
> File ppapi/tests/test_url_loader.cc (right):
> 
>
http://codereview.chromium.org/8682023/diff/3001/ppapi/tests/test_url_loader....
> ppapi/tests/test_url_loader.cc:466: // FileIO is not yet support by
ppapi/proxy.
> On 2011/11/29 18:04:51, bbudge1 wrote:
> > support->supported
> 
> Done.

Committed revision 112065.