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

WordPress Trac noreply at wordpress.org
Tue Mar 12 14:25:46 UTC 2024


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

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"57815" 57815]:
 {{{
 #!CommitTicketReference repository="" revision="57815"
 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

 Follow-up to [55706], [56941], [57348].

 Reviewed by swissspidy.
 Merges [57805] to the to the 6.5 branch.

 Props dmsnell, bernhard-reiter, jonsurrell, westonruter.
 Fixes #60697.
 }}}

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


More information about the wp-trac mailing list