wikimedia / parsoid / 3022

85%

push

travis-ci

<a href="https://github.com/wikimedia/parsoid/commit/6187b179a">Fix inefficiencies looking up id attributes in storeDataAttribs

* This is a second attempt to address PHP DOM issues with getElementById
  and Zest&#39;s ineffiencies in looking up ids on the DOM via XPath.

* This fixes performance on the tagged task.

* In the first attempt of this patch, I had used a querySelectorAll([&#39;id])
  but a manual visitDOM turned out to be much faster than that even
  though the querySelectorAll did fix the n^2 issue.

* Also condition the check in visitAndStoreDataAttribs on whether
  we need to storeInPageBundle since it turns out html2wt had slowed
  down 2x since 6fa28acd which unconditionally used the querySelector
  hack in visitAndStoreDataAttribs.

* Flamegraph confirms the fix.

Bug: T232180
Change-Id: I</a><a class="double-link" href="https://github.com/wikimedia/parsoid/commit/bb80bd81c">bb80bd81c</a>

19282 of 24282 branches covered (79.41%)

30146 of 35550 relevant lines covered (84.8%)

96869.73 hits per line