Closed
Bug 1704293
Opened 3 years ago
Closed 3 years ago
Use Maybe<PropertyDescriptor> for Proxy code
Categories
(Core :: JavaScript Engine, task, P1)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: evilpie, Assigned: evilpie)
References
Details
Attachments
(4 files)
No description provided.
Assignee | ||
Comment 1•3 years ago
|
||
Updated•3 years ago
|
Assignee: nobody → evilpies
Status: NEW → ASSIGNED
Assignee | ||
Updated•3 years ago
|
Keywords: leave-open
Pushed by evilpies@gmail.com: https://hg.mozilla.org/integration/autoland/rev/9bae0160b0d6 Use Maybe<PropertyDescriptor> for SetPropertyIgnoringNamedGetter. r=jandem
Updated•3 years ago
|
Severity: -- → N/A
Priority: -- → P1
Comment 3•3 years ago
|
||
bugherder |
Assignee | ||
Comment 4•3 years ago
|
||
Updated•3 years ago
|
Attachment #9215274 -
Attachment description: WIP: Bug 1704293 - Use Maybe<PropertyDescriptor> for BaseProxyHandler::getOwnPropertyDescriptor in js/. r?jandem → Bug 1704293 - Use Maybe<PropertyDescriptor> for BaseProxyHandler::getOwnPropertyDescriptor in js/. r?jandem
Assignee | ||
Comment 5•3 years ago
|
||
Depends on D111730
Assignee | ||
Comment 6•3 years ago
|
||
Also switches the "default" GetOwnPropertyDescriptor implementation to Maybe<PropertyDescriptor>.
Pushed by evilpies@gmail.com: https://hg.mozilla.org/integration/autoland/rev/21a46ef4dbc8 Use Maybe<PropertyDescriptor> for BaseProxyHandler::getOwnPropertyDescriptor in js/. r=jandem https://hg.mozilla.org/integration/autoland/rev/ec33e5f0fa0b Use Maybe<PropertyDescriptor> for BaseProxyHandler::getOwnPropertyDescriptor in DOM and XPConnect. r=jandem,smaug https://hg.mozilla.org/integration/autoland/rev/603e149b2695 Use Maybe<PropertyDescriptor> for GetOwnPropertyOp. r=jandem
Pushed by nbeleuzu@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8c489939b8b8 Fix lint on Codegen.py . CLOSED TREE
Comment 9•3 years ago
|
||
bugherder |
Assignee | ||
Updated•3 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•