diff options
author | janekptacijarabaci <janekptacijarabaci@seznam.cz> | 2018-04-15 09:48:39 +0200 |
---|---|---|
committer | janekptacijarabaci <janekptacijarabaci@seznam.cz> | 2018-04-15 09:48:39 +0200 |
commit | 451cbfc0393ee0f3b8d18f856e6c8ad88d57b172 (patch) | |
tree | 0c62961b5b85901abf8883356562baee88fb63e3 /application/palemoon/base/content | |
parent | f2b4134845cb61831847e6746de2f15051d65a08 (diff) | |
download | UXP-451cbfc0393ee0f3b8d18f856e6c8ad88d57b172.tar UXP-451cbfc0393ee0f3b8d18f856e6c8ad88d57b172.tar.gz UXP-451cbfc0393ee0f3b8d18f856e6c8ad88d57b172.tar.lz UXP-451cbfc0393ee0f3b8d18f856e6c8ad88d57b172.tar.xz UXP-451cbfc0393ee0f3b8d18f856e6c8ad88d57b172.zip |
Bug 1100291 - Make getShortcutOrURIAndPostData() async by calling the passed callback off a zero timeout
Issue #112
Diffstat (limited to 'application/palemoon/base/content')
-rw-r--r-- | application/palemoon/base/content/browser.js | 6 | ||||
-rw-r--r-- | application/palemoon/base/content/urlbarBindings.xml | 12 |
2 files changed, 8 insertions, 10 deletions
diff --git a/application/palemoon/base/content/browser.js b/application/palemoon/base/content/browser.js index 154badae5..be7993961 100644 --- a/application/palemoon/base/content/browser.js +++ b/application/palemoon/base/content/browser.js @@ -1906,6 +1906,12 @@ function getShortcutOrURIAndPostData(aURL, aCallback) { let keyword = aURL; let param = ""; + // XXX Bug 1100294 will remove this little hack by using an async version of + // PlacesUtils.getURLAndPostDataForKeyword(). For now we simulate an async + // execution with at least a setTimeout(fn, 0). + let originalCallback = aCallback; + aCallback = data => setTimeout(() => originalCallback(data)); + let offset = aURL.indexOf(" "); if (offset > 0) { keyword = aURL.substr(0, offset); diff --git a/application/palemoon/base/content/urlbarBindings.xml b/application/palemoon/base/content/urlbarBindings.xml index 44b398304..ec9feb22d 100644 --- a/application/palemoon/base/content/urlbarBindings.xml +++ b/application/palemoon/base/content/urlbarBindings.xml @@ -324,18 +324,10 @@ gBrowser.selectedBrowser.focus(); let isMouseEvent = aTriggeringEvent instanceof MouseEvent; - let altEnter = !isMouseEvent && aTriggeringEvent && aTriggeringEvent.altKey; - - if (altEnter) { - // XXX This was added a long time ago, and I'm not sure why it is - // necessary. Alt+Enter's default action might cause a system beep, - // or something like that? - aTriggeringEvent.preventDefault(); - aTriggeringEvent.stopPropagation(); - } // If the current tab is empty, ignore Alt+Enter (just reuse this tab) - altEnter = altEnter && !isTabEmpty(gBrowser.selectedTab); + let altEnter = !isMouseEvent && aTriggeringEvent && + aTriggeringEvent.altKey && !isTabEmpty(gBrowser.selectedTab); if (isMouseEvent || altEnter) { // Use the standard UI link behaviors for clicks or Alt+Enter |