Code review comment for lp:~robertcarr/mir/demo-shell

Fixed licenses, removed demo_shell_placement_strategy.h, refactored returns.

Tab key only works as root due to not having permissions on /dev/input by default. So expected I guess.

I agree we should stick to one demo shell. This one is a little verbose, it was intended in a way that would be easiest to grow (have a few more things to play around with perhaps). It's intended to use the interfaces in a way Unity might, rather than attempt to express the features as small as possible (i.e. one lambda in default server configuration).

The stub/null shell is DefaultServerConfiguration I think.