testing: Fix incorrect docs for gen_test

Pointed out in #1154

Merge pull request #2083 from bdarnell/log-cleanup

Clean up test logging

test: Make log detection sensitive to info logs

Silence one in httpserver_test

testing: Remove deprecated LogTrapTestCase

Move the tests that relied on this in concurrent_test.py to use
log.debug instead. (The redbot test has been broken and needs work
anyway.

Merge pull request #2082 from bdarnell/timeouterror

ioloop,gen: Unify TimeoutErrors

ioloop,gen: Unify TimeoutErrors

Fixes #1929

Merge pull request #2077 from vladimirbright/clarify_timeouts_in_queues

Added information about timeout arg in Queue get and put methods

Merge pull request #2074 from ploxiln/http1_connection_close

http1connection: add "Connection: close" header if appropriate

Merge pull request #2058 from jehiah/http_request_line_bad_request_2058

Return HTTP 400 on bad request lines

Added information about timeout arg in Queue get and put methods

httpserver: remove unused no_keep_alive attribute

still a parameter passed on to HTTP1Connection

http1connection: add "Connection: close" header if appropriate

When HTTP1Connection acts as the server, it closes the connection
after writing the response if the client includes the
"Connection: close" header in the request, or if the `no_keep_alive`
option is set to True in the constructor.

According to https://tools.ietf.org/html/rfc7230#section-6.6

> The server SHOULD send a "close" connection option in its final
response on that connection.

It was possible for an Application to set the Connection header
appropriately. But it is very helpful for tornado to take care of
this automatically, particularly if "close" was specified in a
request header.

Merge pull request #2072 from bdarnell/asyncio-default

ioloop: Make asyncio the default when available

ioloop: Make asyncio the default when available

In addition to changing the configurable default, add a special case
in IOLoop.current() so that the current IOLoop will be backed by
the main asyncio event loop.

Return HTTP 400 (Bad Request) on malformed requests

ioloop: Add tests for IOLoop configuration

Merge pull request #2063 from bdarnell/ioloop-instance

ioloop: Redefine instance() in terms of current()

Merge pull request #2064 from bdarnell/cloexec

tcpclient,netutil: Set FD_CLOEXEC on all sockets created by Tornado

tcpclient,netutil: Set FD_CLOEXEC on all sockets created by Tornado

PR #1984 was based on the mistaken belief that we were already
doing this (and in python 3.4+, it's true, thanks to PEP 446). This
fixes a regression introduced in Tornado 4.5 in which autoreload would
leak file descriptors and leave client connections hanging.

Fixes #2057

ioloop: Redefine instance() in terms of current()

This aligns us more closely with asyncio, which only has a single
get_event_loop() function. Most uses of instance() are simply a
holdover from before current() was introduced and are not actually
relying on its slightly different behavior, so we redefine it (and the
related methods install(), initialized(), and clear_instance()) in
terms of current().

Merge pull request #2062 from bdarnell/configure-pollioloop

ioloop: Make PollIOLoop separately configurable

ioloop: Make PollIOLoop separately configurable

This makes it possible to construct a PollIOLoop even when the default
IOLoop is configured to something else.

util: Enable multiple configurable bases in one hierarchy

This allows a mid-level class like PollIOLoop to have its own
configurability.

Merge pull request #2052 from bdarnell/deprecated

all: Remove deprecated io_loop arguments

Merge pull request #2028 from pitrou/close_exc_info

Avoid IOStream.close(exc_info=True)

concurrent: Remove use of self.io_loop from run_on_executor

all: Remove deprecated io_loop arguments

IOLoop.current() is now used in all cases; it is no longer possible to
pass IOLoops around and use them directly. This occasionally requires
awkward workarounds with IOLoop.run_sync, but mainly in test code
(and the synchronous HTTPClient).

testing: Remove deprecated get_unused_port()

This function never did a good job and has long since been deprecated
in favor of bind_unused_port.

Merge pull request #2002 from nvllsvm/master

Fetch absolute URL in AsyncHTTPTestCase

Merge pull request #2051 from bdarnell/appveyor-speedup

build: Speed up appveyor tests by not running the full matrix.

build: Speed up appveyor tests by not running the full matrix.

Only run partial tests in two of the four environments.

Merge pull request #2050 from bdarnell/appveyor-config

build: Don't double-test on appveyor

Merge pull request #2048 from bdarnell/option-redefinition

options: Report redefinition errors when underscores are used

Merge pull request #2045 from bdarnell/websocket-raise

websocket: Don't swallow exceptions in _write_frame

Merge pull request #2038 from schlamar/patch-2

don't check error code in test_set_close_exec

Merge pull request #2047 from bdarnell/assert-pid

ioloop: Document the incompatibility of IOLoop and fork()

Merge pull request #2046 from bdarnell/cookie-docs

web: Clarify cookie method docs

build: Don't double-test on appveyor

Pointed out in #2028

Test fetch full URL if applicable

Merge pull request #2024 from pjknkda/master

websocket: optimize C websocket_mask function

Merge pull request #2022 from coldnight/master

Fix CurlAsyncHTTPClient cause memory leak with `force_instance=True`

options: Report redefinition errors when underscores are used

Fixes #2020

websocket: Don't swallow exceptions in _write_frame

Swallowing the exception violated the method's interface (by returning
None instead of a Future), and differs from stream-closed behavior in
other contexts in Tornado.

Fixes #1980

ioloop: Document the incompatibility of IOLoop and fork()

Fixes #2004

web: Clarify cookie method docs

Be explicit that get_cookie cannot see the cookies set within the
current request, avoiding confusion as seen in #1993

Merge pull request #1979 from SuminAndrew/responses-py2-py34

web: Don't raise on unknown HTTP response codes

Bump master version to 5.0.dev1

don't check error code in test_set_close_exec

Avoid IOStream.close(exc_info=True)

close(exc_info=True) calls sys.exc_info() to get the "current" exception.
Unfortunately, on Python 2 this is the last raised exception even if it
was caught and silenced (by contrast with Python 3, which has lexically
nested exceptions).  This could set ``IOStream.error`` and therefore
``TCPClient.connect``'s raised exception to the wrong error.

Fix by passing the explicit error instance instead.

Merge pull request #2025 from chrisgavin/python3

Fix file upload demo on Python 2.

fix build failure in MSVC compiler

Fix file upload demo on Python 2.

websocket: optimize C mask extension

Fix CurlAsyncHTTPClient cause memory leak with `force_instance=True`

The CurlAsyncHTTPClient will cause memory leak when set `force_instance=True`,
because the `self._multi` and `self._force_timeout_callback` hold some methods
that belong the instance of `CurlAsyncHTTPClient`, it will cause circular reference.

Fix header for 4.5.1 release notes

Version 4.5.1

Merge pull request #2019 from bdarnell/colorama-fix

log: Fix color logging detection

log: Fix color logging detection

The previous logic would fail whenever curses is missing (which is
always true on windows, and is occasionally true on other platforms)
and colorama is installed but not initialized in this process (this is
a common occurrence for users of jupyter and ipython on windows).

Fixes #2013
Fixes #2015

Merge pull request #2018 from aetherson/master

fix backwards compatibility of url_concat for args=None

fix backwards compatibility of url_concat for args=None

Set version to 4.5

Merge pull request #2011 from bdarnell/websocket-render

websocket: Fix use of render_string in websocket handlers

websocket: Fix use of render_string in websocket handlers

PR #1917 caused websocket connections to be "finished" early, which
broke the use of render_string by setting self.ui to None.

Fixes #2010

Merge pull request #2009 from methane/travis

travis: fix using wrong Python version

travis: Skip "nodeps" build on pypy3

virtualenv (or pip, or setuptools) no longer supports py32, so this
won't work until travis has pypy3 3.5 available.

use new virtualenv

travis: fix using wrong Python version

Bump to 4.5b2, update release notes

Merge pull request #2006 from bdarnell/cleanups

Cleanups from various linters

Merge pull request #2005 from bdarnell/auth-facebook-expires

auth: Update, doc, and test the facebook session_expires field

docs: Add uncovered function

*: Fix pyflakes warnings

*: Run the new autopep8

Mainly adds a bunch of blank lines.

maint: Update requirements

auth: Update, doc, and test the facebook session_expires field

This was recently broken by the move from url-encoding to json.

Closes #2001

Fetch full URL if applicable

Merge pull request #1999 from bdarnell/tcpclient-test

tcpclient_test: Attempt to deflake on windows

websocket: Impose a size limit on incoming websocket messages (#1997)

tcpclient_test: Attempt to deflake on windows

It appears that on windows the client connect call can sometimes
succeed before the server has processed the request, so introduce
additional synchronization.

Merge branch 'branch4.4'

Version bump and release notes for 4.4.3

auth: Facebook now returns auth tokens in json instead of url-encoded

Fixes #1992

auth: Facebook now returns auth tokens in json instead of url-encoded

Fixes #1992

Merge pull request #1989 from bdarnell/appveyor36

appveyor: Build for python 3.5 and 3.6

do not raise exceptions on unknown status codes

appveyor: Build for python 3.5 and 3.6

Release notes and doc updates for 4.5

Merge pull request #1917 from retinator/master

Allow custom websocket upgrade response headers

Merge pull request #1988 from bdarnell/subprocess-test-detail

process_test: Report more details when test_sigchild_signal fails

Check in test script for circular references

From #1936

Merge pull request #1936 from homm/break-circrefs

Break circular reference to exc_info when exception is thrown

Merge pull request #1987 from bdarnell/iostream-orphan-future

Fix IOStream.write() to never orphan Future

Merge pull request #1927 from minrk/copy-list-in-close

copy handler list in IOLoop.close(all_fds=True)

Merge branch 'master' into master

process_test: Report more details when test_sigchild_signal fails

This test occasionally fails on pypy on travis-ci.

Merge pull request #1986 from bdarnell/yield-none-test

gen: Add tests for `yield None`

Fix IOStream.write() to never orphan Future

The current behaviour is error-prone and makes it difficult to use the
Future-returning variant of IOStream.write().  This change makes sure
the returned Future is triggered as soon as the corresponding write
is issued.

Merge pull request #1985 from bdarnell/websocket-convert-yielded

websocket: Avoid calling convert_yielded twice on the same object

Merge pull request #1864 from leynos/master

Allow specificion of trusted downstream proxies for xheaders

gen: Add tests for `yield None`

See #1716