Mark ol.Observable.prototype.unByKey deprecated.

The ol.Observable.prototype.unByKey was supposed to handle removing of
listeners registered using `ol.Observable.prototype.on` and
`ol.Observable.prototype.once`.

However, the underlying google library code called by unByKey only requires
the key, not the `ol.Observable` object on which the registration was
triggered. Moreover, this method was unconvenient and in practice misused.
Example or real world code:
```javascript
var map = new ol.Map();
var key = map.on(listener);
view.unByKey(key);
```

A function was added in #2794, in particular to fix this kind of code.

This PR pushes forward the removal process of the old method in favor of the
unByKey function by marking it `@deprecated` and pointing to the right
usage.

Please merge when deemed appropriate.
I haven't checked how this shows up in the documentation, but I think it is overkill to mark it deprecated (especially if it shows up in an obnoxious way on every observable in the docs).  It was just over a month ago that @elemoine agreed (see https://github.com/openlayers/ol3/pull/2794#issuecomment-57938624).