Closed
Bug 1412463
Opened 7 years ago
Closed 7 months ago
Consider removing the chromeonly datatransfer.types.contains hack
Categories
(Core :: DOM: Core & HTML, enhancement, P3)
Tracking
()
RESOLVED
FIXED
119 Branch
| Tracking | Status | |
|---|---|---|
| firefox119 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: standard8)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
We added this in bug 1309970 to avoid breaking extensions, but at this point we should be able to remove it, I would think.
|
||
Updated•7 years ago
|
Priority: -- → P3
|
||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
|
||
Updated•2 years ago
|
Severity: normal → S3
|
Assignee | |
Comment 1•7 months ago
|
||
I believe we can do this now, I've looked at all the .types usages in mozilla-central and we don't seem to use .contains. The privileged extensions that we manage also don't seem to have any .types usage.
Try push: (oranges appear to be intermittents): https://treeherder.mozilla.org/jobs?repo=try&revision=da3647b26a02aa2a45e1d69a217fb96cad38fabf&selectedTaskRun=IAbIyNMzRkG1majC5mMagA.0
Assignee: nobody → standard8
|
|
Assignee | |
Comment 2•7 months ago
|
||
This was originally added for backwards compatibility for extensions when fixing the DataTransfer.types type.
The extensions are no longer supported.
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/062ff0141633 Remove the chromeonly DataTransfer.types.contains function. r=dom-core,emilio,edgar
|
||
Comment 4•7 months ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 months ago
status-firefox119:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•