Pass name to BaseEventLoop task factory

The task factory is used in the BaseEventLoop create_task method when
set. This commit passes the `name` kwarg to this factory to set the
name of the task, instead of setting the task in the loop method. This
is more consistent with the eager_task_factory methods.

Author: ordinary-jamie

Doc/library/asyncio-eventloop.rst

@@ -379,8 +379,8 @@ Creating Futures and Tasks
 
    If *factory* is ``None`` the default task factory will be set.
    Otherwise, *factory* must be a *callable* with the signature matching
-   ``(loop, coro, context=None)``, where *loop* is a reference to the active
-   event loop, and *coro* is a coroutine object.  The callable
+   ``(loop, coro, name=None, context=None)``, where *loop* is a reference
+   to the active event loop, and *coro* is a coroutine object. The callable
    must return a :class:`asyncio.Future`-compatible object.
 
 .. method:: loop.get_task_factory()

Lib/asyncio/base_events.py

@@ -461,11 +461,9 @@ def create_task(self, coro, *, name=None, context=None):
         else:
             if context is None:
                 # Use legacy API if context is not needed
-                task = self._task_factory(self, coro)
+                task = self._task_factory(self, coro, name=name)
             else:
-                task = self._task_factory(self, coro, context=context)
-
-            task.set_name(name)
+                task = self._task_factory(self, coro, name=name, context=context)
 
         return task
 

Lib/test/test_asyncio/test_base_events.py

@@ -722,7 +722,7 @@ class MyTask(asyncio.Task):
         async def coro():
             pass
 
-        factory = lambda loop, coro: MyTask(coro, loop=loop)
+        factory = lambda loop, coro, **kwargs: MyTask(coro, loop=loop, **kwargs)
 
         self.assertIsNone(self.loop.get_task_factory())
         self.loop.set_task_factory(factory)
@@ -816,8 +816,8 @@ async def test():
             loop.close()
 
     def test_create_named_task_with_custom_factory(self):
-        def task_factory(loop, coro):
-            return asyncio.Task(coro, loop=loop)
+        def task_factory(loop, coro, **kwargs):
+            return asyncio.Task(coro, loop=loop, **kwargs)
 
         async def test():
             pass

Lib/test/test_asyncio/test_tasks.py

@@ -3326,7 +3326,7 @@ def test_run_coroutine_threadsafe_task_factory_exception(self):
         """Test coroutine submission from a thread to an event loop
         when the task factory raise an exception."""
 
-        def task_factory(loop, coro):
+        def task_factory(loop, coro, **kwargs):
             raise NameError
 
         run = self.loop.run_in_executor(