[wp-trac] [WordPress Trac] #60697: HTML API: Defer applying updates until necessary.

WordPress Trac noreply at wordpress.org
Mon Mar 11 23:53:24 UTC 2024


#60697: HTML API: Defer applying updates until necessary.
--------------------------------------+------------------------------
 Reporter:  dmsnell                   |       Owner:  dmsnell
     Type:  defect (bug)              |      Status:  closed
 Priority:  normal                    |   Milestone:  Awaiting Review
Component:  HTML API                  |     Version:  6.3
 Severity:  normal                    |  Resolution:  fixed
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+------------------------------
Changes (by dmsnell):

 * owner:  (none) => dmsnell
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"57805" 57805]:
 {{{
 #!CommitTicketReference repository="" revision="57805"
 HTML API: Defer applying attribute updates until necessary.

 When making repeated updates to a document, the Tag Processor will end
 up copying the entire document once for every update. This can lead to
 catastrophic behavior in the worse case.

 However, when batch-applying updates it's able to copy chunks of the
 document in one thread and only end up copying the entire document once
 for the entire batch.

 Previously the Tag Processor has been eagerly applying udpates, but in
 this patch it defers applying those updates as long as is possible.

 Developed in https://github.com/WordPress/wordpress-develop/pull/6120
 Discussed in https://core.trac.wordpress.org/ticket/60697

 Props: dmsnell, bernhard-reiter, jonsurrell, westonruter.
 Fixes #60697.
 Follow-up to [55706], [56941], [57348].
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/60697#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list