Refactor method _keyDownHandler in frost-select-dropdown.js

[juwara0]

Problem Description

There are multiple return statements exiting different pathways from this method:

ember-frost-core/addon/components/frost-select-dropdown.js

Lines 513 to 534 in 38d74e0

	this._keyDownHandler = (e) => {
	if (this.isDestroyed || this.isDestroying) return
	if ([DOWN_ARROW, UP_ARROW].indexOf(e.keyCode) !== -1) {
	e.preventDefault() // Keep arrow keys from scrolling document
	this._handleArrowKey(e.keyCode === UP_ARROW)
	}
	switch (e.keyCode) {
	case ENTER:
	this._handleEnterKey()
	return
	case ESCAPE:
	this.get('onClose')()
	return
	case TAB:
	this.get('onClose')()
	return // eslint-disable-line no-useless-return
	}
	}

Solution

Investigate refactoring this method, perhaps set a flag and check it instead, so that there are not many different exit paths.