Editorial: use newer method/getter/setter styles #7294
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For many (but not all) method/getter/setter definitions, update to use:
I had the ambition of removing all "this X object" uses, but some are more complicated and I gave up about 2/3 of the way through, realizing I'd spent too much time on this already. Still, a nice step in the right direction. Searching for
this[\s\n]+<code>[^<]+</code>(?![\s\n]+element)
now gives 64 matches, mostly OffscreenCanvas, CustomElementRegistry, and ElementInternals.Most changes here are mechanical but the canvas methods I updated probably need a bit closer review than the rest.
/browsers.html ( diff )
/canvas.html ( diff )
/common-dom-interfaces.html ( diff )
/dynamic-markup-insertion.html ( diff )
/form-elements.html ( diff )
/history.html ( diff )
/imagebitmap-and-animations.html ( diff )
/links.html ( diff )
/media.html ( diff )
/web-messaging.html ( diff )
/window-object.html ( diff )
/workers.html ( diff )