Clean up _getFileExtension() method and fix all its callers
fixed unnecessary escaping
+fixes
getLastIndexOf > lastIndexOf
- unnecessary var decl
Merge branch 'master' into jeff/pr4846
Final changes to code review needed to get this into sprint 31
Split menu specs into HTML and Native.  Add HTML and Native versions of createTestWindowAndRun.
Set brackets.nativeMenus based on hasNativeMenus URL parameter.
Performance improvements for Find, focused on the initial keystroke or two
when the number of results is usually very high:
- Batch tickmark creation (saves 120ms in med-size files like FindReplace)
- Cut off tickmark & inline highlighting when tons of search results found
(saves ~1 sec in huge files like codemirror.js)
clear highlights and Find selection color when modal bar closes
API changes
only need to handle closeOk and close events
Amend PR #5311 with an comment and issue ref.

As per discussion in PR #5311, related to issue #5310.
When switching documents from QuickOpen, pop the modal bar out before switching and animate after
code cleanup
hasNativeMenus param code cleanup
Code review: use more consts in Find; remove unneeded single-tickmark API
Update to decompress-zip that includes fix for our issue.
Merge pull request #5290 from lkcampbell/menu-unit-test

Reactivate HTML Menu unit tests as part of the test suite