Bug 7085 - History->Back does not work while viewing PDF
Summary: History->Back does not work while viewing PDF
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: PDF (show other bugs)
Version: 420+
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: John Sullivan
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2006-02-05 09:47 PST by David Kilzer (:ddkilzer)
Modified: 2011-08-09 16:46 PDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2006-02-05 09:47:57 PST
Summary:

The History->Back menu choice doesn't work after viewing a PDF.  (Note that the "Back Arrow" button on Safari itself works, though.)

Steps to Reproduce: 

1. Open Safari+WebKit ToT.  (I used revision r12570.)
2. Open a PDF URL like this: http://developer.apple.com/hardware/ipod/ipodnotereader.pdf
3. Select "Back" from the "History" menu.

Expected Results: 

Safari should go back one page out of the PDF.

Actual Results: 

Safari does nothing.  You must click the "Back Arrow" button in Safari itself to make the page go back.

Regression: 

This is a regression.  This procedure works on Safari 2.0.3 (417.8) on 10.4.4.

Notes: 

None.
Comment 1 John Sullivan 2006-02-05 11:04:33 PST
Radar bug <rdar://problem/4403078> is about problems with PDFs vs. Back/Forward, but is not a regression.
Comment 2 Darin Adler 2006-02-05 12:53:04 PST
What's happening here is that the menu item says goBack: but does not specify a target. The PDFView is in the responder chain and it has a goBack: method.
Comment 3 David Kilzer (:ddkilzer) 2006-02-06 06:58:04 PST
Bug 7102 may be related to this bug as it involves viewing a PDF, then hitting the "Back" button.
Comment 4 John Sullivan 2006-02-20 10:04:51 PST
These issues are also in Radar as <rdar://problem/4347183>. The basic problem is that when the PDF view is focused, it steals "goBack:" from the menu and keyboard equivalent, and conversely "goBack:" from the toolbar is never sent to the PDF view. So the two ways to invoke "Back" each have different problems interacting with PDFs.

The only aspect of this that's a regression is that due to other fixes, the PDF view is (correctly) focused more often than it used to be, so stealing "goBack:" from the menu and keyboard equivalent is more common.
Comment 5 David Kilzer (:ddkilzer) 2007-04-29 13:23:19 PDT
ToT WebKit r21170 behaves the same as shipping Safari 2.0.4 (419.3) on Mac OS X 10.4.9 (8P135) now:

Works:
- Back button on browser
- Cmd-Left-Arrow
- Backspace

Does not work:
- History->Back menu item
- Cmd-[ (mapped to History->Back menu item)