Closed
Bug 240340
Opened 21 years ago
Closed 21 years ago
Key events broken in plugins on MacOSX
Categories
(Core Graveyard :: Plug-ins, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.7final
People
(Reporter: jst, Assigned: jst)
References
()
Details
(Keywords: fixed1.7, regression)
Attachments
(1 file)
13.31 KB,
patch
|
bryner
:
review+
bryner
:
superreview+
asa
:
approval1.7+
|
Details | Diff | Splinter Review |
Bug 233142 regressed key events (among other events, see bug 238449) in plugins,
especially on Mac OSX. The old assumption that returning an error code from
event code prevented the events default action is no longer true, most of the
plugin frame code played nice, but not all of it.
Patch coming up.
Assignee | ||
Comment 1•21 years ago
|
||
Assignee | ||
Updated•21 years ago
|
Attachment #145956 -
Flags: superreview?(bryner)
Attachment #145956 -
Flags: review?(bryner)
Assignee | ||
Updated•21 years ago
|
Status: NEW → ASSIGNED
Flags: blocking1.7?
Target Milestone: --- → mozilla1.7final
Updated•21 years ago
|
Attachment #145956 -
Flags: superreview?(bryner)
Attachment #145956 -
Flags: superreview+
Attachment #145956 -
Flags: review?(bryner)
Attachment #145956 -
Flags: review+
Assignee | ||
Comment 2•21 years ago
|
||
Comment on attachment 145956 [details] [diff] [review]
PreventDefault() where needed, and no more bogus error returns
Requesting approval for this plugin event regression.
Attachment #145956 -
Flags: approval1.7?
Assignee | ||
Comment 3•21 years ago
|
||
Fix checked in on the trunk.
Comment 4•21 years ago
|
||
Comment on attachment 145956 [details] [diff] [review]
PreventDefault() where needed, and no more bogus error returns
a=asa (on behalf of drivers) for checkin to 1.7
Attachment #145956 -
Flags: approval1.7? → approval1.7+
Assignee | ||
Comment 5•21 years ago
|
||
Fixed on the 1.7 branch now too!
Updated•21 years ago
|
Flags: blocking1.7?
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•