I think it's not bad to have it checked by default but a bit more intelligent it surely can be. The biggest annoyance and source of confusion is when you make a merge request based on a protected branch. The checkbox is still there and checked but it won't actually delete the protected branch (well, that's good ;-)).

In that case the checkbox should just be not displayed at all.