Closed Bug 714602 Opened 12 years ago Closed 12 years ago

Use defaultPrevented instead of the deprecated getPreventDefault()

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Firefox 12

People

(Reporter: dao, Assigned: mfinkle)

References

Details

(Whiteboard: not-fennec-11)

Attachments

(1 file)

patch
1.05 KB, patch
mbrubeck
: review+
Details | Diff | Splinter Review 
+++ This bug was initially created as a clone of Bug #707677 +++

event.getPreventDefault is deprecated, we should use event.defaultPrevented instead.

http://mxr.mozilla.org/mozilla-central/search?string=getPreventDefault&case=1&find=mobile/android/&filter=\.%28js|xml|xul|html%29&hitlimit=&tree=mozilla-central
Attached patch patch — — Splinter Review 
Simple switch. Context menus appear, so the change didn't break anything.
Assignee: nobody → mark.finkle
Attachment #585325 - Flags: review?(mbrubeck)
Attachment #585325 - Flags: review?(mbrubeck) → review+
https://hg.mozilla.org/mozilla-central/rev/20ec67ea2f4a
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Finkle, please request aurora approval
Whiteboard: not-fennec-11
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.