<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Style resolver should be updated lazily"
href="https://bugs.webkit.org/show_bug.cgi?id=163721#c14">Comment # 14</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Style resolver should be updated lazily"
href="https://bugs.webkit.org/show_bug.cgi?id=163721">bug 163721</a>
from <span class="vcard"><a class="email" href="mailto:koivisto@iki.fi" title="Antti Koivisto <koivisto@iki.fi>"> <span class="fn">Antti Koivisto</span></a>
</span></b>
<pre><span class="quote">> IIUC this is needed for hasPendingForcedStyleRecalc() to work correctly. Is
> that accurate? If so, do we also need to do this in
> Element::needsStyleInvalidation()?</span >
No because needsStyleInvalidation() is used to avoid invalidation work when we know we are already invalidated. Calling flushPendingUpdates() would catch more cases like that but it is way more expensive than per-element work that needsStyleInvalidation() is trying to optimize.
<span class="quote">>
> > Source/WebCore/testing/Internals.cpp:385
> > + page.group().captionPreferences().setTestingMode(true);
>
> This deserves a ChangeLog comment at least. :P</span >
Yea. It is about avoiding inserting magical caption UA sheet that screws up some inspector tests. The existing code that was avoiding that was racy.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>