Deprecate debug parameter from app.make_hander()

[asvetlov]

Use app.debug as a value.
If app.debug conflicts with explicitly passes debug param -- raise ValueError.

The change is backward incompatible a little but it worth to be applied.

[f0t0n]

And what about the same story for loop parameter?
Currently we pass application's loop to the RequestHandlerFactory's constructor:

self._handler_factory(self, self.router, debug=self.debug, loop=self.loop, **kwargs)

But user reads in the documentation:

    kwargs –
            additional parameters for RequestHandlerFactory
            constructor.

So user could decide to pass a loop within kwargs that will cause a TypeError: got multiple values for keyword arg....

So my idea is, if we always pass application's loop as a parameter to the RequestHandlerFactory's constructor we should not allow user to do pass own loop as we did with debug.

Makes sense?

But the code of the make_handler will become ugly with all those checks for different parameters... :/

[asvetlov]

TypeError: got multiple values is clean and obvious to me.
Let's keep it as is.

[asvetlov]

Fixed by #1123

[lock]

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.