I know that we don't RTBC our own patches, but I'm hoping this one might be an exception since I don't think there are many other core devs who have an IIS environment set up. If @David_Rothstein (who seems to be committing 7.x patches this week, which is why I'm taking this action) or anyone else feels I'm wrong for doing so, then set it back to Needs Review. If there is someone other than myself who has an IIS dev environment, please take a minute to apply it. It is pretty trivial to test.

This is the same basic change that was applied to 8.x. I can confirm that before applying the patch misc/favicon.ico could not be accessed. After applying the patch the favicon could be accessed normally, appeared on browser tabs, and appeared in browser bookmarks.

It looked safe to commit given the above testing and the fact that it's very similar to code that's already in that file. Although I'm curious why <conditions> was placed after <action> in this case, rather than before it like the others. If there's any chance that could cause functional issues in some particular setup, someone please let me know :) But I assume it's fine, and just a stylistic thing.

Thanks for the additional testing! This is a recently-committed patch, so please leave this at just "fixed" so it stays visible in the issue queue for people who might need to find it. (It will be automatically closed after a while once there are no more comments.)