Summary: | REGRESSION(r219045): The <body> element does not get repainted when its background image finishes decoding | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Said Abou-Hallawa <sabouhallawa> | ||||||||
Component: | Images | Assignee: | Said Abou-Hallawa <sabouhallawa> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | buildbot, cdumez, commit-queue, darin, dbates, japhet, simon.fraser, thorton, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Said Abou-Hallawa
2017-07-11 11:21:06 PDT
Created attachment 315134 [details]
Patch
An example of this is the background image in http://browserbench.org/JetStream/ Comment on attachment 315134 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=315134&action=review > Source/WebCore/loader/cache/CachedImage.cpp:151 > + // The <html> element draws the background image of the <body> element. The <body> > + // element is a client to this CachedImage but the <html> element is not. In this > + // case add all the CachedImage's clients and cancel the repaint optimization we > + // do in CachedImage::imageFrameAvailable(). This comment isn't accurate. The <body> background only propagates to the root if the <html> does not have its own background specified. See RenderView::rendererForRootBackground() and code in RenderBox::repaintLayerRectsForImage, and elsewhere. Created attachment 315144 [details]
Patch
Comment on attachment 315144 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=315144&action=review > Source/WebCore/loader/cache/CachedImage.cpp:154 > + CachedResourceClientWalker<CachedImageClient> w(m_clients); w -> walker > Source/WebCore/loader/cache/CachedImage.cpp:155 > + while (CachedImageClient* c = w.next()) while (auto* client = walker.next()) Created attachment 315161 [details]
Patch
Comment on attachment 315161 [details] Patch Clearing flags on attachment: 315161 Committed r219364: <http://trac.webkit.org/changeset/219364> All reviewed patches have been landed. Closing bug. Comment on attachment 315161 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=315161&action=review > Source/WebCore/loader/cache/CachedImage.cpp:148 > + // If the <html> element does not have its own background sepecfied, painting the root box "sepecfied" |