Set tls_client_cert_optional in net/.../testserver.py.

Set tls_client_cert_optional in net/.../testserver.py.

Before this value wasn't set at all leading to crashes of the test server if tsls_client_auth was enabled.

BUG=NONE
TBR=rsleevi@chromium.org

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

[pneubeck (no reviews), 2014-07-21 08:44:50]

The CQ bit was checked by pneubeck@chromium.org

[commit-bot: I haz the power, 2014-07-21 08:45:16]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/pneubeck@chromium.org/407653005/1

[commit-bot: I haz the power, 2014-07-21 14:22:51]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_rel/buil...)

[pneubeck (no reviews), 2014-07-21 15:20:34]

The CQ bit was unchecked by pneubeck@chromium.org

[pneubeck (no reviews), 2014-07-21 15:22:45]

Committed patchset #1 manually as r284431 (presubmit successful).

[Ryan Sleevi, 2014-07-21 18:30:21]

Note, LGTM, but my comment on the original CL about splitting this out was so
that you could file a bug and actually document what the behaviour was.

Considering that we largely (weren't) getting crashes in testserver.py suggests
there's missing details here, which is why I was trying to follow up separately
:)

but thanks for committing.

[pneubeck (no reviews), 2014-07-21 18:39:23]

On 2014/07/21 18:30:21, Ryan Sleevi wrote:
> Note, LGTM, but my comment on the original CL about splitting this out was so
> that you could file a bug and actually document what the behaviour was.
> 
> Considering that we largely (weren't) getting crashes in testserver.py
suggests
> there's missing details here, which is why I was trying to follow up
separately
> :)
Note, that when I was running that SSLUITest.TestWSSClientCert locally on my
machine it failed exactly because of this missing property.
 
> but thanks for committing.

eieiei. Seems that I was overly optimistically interpreting your
 'testserver.py fix can/should have been its own CL, since it seems unrelated'
:-/
sorry.