Remove two URLRequestTest cases from net_unittest disabled list on Android.

Remove two URLRequestTest cases from net_unittest disabled list on Android.

For URLRequestTest.FileDirRedirectNoCrash, it passes in upstream net_unittest.
For URLRequestTest.CookiePolicy_ForceSession, the force_session cookie test case
is removed from URLRequestTest.

BUG=
TEST=net_unittests_apk test pass


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

[Shouqun Liu, 2012-07-14 08:20:07]

Hi, the patch removes two URLRequestTest cases from disable list. Maybe next
time I should wrap all the clean case in a single patch:-)
FileDirRedirectNoCrash gets pass in upstream code with the existence of
directory "/data/local/tmp/net/data/url_request_unittest" during test.
CookiePolicy_ForceSession is a deprecated case and removed.

For remaining disabled URLRequestTest cases, they need to run  the test server
on the remote host. While this is not ready in upstream, is it ready in
downstream? or maybe I can look into it.

And I do not have CQ try server access, can you help run "android_test" for this
issue?  Thanks!

[Shouqun Liu, 2012-07-14 08:20:08]

Hi, the patch removes two URLRequestTest cases from disable list. Maybe next
time I should wrap all the clean case in a single patch:-)
FileDirRedirectNoCrash gets pass in upstream code with the existence of
directory "/data/local/tmp/net/data/url_request_unittest" during test.
CookiePolicy_ForceSession is a deprecated case and removed.

For remaining disabled URLRequestTest cases, they need to run  the test server
on the remote host. While this is not ready in upstream, is it ready in
downstream? or maybe I can look into it.

And I do not have CQ try server access, can you help run "android_test" for this
issue?  Thanks!

[nilesh, 2012-07-16 20:41:48]

On 2012/07/14 08:20:08, Shouqun Liu wrote:
> Hi, the patch removes two URLRequestTest cases from disable list. Maybe next
> time I should wrap all the clean case in a single patch:-)
> FileDirRedirectNoCrash gets pass in upstream code with the existence of
> directory "/data/local/tmp/net/data/url_request_unittest" during test.
> CookiePolicy_ForceSession is a deprecated case and removed.
> 
> For remaining disabled URLRequestTest cases, they need to run  the test server
> on the remote host. While this is not ready in upstream, is it ready in
> downstream? or maybe I can look into it.
> 
> And I do not have CQ try server access, can you help run "android_test" for
this
> issue?  Thanks!

I will try the change for you.

[nilesh, 2012-07-16 21:35:56]

On 2012/07/16 20:41:48, nileshagrawal1 wrote:
> On 2012/07/14 08:20:08, Shouqun Liu wrote:
> > Hi, the patch removes two URLRequestTest cases from disable list. Maybe next
> > time I should wrap all the clean case in a single patch:-)
> > FileDirRedirectNoCrash gets pass in upstream code with the existence of
> > directory "/data/local/tmp/net/data/url_request_unittest" during test.
> > CookiePolicy_ForceSession is a deprecated case and removed.
> > 
> > For remaining disabled URLRequestTest cases, they need to run  the test
server
> > on the remote host. While this is not ready in upstream, is it ready in
> > downstream? or maybe I can look into it.
> > 
> > And I do not have CQ try server access, can you help run "android_test" for
> this
> > issue?  Thanks!
> 
> I will try the change for you.

Patch CL: http://codereview.chromium.org/10793018/

[nilesh, 2012-07-17 00:18:12]

On 2012/07/16 21:35:56, nileshagrawal1 wrote:
> On 2012/07/16 20:41:48, nileshagrawal1 wrote:
> > On 2012/07/14 08:20:08, Shouqun Liu wrote:
> > > Hi, the patch removes two URLRequestTest cases from disable list. Maybe
next
> > > time I should wrap all the clean case in a single patch:-)
> > > FileDirRedirectNoCrash gets pass in upstream code with the existence of
> > > directory "/data/local/tmp/net/data/url_request_unittest" during test.
> > > CookiePolicy_ForceSession is a deprecated case and removed.
> > > 
> > > For remaining disabled URLRequestTest cases, they need to run  the test
> server
> > > on the remote host. While this is not ready in upstream, is it ready in
> > > downstream? or maybe I can look into it.
> > > 
> > > And I do not have CQ try server access, can you help run "android_test"
for
> > this
> > > issue?  Thanks!
> > 
> > I will try the change for you.
> 
> Patch CL: http://codereview.chromium.org/10793018/

LGTM

[nilesh, 2012-07-17 00:18:39]

On 2012/07/17 00:18:12, nileshagrawal1 wrote:
> On 2012/07/16 21:35:56, nileshagrawal1 wrote:
> > On 2012/07/16 20:41:48, nileshagrawal1 wrote:
> > > On 2012/07/14 08:20:08, Shouqun Liu wrote:
> > > > Hi, the patch removes two URLRequestTest cases from disable list. Maybe
> next
> > > > time I should wrap all the clean case in a single patch:-)
> > > > FileDirRedirectNoCrash gets pass in upstream code with the existence of
> > > > directory "/data/local/tmp/net/data/url_request_unittest" during test.
> > > > CookiePolicy_ForceSession is a deprecated case and removed.
> > > > 
> > > > For remaining disabled URLRequestTest cases, they need to run  the test
> > server
> > > > on the remote host. While this is not ready in upstream, is it ready in
> > > > downstream? or maybe I can look into it.
> > > > 
> > > > And I do not have CQ try server access, can you help run "android_test"
> for
> > > this
> > > > issue?  Thanks!
> > > 
> > > I will try the change for you.
> > 
> > Patch CL: http://codereview.chromium.org/10793018/
> 
> LGTM

jrg, can you please give OWNERS approval.

[John Grabowski, 2012-07-17 01:17:26]

OWNERS LGTM

[commit-bot: I haz the power, 2012-07-17 01:27:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/shouqun.liu@intel.com/10781009/1

[commit-bot: I haz the power, 2012-07-17 03:31:43]

Change committed as 146951