Closed Bug 1850293 Opened 1 year ago Closed 1 year ago

Remove some more intrinsic states.

Categories

(Core :: DOM: Core & HTML, task)

task

Tracking

()

RESOLVED FIXED
119 Branch
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
No description provided.

Allows us to reclaim some node bits too.

Depends on D186877

Assignee: nobody → emilio
Status: NEW → ASSIGNED

This one needs extra code because we have an UpdateState call after
AfterSetAttr which took care of it.

Depends on D186894

This one is tricky because form controls, though I think I got it right...

Depends on D186895

These only apply to input, so it's rather trivial-ish.

Depends on D186933

Keywords: leave-open
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/fa4b8bfa49a9 Make directionality state not intrinsic. r=smaug
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a9ca65a951fc Make INCREMENT_SCRIPT_LEVEL state non intrinsic. r=smaug
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/13449f0dcd4d Make <meter> styles not intrinsic. r=edgar
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/95fb96c0db31 Make PLACEHOLDER_SHOWN not an intrinsic state. r=smaug

Add a RAII helper to notify of multiple state changes together for
these.

Depends on D187005

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.

Blocks: 1850161
No longer depends on: 1850161
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4b02c7357724 Make editable states not intrinsic. r=smaug,masayuki https://hg.mozilla.org/integration/autoland/rev/e2901434f15d Make CHECKED/DEFAULT states not intrinsic. r=smaug https://hg.mozilla.org/integration/autoland/rev/a033e3702f2e Make in-range/out-of-range states non-intrinsic. r=smaug https://hg.mozilla.org/integration/autoland/rev/beb7d600e03d Make indeterminate state non-intrinsic. r=edgar https://hg.mozilla.org/integration/autoland/rev/083153001b0d Make validity states non-intrinsic. r=smaug
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41710 for changes under testing/web-platform/tests
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d88a18607fdb Don't use IntrinsicState() in VideoFrame constructor. r=smaug
Upstream PR merged by moz-wptsync-bot
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Keywords: leave-open
Target Milestone: --- → 119 Branch
Regressions: 1861027
Regressions: 1861346
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: