Don't remove mappings when using a custom handler

When changing `g:ag_*handler` it would only make sense to remove the
quickfix/location list mappings if the new handler didn't use one of
those lists.

Otherwise removing the mappings is (imo) a separate decision from
writing your own handler, so it would be best not to mix the two.
Use the quickfix/location list height in `gv`

Before this was just being set to something similar to the default
height. Though this code is harder to read, it fixes the following:
- Close the right list (the location list or the quickfix window). This
  is fixed for the `e` and `q` commands too.
- If someone specifies a height (e.g. `let g:ag_qhandler="copen 50"`)
  then `gv` will be less jarring because when the quickfix window is
  re-opened it'll be 50 lines again.

The `gv`, `e`, and `q` mappings depended on a local variable that won't
be available when the mapping was used, so |:execute| was used for those
mappings to evaluate that variable when the mappings are made.

Fixes #11

Thanks to @sigmavirus24 for helping out with this one :)
Merge pull request #30 from joshuarh/gv-use-previous-window-height

Use the quickfix/location list height in `gv`
Merge pull request #31 from joshuarh/separate-apply-mappings-from-handlers

Don't remove mappings when using a custom handler
Enable mappings by default

Fixes https://github.com/rking/ag.vim/issues/36
Helptags should not be camel case
Merge pull request #38 from calid/Helptags-doc-fix

Helptags should not be camel case
remove mapping conflicting with colemak