Remove Event.prototype.getPreventDefault

NEW
Unassigned

Status

()

Core
DOM: Events
6 years ago
9 months ago

People

(Reporter: David Bruant, Unassigned)

Tracking

(Depends on: 1 bug, {addon-compat, dev-doc-needed, site-compat})

Trunk
addon-compat, dev-doc-needed, site-compat
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
This is not standard and has a standard alternative (defaultPrevented property).
There is no reason to keep this around forever.

Comment 1

6 years ago
Need to first remove all its usage from Firefox / Gecko / Thunderbird / etc. then start
warn about its usage, and then remove it.
But yes,  we should try to remove it.
Looks like that wouldn't be too hard: <http://mxr.mozilla.org/comm-central/search?string=getPreventDefault&filter=\bgetPreventDefault>.

Updated

6 years ago
OS: Linux → All
Hardware: x86 → All
Version: 7 Branch → Trunk

Updated

6 years ago
Depends on: 707677
(Reporter)

Updated

5 years ago
Keywords: dev-doc-needed

Updated

5 years ago
Keywords: addon-compat

Updated

5 years ago
Depends on: 708702

Updated

5 years ago
Depends on: 708703

Updated

5 years ago
Depends on: 714601

Updated

5 years ago
Depends on: 714602

Updated

5 years ago
Depends on: 715110

Updated

5 years ago
Depends on: 726933

Updated

4 years ago
Keywords: site-compat

Comment 3

4 years ago
(In reply to Olli Pettay from comment #1)
> Need to first remove all its usage from Firefox / Gecko / Thunderbird / etc.
> then start warn about its usage, and then remove it.
> But yes,  we should try to remove it.

Looks like we started warning about it when there are still 11 in-tree callers...
Oops, I didn't realize the existence of C++ callers. Reverting for now.
Depends on: 876040
Depends on: 876054
It appears that wikipedia still uses this.

Updated

3 years ago
Depends on: 1024488
Posted the site compatibility doc: https://www.fxsitecompat.com/en-US/docs/2015/event-getpreventdefault-will-be-removed/

Comment 7

10 months ago
Are there any estimates on when getPreventDefault will finally be removed?
You need to log in before you can comment on or make changes to this bug.