Use ssl._create_unverified_context() only if available

Use ssl._create_unverified_context() only if available

It's only available Python 2.7.9 or later.

BUG=#66

[bashi, 2016-03-22 03:00:15]

PTAL

[nednguyen, 2016-03-22 03:49:34]

https://codereview.appspot.com/289570043/diff/1/httpclient.py
File httpclient.py (right):

https://codereview.appspot.com/289570043/diff/1/httpclient.py#newcode190
httpclient.py:190: try:
Instead of try except, how about:
if hasattr(ssl, "_create_unverified_context"):
    ...
else:

[bashi, 2016-03-22 03:53:08]

Thanks for review!

https://codereview.appspot.com/289570043/diff/1/httpclient.py
File httpclient.py (right):

https://codereview.appspot.com/289570043/diff/1/httpclient.py#newcode190
httpclient.py:190: try:
On 2016/03/22 03:49:34, nednguyen wrote:
> Instead of try except, how about:
> if hasattr(ssl, "_create_unverified_context"):
>     ...
> else:

Done.

[nednguyen, 2016-03-22 03:54:27]

On 2016/03/22 03:53:08, bashi wrote:
> Thanks for review!
> 
> https://codereview.appspot.com/289570043/diff/1/httpclient.py
> File httpclient.py (right):
> 
> https://codereview.appspot.com/289570043/diff/1/httpclient.py#newcode190
> httpclient.py:190: try:
> On 2016/03/22 03:49:34, nednguyen wrote:
> > Instead of try except, how about:
> > if hasattr(ssl, "_create_unverified_context"):
> >     ...
> > else:
> 
> Done.

lgtm

After I helped with commiting this patch, can you manually update the
web-page-replay version in catapult/telemetry/third_party as well?

[bashi, 2016-03-22 03:56:22]

On 2016/03/22 03:54:27, nednguyen wrote:
> On 2016/03/22 03:53:08, bashi wrote:
> > Thanks for review!
> > 
> > https://codereview.appspot.com/289570043/diff/1/httpclient.py
> > File httpclient.py (right):
> > 
> > https://codereview.appspot.com/289570043/diff/1/httpclient.py#newcode190
> > httpclient.py:190: try:
> > On 2016/03/22 03:49:34, nednguyen wrote:
> > > Instead of try except, how about:
> > > if hasattr(ssl, "_create_unverified_context"):
> > >     ...
> > > else:
> > 
> > Done.
> 
> lgtm
> 
> After I helped with commiting this patch, can you manually update the
> web-page-replay version in catapult/telemetry/third_party as well?

Sure. Thanks for the help!

[nednguyen, 2016-03-22 04:01:08]

On 2016/03/22 03:56:22, bashi wrote:
> On 2016/03/22 03:54:27, nednguyen wrote:
> > On 2016/03/22 03:53:08, bashi wrote:
> > > Thanks for review!
> > > 
> > > https://codereview.appspot.com/289570043/diff/1/httpclient.py
> > > File httpclient.py (right):
> > > 
> > > https://codereview.appspot.com/289570043/diff/1/httpclient.py#newcode190
> > > httpclient.py:190: try:
> > > On 2016/03/22 03:49:34, nednguyen wrote:
> > > > Instead of try except, how about:
> > > > if hasattr(ssl, "_create_unverified_context"):
> > > >     ...
> > > > else:
> > > 
> > > Done.
> > 
> > lgtm
> > 
> > After I helped with commiting this patch, can you manually update the
> > web-page-replay version in catapult/telemetry/third_party as well?
> 
> Sure. Thanks for the help!

Patch landed in
https://github.com/chromium/web-page-replay/commit/b03f84da6e90951d4275b56755...

[bashi, 2016-03-22 04:19:23]

On 2016/03/22 04:01:08, nednguyen wrote:
> On 2016/03/22 03:56:22, bashi wrote:
> > On 2016/03/22 03:54:27, nednguyen wrote:
> > > On 2016/03/22 03:53:08, bashi wrote:
> > > > Thanks for review!
> > > > 
> > > > https://codereview.appspot.com/289570043/diff/1/httpclient.py
> > > > File httpclient.py (right):
> > > > 
> > > > https://codereview.appspot.com/289570043/diff/1/httpclient.py#newcode190
> > > > httpclient.py:190: try:
> > > > On 2016/03/22 03:49:34, nednguyen wrote:
> > > > > Instead of try except, how about:
> > > > > if hasattr(ssl, "_create_unverified_context"):
> > > > >     ...
> > > > > else:
> > > > 
> > > > Done.
> > > 
> > > lgtm
> > > 
> > > After I helped with commiting this patch, can you manually update the
> > > web-page-replay version in catapult/telemetry/third_party as well?
> > 
> > Sure. Thanks for the help!
> 
> Patch landed in
>
https://github.com/chromium/web-page-replay/commit/b03f84da6e90951d4275b56755...

Thanks. Prepared roll:
https://codereview.chromium.org/1821953002/