Remove some more intrinsic states.
Categories
(Core :: DOM: Core & HTML, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox119 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(10 files)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review |
| Assignee | ||
Comment 1•2 years ago
|
||
Allows us to reclaim some node bits too.
Depends on D186877
Updated•2 years ago
|
| Assignee | ||
Comment 2•2 years ago
|
||
Depends on D186893
| Assignee | ||
Comment 3•2 years ago
|
||
This one needs extra code because we have an UpdateState call after
AfterSetAttr which took care of it.
Depends on D186894
| Assignee | ||
Comment 4•2 years ago
|
||
This one is tricky because form controls, though I think I got it right...
Depends on D186895
| Assignee | ||
Comment 5•2 years ago
|
||
Depends on D186896
| Assignee | ||
Comment 6•2 years ago
|
||
Depends on D186932
| Assignee | ||
Comment 7•2 years ago
|
||
These only apply to input, so it's rather trivial-ish.
Depends on D186933
| Assignee | ||
Updated•2 years ago
|
Comment 10•2 years ago
|
||
Comment 11•2 years ago
|
||
| Assignee | ||
Comment 12•2 years ago
|
||
Comment 13•2 years ago
|
||
| bugherder | ||
| Assignee | ||
Comment 14•2 years ago
|
||
Add a RAII helper to notify of multiple state changes together for
these.
Depends on D187005
| Assignee | ||
Comment 15•2 years ago
|
||
State() is cheaper, and non-virtual, and should be up-to-date. I'm
removing the concept of "intrinsic" states and this caught my eye.
Updated•2 years ago
|
Comment 16•2 years ago
|
||
Comment 18•2 years ago
|
||
Comment 19•2 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/4b02c7357724
https://hg.mozilla.org/mozilla-central/rev/e2901434f15d
https://hg.mozilla.org/mozilla-central/rev/a033e3702f2e
https://hg.mozilla.org/mozilla-central/rev/beb7d600e03d
https://hg.mozilla.org/mozilla-central/rev/083153001b0d
https://hg.mozilla.org/mozilla-central/rev/d88a18607fdb
| Assignee | ||
Updated•2 years ago
|
Updated•2 years ago
|
Description
•