move_on_ and fail_ functions should accept shield as a keyword argument

Support a simpler syntax for:
```
async with trio.move_on_after(1) as scope:
  scope.shield = True
```
as a one-liner:
```
async with trio.move_on_after(1, shield=True) as scope:
  ...
```