If [ -n ] evaluates to true for some reason, which will happen if CFX_FIREFOX is undefined.
if [ -n "$CFX_FIREFOX" ]; should work