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>
Status: RESOLVED FIXED    
Severity: Normal CC: darin
Priority: P1 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.6   
Attachments:
Description Flags
proposed fix mitz: review+

Alexey Proskuryakov
Reported 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. <rdar://problem/8806664>
Attachments
proposed fix (1.59 KB, patch)
2011-03-16 12:26 PDT, Alexey Proskuryakov 		mitz: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2011-03-16 12:26:28 PDT
Created attachment 85955 [details] proposed fix
Alexey Proskuryakov
Comment 2 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).
mitz
Comment 3 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?
Alexey Proskuryakov
Comment 4 2011-03-16 12:49:54 PDT
I guess we can. Committed <http://trac.webkit.org/changeset/81269>.
Darin Adler
Comment 5 2011-03-16 13:13:06 PDT
Don’t we want to restore the old current event on return?
Note You need to log in before you can comment on or make changes to this bug.