This reverts commit f4a5a9ea, reversing
changes made to 7ccb4821.

Reason:

The logic is different, the first call to #option_value_selected? is for
the :selected option (the argument is the "selected" variable), the second
call is for the :disabled option (the argument is the "disabled" variable).

Closes #12831

Revert "Used Yield instead of block.call" -- this causes all of atom_feed_helper_test.rb to fail with "SystemStackError: stack level too deep".

This reverts commit d3a1ce1c.

Builder's #tag! takes either String or Symbol as the first parameter

This is the only argument that changes over the method calls so it is
better to it be the first one

Revert "drop one more string allocation"

This reverts commit 4d15661d.

rdoc: favicon source shouldn't begin with a slash to reference asset pipeline resource [ci skip]
Conflicts:
	actionview/lib/action_view/helpers/asset_tag_helper.rb

Same as 4d4ff531

Rails 3.2 API allowed arbitrary input for cdata_section;
this change re-introduces the old behaviour.

Avoid unnecessary catching of Exception instead of StandardError (converting Exceptions into StandardErrors)

add a new local variable to track if digests are being stored, to ensure the cleanup works correctly

ActionView::Resolver.caching = false.

If config.action_view.cache_template_loading = false, most likely in a development
configuration if config.cache_classes = false &
config.action_controller.perform_caching = true.

config.action_view.cache_template_loading defaults to the value of config.cache_classes