move window frame module to be in `logical_plan`

I agree it would make more sense for this module to be in `logical_plan` (I think it would also be fine to do as a follow on PR)

__Originally posted by @alamb in https://github.com/apache/arrow-datafusion/pull/506#discussion_r646116862__