Disable unittests that depends on OpenSSL when use_openssl==0

Disable unittests that depends on OpenSSL when use_openssl==0

These unittests include headers that includes openssl/ssl.h
(sm_connection.h). For platforms that don't have access to
OpenSSL this won't work.

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

[yhirano, 2013-09-09 11:11:06]

https://codereview.chromium.org/23724020/diff/1/net/net.gyp
File net/net.gyp (right):

https://codereview.chromium.org/23724020/diff/1/net/net.gyp#newcode2003
net/net.gyp:2003: 'tools/flip_server/http_interface_test.cc',
I think you can exclude all files in tools/flip_server using ['exclude',
'^tools/flip_server'].

[robertn, 2013-09-09 12:04:05]

On 2013/09/09 11:11:06, yhirano wrote:
> https://codereview.chromium.org/23724020/diff/1/net/net.gyp
> File net/net.gyp (right):
> 
> https://codereview.chromium.org/23724020/diff/1/net/net.gyp#newcode2003
> net/net.gyp:2003: 'tools/flip_server/http_interface_test.cc',
> I think you can exclude all files in tools/flip_server using ['exclude',
> '^tools/flip_server'].

Fixed in patch set 2.

[yhirano, 2013-09-09 12:45:42]

lgtm

[robertn, 2013-09-09 13:32:57]

On 2013/09/09 12:45:42, yhirano wrote:
> lgtm

Thanks, do I just check the commit-checkbox now to get it merged? (First time
submitting a patch.)

[commit-bot: I haz the power, 2013-09-09 13:34:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/robertn@opera.com/23724020/5001

[yhirano, 2013-09-09 13:40:39]

> do I just check the commit-checkbox now to get it merged? (First time
submitting a patch.)
You have to get an OWNER's lgtm.
I am not an OWNER of the file, but agl is.
https://code.google.com/p/chromium/codesearch#chromium/src/net/OWNERS&q=net/O...

[robertn, 2013-09-09 13:42:23]

On 2013/09/09 13:40:39, yhirano wrote:
> > do I just check the commit-checkbox now to get it merged? (First time
> submitting a patch.)
> You have to get an OWNER's lgtm.
> I am not an OWNER of the file, but agl is.
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/OWNERS&q=net/O...

Ah I see, thank you.

[commit-bot: I haz the power, 2013-09-09 13:48:19]

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

[robertn, 2013-09-11 08:20:30]

@agl: Does this look good to you?

[robertn, 2013-09-18 13:16:51]

On 2013/09/11 08:20:30, robertn wrote:
> @agl: Does this look good to you?

Ping

[agl, 2013-09-18 13:34:04]

lgtm

[commit-bot: I haz the power, 2013-09-18 13:34:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/robertn@opera.com/23724020/5001

[commit-bot: I haz the power, 2013-09-18 17:19:07]

Change committed as 223893