Fuchsia binary size increase

[Scott Violet]

Hello,

A while back we added a new html parser to blink. This parser is used from setinner/outerhtml as well as DOMParser. This parser significantly improved Chrome's score on Speedometer. Because of size concerns this parser is not enabled on fuchsia. I'm wondering if we are still concerned with binary size for fuchsia? The size bot on fuchsia says enabling the parser results in an additional 16384 bytes. See https://chromium-review.googlesource.com/c/chromium/src/+/5189682?tab=checks . Any guidance would be appreciated.

Thanks!

  -Scott

[Zijie He]

Hey, Scott,

16K shouldn't be a concern as we are facing almost 100K/month organic growth.

.Hzj_jie

--
You received this message because you are subscribed to the Google Groups "fuchsia-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to fuchsia-dev...@chromium.org.
To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/fuchsia-dev/CAKARY_m81i-BK%3DTpXZF00Fu9WQWxaD-2z2ALxu1FENHpjcT-Jw%40mail.gmail.com.
For more options, visit https://groups.google.com/a/chromium.org/d/optout.

[Scott Violet]

Great! I sent the review your way: https://chromium-review.googlesource.com/c/chromium/src/+/5189682 .

  -Scott

[Zijie He]

Thank you Scott, LGTM'ed.

To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/fuchsia-dev/CAKARY_kYdFTeBGLG7vFwT2pXyG8Rx6kcBo-Ds9qSBS6_DLk8zA%40mail.gmail.com.