[webkit-reviews] review granted: [Bug 230459] [lFC][Integration] Enable some text painting features : [Attachment 438594] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Sep 19 09:44:33 PDT 2021


Sam Weinig <sam at webkit.org> has granted Antti Koivisto <koivisto at iki.fi>'s
request for review:
Bug 230459: [lFC][Integration] Enable some text painting features
https://bugs.webkit.org/show_bug.cgi?id=230459

Attachment 438594: Patch

https://bugs.webkit.org/attachment.cgi?id=438594&action=review




--- Comment #2 from Sam Weinig <sam at webkit.org> ---
Comment on attachment 438594
  --> https://bugs.webkit.org/attachment.cgi?id=438594
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=438594&action=review

> Source/WebCore/layout/integration/LayoutIntegrationCoverage.h:63
> +    // Unused				    = 1LLU  << 23,

Are you doing this so you don't have to change all the numbers after this one?
Or do these need to stay consistent across builds for serialization reasons?
(either is ok, just curious)


More information about the webkit-reviews mailing list