Enable net_unittests that rely on device time settings on Android.

Enable net_unittests that rely on device time settings on Android.

Since http://crbug.com/164180 is now fixed, we can enable ~10
unittests that depend on the device time settings and were
failing exclusively on bots.

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

[ppi, 2013-04-18 16:02:58]

Could you guys have a quick look? Should be easy to review.

[Philippe, 2013-04-18 16:13:37]

On 2013/04/18 16:02:58, ppi wrote:
> Could you guys have a quick look? Should be easy to review.

lgtm, thanks Przemek!

[ppi, 2013-04-18 16:14:35]

On 2013/04/18 16:13:37, Philippe wrote:
> On 2013/04/18 16:02:58, ppi wrote:
> > Could you guys have a quick look? Should be easy to review.
> 
> lgtm, thanks Przemek!

Thanks a lot!

[commit-bot: I haz the power, 2013-04-18 16:14:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ppi@chromium.org/13947040/4002

[digit1, 2013-04-18 16:16:05]

lgtm. that's great :)

[wtc, 2013-04-18 17:26:39]

Patch set 2 LGTM. I have two questions.

https://codereview.chromium.org/13947040/diff/4002/build/android/pylib/gtest/...
File build/android/pylib/gtest/filter/net_unittests_disabled (right):

https://codereview.chromium.org/13947040/diff/4002/build/android/pylib/gtest/...
build/android/pylib/gtest/filter/net_unittests_disabled:4:
MultiThreadedCertVerifierTest.CancelRequest

Should we list this test in alphabetical order? The
original code did that.

Or do you consider this test as a separate category of
suppressions?

https://codereview.chromium.org/13947040/diff/4002/build/android/pylib/gtest/...
build/android/pylib/gtest/filter/net_unittests_disabled:6:
CertVerifyProcTest.InvalidKeyUsage

I guess this test stilll fails, but not just on bots, right?

[ppi, 2013-04-18 18:07:58]

https://codereview.chromium.org/13947040/diff/4002/build/android/pylib/gtest/...
File build/android/pylib/gtest/filter/net_unittests_disabled (right):

https://codereview.chromium.org/13947040/diff/4002/build/android/pylib/gtest/...
build/android/pylib/gtest/filter/net_unittests_disabled:4:
MultiThreadedCertVerifierTest.CancelRequest
On 2013/04/18 17:26:39, wtc wrote:
> 
> Should we list this test in alphabetical order? The
> original code did that.
> 
> Or do you consider this test as a separate category of
> suppressions?

Apologies, you are correct, this change should not happen. I will fix that in
the next change.

https://codereview.chromium.org/13947040/diff/4002/build/android/pylib/gtest/...
build/android/pylib/gtest/filter/net_unittests_disabled:6:
CertVerifyProcTest.InvalidKeyUsage
On 2013/04/18 17:26:39, wtc wrote:
> 
> I guess this test stilll fails, but not just on bots, right?

Actually now it crashes (everywhere) - I've filed a bug: http://crbug.com/233150
.

[commit-bot: I haz the power, 2013-04-18 18:39:28]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[ppi, 2013-04-19 11:59:48]

https://codereview.chromium.org/13947040/diff/4002/build/android/pylib/gtest/...
File build/android/pylib/gtest/filter/net_unittests_disabled (right):

https://codereview.chromium.org/13947040/diff/4002/build/android/pylib/gtest/...
build/android/pylib/gtest/filter/net_unittests_disabled:4:
MultiThreadedCertVerifierTest.CancelRequest
On 2013/04/18 18:07:58, ppi wrote:
> On 2013/04/18 17:26:39, wtc wrote:
> > 
> > Should we list this test in alphabetical order? The
> > original code did that.
> > 
> > Or do you consider this test as a separate category of
> > suppressions?
> 
> Apologies, you are correct, this change should not happen. I will fix that in
> the next change.

Fixed in patch set 3, thanks!

[commit-bot: I haz the power, 2013-04-19 12:00:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ppi@chromium.org/13947040/33001

[commit-bot: I haz the power, 2013-04-19 13:36:53]

Retried try job too often on win7_aura for step(s) content_browsertests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-04-19 17:02:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ppi@chromium.org/13947040/33001

[wtc, 2013-04-19 17:23:19]

Patch set 3 LGTM. Thanks!

[commit-bot: I haz the power, 2013-04-19 17:48:50]

Change committed as 195225