Use setTimeout instead of setImmediate in MenuManager

This should fix the flaky test failures that have been occurring on windows VSTS builds
This commit is contained in:
Max Brunsfeld
2018-08-10 12:32:26 -07:00
parent 87784989ad
commit 6d54578daa
2 changed files with 15 additions and 22 deletions

View File

@@ -149,9 +149,9 @@ class MenuManager
update: ->
return unless @initialized
clearImmediate(@pendingUpdateOperation) if @pendingUpdateOperation?
clearTimeout(@pendingUpdateOperation) if @pendingUpdateOperation?
@pendingUpdateOperation = setImmediate =>
@pendingUpdateOperation = setTimeout(=>
unsetKeystrokes = new Set
for binding in @keymapManager.getKeyBindings()
if binding.command is 'unset!'
@@ -168,6 +168,7 @@ class MenuManager
keystrokesByCommand[binding.command].unshift binding.keystrokes
@sendToBrowserProcess(@template, keystrokesByCommand)
, 1)
loadPlatformItems: ->
if platformMenu?