Closed
Bug 791222
Opened 13 years ago
Closed 13 years ago
Do the event.target check for pageShowEventHandlers before setting a timeout.
Categories
(SeaMonkey :: General, defect)
SeaMonkey
General
Tracking
(seamonkey2.15 fixed)
RESOLVED
FIXED
seamonkey2.15
| Tracking | Status | |
|---|---|---|
| seamonkey2.15 | --- | fixed |
People
(Reporter: philip.chee, Assigned: philip.chee)
Details
Attachments
(1 file)
|
2.21 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
Bug 780594 Comment 168:
> See bug 784792 comment 12 and bug 786940 comment 3.
Bug 786940 comment 3:
> pageShowEventHandlers runs off a timeout in response to all pageshow events. It's a
> no-op when the pageshow event is for a subframe. We could avoid setting the timeout
> entirely in that case.
|
||
Comment 2•13 years ago
|
||
Comment on attachment 661195 [details] [diff] [review]
Patch v1.0 Proposed fix.
> function pageShowEventHandlers(event)
> {
>+ checkForDirectoryListing();
> }
Not much left to do here is there ;-)
(Historically, this also used to:
1. Update bookmarks as being visited. Places does this I guess.
2. Copy search results to the search sidebar.
3. Copy the URL to a native widget...)
Attachment #661195 -
Flags: review?(neil) → review+
|
Assignee | |
Comment 3•13 years ago
|
||
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/c74b306806ae
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
status-seamonkey2.15:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.15
You need to log in
before you can comment on or make changes to this bug.
Description
•