Bug 56477

Summary: WebKit2: Sometimes Command-[ opens a new tab and loads the back page in it
Product: WebKit Reporter: Alexey Proskuryakov <ap>
Component: WebKit2Assignee: Alexey Proskuryakov <ap>
Severity: Normal CC: darin
Priority: P1 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.6   
Description Flags
proposed fix mitz: review+

Description Alexey Proskuryakov 2011-03-16 12:21:00 PDT
Sometimes pressing Command-[ opens a new tab and navigates to the last item in the current tab’s back list in that new tab.

Comment 1 Alexey Proskuryakov 2011-03-16 12:26:28 PDT
Created attachment 85955 [details]
proposed fix
Comment 2 Alexey Proskuryakov 2011-03-16 12:27:22 PDT
As far as I can tell, there is no need to reset the current event back (or to nil).
Comment 3 mitz 2011-03-16 12:40:03 PDT
Comment on attachment 85955 [details]
proposed fix

Can we use the NSApp global instead of messaging NSApplication twice?
Comment 4 Alexey Proskuryakov 2011-03-16 12:49:54 PDT
I guess we can.

Committed <http://trac.webkit.org/changeset/81269>.
Comment 5 Darin Adler 2011-03-16 13:13:06 PDT
Don’t we want to restore the old current event on return?