<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[205586] releases/WebKitGTK/webkit-2.14/Source/WebCore</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/205586">205586</a></dd>
<dt>Author</dt> <dd>carlosgc@webkit.org</dd>
<dt>Date</dt> <dd>2016-09-07 23:31:44 -0700 (Wed, 07 Sep 2016)</dd>
</dl>
<h3>Log Message</h3>
<pre>Merge <a href="http://trac.webkit.org/projects/webkit/changeset/205181">r205181</a> - Remove StylePendingImage
https://bugs.webkit.org/show_bug.cgi?id=161245
Reviewed by Andreas Kling.
Instances of this confusing type are used as placeholders during style resolution.
We can get rid of it and make "pending" a state of StyleImage. This simplies the code and
will allow further improvements.
* WebCore.xcodeproj/project.pbxproj:
* css/CSSCrossfadeValue.cpp:
(WebCore::CSSCrossfadeValue::isPending):
(WebCore::CSSCrossfadeValue::blend):
* css/CSSCrossfadeValue.h:
* css/CSSCursorImageValue.cpp:
(WebCore::CSSCursorImageValue::CSSCursorImageValue):
(WebCore::CSSCursorImageValue::~CSSCursorImageValue):
Save the original URL since the underlying CSSImageValue may change.
(WebCore::CSSCursorImageValue::updateCursorElement):
(WebCore::CSSCursorImageValue::cursorElementRemoved):
(WebCore::CSSCursorImageValue::cursorElementChanged):
(WebCore::CSSCursorImageValue::loadImage):
(WebCore::CSSCursorImageValue::styleImage):
(WebCore::CSSCursorImageValue::isSVGCursor):
(WebCore::CSSCursorImageValue::cachedImageURL):
(WebCore::CSSCursorImageValue::updateCachedImage):
(WebCore::CSSCursorImageValue::detachPendingImage): Deleted.
(WebCore::CSSCursorImageValue::cachedImage): Deleted.
(WebCore::CSSCursorImageValue::cachedOrPendingImage): Deleted.
(WebCore::CSSCursorImageValue::clearCachedImage): Deleted.
* css/CSSCursorImageValue.h:
* css/CSSFilterImageValue.cpp:
(WebCore::CSSFilterImageValue::isPending):
* css/CSSFilterImageValue.h:
* css/CSSImageGeneratorValue.cpp:
(WebCore::CSSImageGeneratorValue::isPending):
(WebCore::CSSImageGeneratorValue::subimageIsPending):
(WebCore::CSSImageGeneratorValue::cachedImageForCSSValue):
* css/CSSImageGeneratorValue.h:
* css/CSSImageSetValue.cpp:
(WebCore::CSSImageSetValue::~CSSImageSetValue):
(WebCore::CSSImageSetValue::loadBestFitImage):
(WebCore::CSSImageSetValue::styleImage):
(WebCore::CSSImageSetValue::traverseSubresources):
(WebCore::CSSImageSetValue::detachPendingImage): Deleted.
(WebCore::CSSImageSetValue::bestFitImage): Deleted.
(WebCore::CSSImageSetValue::cachedOrPendingImageSet): Deleted.
* css/CSSImageSetValue.h:
* css/CSSImageValue.cpp:
(WebCore::CSSImageValue::CSSImageValue):
(WebCore::CSSImageValue::~CSSImageValue):
(WebCore::CSSImageValue::isPending):
(WebCore::CSSImageValue::styleImage):
(WebCore::CSSImageValue::loadImage):
(WebCore::CSSImageValue::traverseSubresources):
(WebCore::CSSImageValue::detachPendingImage): Deleted.
(WebCore::CSSImageValue::cachedOrPendingImage): Deleted.
(WebCore::CSSImageValue::cachedImage): Deleted.
* css/CSSImageValue.h:
(WebCore::CSSImageValue::create):
* css/StyleResolver.cpp:
(WebCore::StyleResolver::cachedOrPendingFromValue):
(WebCore::StyleResolver::generatedOrPendingFromValue):
(WebCore::StyleResolver::setOrPendingFromValue):
(WebCore::StyleResolver::cursorOrPendingFromValue):
* page/PageSerializer.cpp:
(WebCore::PageSerializer::retrieveResourcesForProperties):
* page/animation/CSSPropertyAnimation.cpp:
(WebCore::blendFilter):
(WebCore::crossfadeBlend):
* rendering/RenderImageResourceStyleImage.cpp:
(WebCore::RenderImageResourceStyleImage::image):
* rendering/shapes/ShapeOutsideInfo.cpp:
(WebCore::ShapeOutsideInfo::createShapeForImage):
* rendering/style/StyleCachedImage.cpp:
(WebCore::StyleCachedImage::StyleCachedImage):
(WebCore::StyleCachedImage::~StyleCachedImage):
(WebCore::StyleCachedImage::setCachedImage):
(WebCore::StyleCachedImage::cssValue):
(WebCore::StyleCachedImage::canRender):
(WebCore::StyleCachedImage::isPending):
(WebCore::StyleCachedImage::isLoaded):
(WebCore::StyleCachedImage::errorOccurred):
(WebCore::StyleCachedImage::imageSize):
(WebCore::StyleCachedImage::imageHasRelativeWidth):
(WebCore::StyleCachedImage::imageHasRelativeHeight):
(WebCore::StyleCachedImage::computeIntrinsicDimensions):
(WebCore::StyleCachedImage::usesImageContainerSize):
(WebCore::StyleCachedImage::setContainerSizeForRenderer):
(WebCore::StyleCachedImage::addClient):
(WebCore::StyleCachedImage::removeClient):
(WebCore::StyleCachedImage::image):
(WebCore::StyleCachedImage::knownToBeOpaque):
* rendering/style/StyleCachedImage.h:
* rendering/style/StyleGeneratedImage.cpp:
(WebCore::StyleGeneratedImage::isPending):
* rendering/style/StyleGeneratedImage.h:
* rendering/style/StyleImage.h:
(WebCore::StyleImage::isCachedImage):
(WebCore::StyleImage::StyleImage):
(WebCore::StyleImage::canRender): Deleted.
(WebCore::StyleImage::isPendingImage): Deleted.
* rendering/style/StylePendingImage.h: Removed.
* style/StylePendingResources.cpp:
(WebCore::Style::loadPendingImage):
(WebCore::Style::loadPendingImages):</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCoreChangeLog">releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCoreWebCorexcodeprojprojectpbxproj">releases/WebKitGTK/webkit-2.14/Source/WebCore/WebCore.xcodeproj/project.pbxproj</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSCrossfadeValuecpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCrossfadeValue.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSCrossfadeValueh">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCrossfadeValue.h</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSCursorImageValuecpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCursorImageValue.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSCursorImageValueh">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCursorImageValue.h</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSFilterImageValuecpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSFilterImageValue.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSFilterImageValueh">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSFilterImageValue.h</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSImageGeneratorValuecpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageGeneratorValue.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSImageGeneratorValueh">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageGeneratorValue.h</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSImageSetValuecpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageSetValue.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSImageSetValueh">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageSetValue.h</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSImageValuecpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageValue.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssCSSImageValueh">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageValue.h</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorecssStyleResolvercpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/css/StyleResolver.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorepagePageSerializercpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/page/PageSerializer.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorepageanimationCSSPropertyAnimationcpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/page/animation/CSSPropertyAnimation.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorerenderingRenderImageResourceStyleImagecpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/RenderImageResourceStyleImage.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorerenderingshapesShapeOutsideInfocpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/shapes/ShapeOutsideInfo.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleCachedImagecpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleCachedImage.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleCachedImageh">releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleCachedImage.h</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleGeneratedImagecpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleGeneratedImage.cpp</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleGeneratedImageh">releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleGeneratedImage.h</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleImageh">releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleImage.h</a></li>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorestyleStylePendingResourcescpp">releases/WebKitGTK/webkit-2.14/Source/WebCore/style/StylePendingResources.cpp</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li><a href="#releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStylePendingImageh">releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StylePendingImage.h</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="releasesWebKitGTKwebkit214SourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/ChangeLog 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -1,3 +1,114 @@
</span><ins>+2016-08-30 Antti Koivisto <antti@apple.com>
+
+ Remove StylePendingImage
+ https://bugs.webkit.org/show_bug.cgi?id=161245
+
+ Reviewed by Andreas Kling.
+
+ Instances of this confusing type are used as placeholders during style resolution.
+ We can get rid of it and make "pending" a state of StyleImage. This simplies the code and
+ will allow further improvements.
+
+ * WebCore.xcodeproj/project.pbxproj:
+ * css/CSSCrossfadeValue.cpp:
+ (WebCore::CSSCrossfadeValue::isPending):
+ (WebCore::CSSCrossfadeValue::blend):
+ * css/CSSCrossfadeValue.h:
+ * css/CSSCursorImageValue.cpp:
+ (WebCore::CSSCursorImageValue::CSSCursorImageValue):
+ (WebCore::CSSCursorImageValue::~CSSCursorImageValue):
+
+ Save the original URL since the underlying CSSImageValue may change.
+
+ (WebCore::CSSCursorImageValue::updateCursorElement):
+ (WebCore::CSSCursorImageValue::cursorElementRemoved):
+ (WebCore::CSSCursorImageValue::cursorElementChanged):
+ (WebCore::CSSCursorImageValue::loadImage):
+ (WebCore::CSSCursorImageValue::styleImage):
+ (WebCore::CSSCursorImageValue::isSVGCursor):
+ (WebCore::CSSCursorImageValue::cachedImageURL):
+ (WebCore::CSSCursorImageValue::updateCachedImage):
+ (WebCore::CSSCursorImageValue::detachPendingImage): Deleted.
+ (WebCore::CSSCursorImageValue::cachedImage): Deleted.
+ (WebCore::CSSCursorImageValue::cachedOrPendingImage): Deleted.
+ (WebCore::CSSCursorImageValue::clearCachedImage): Deleted.
+ * css/CSSCursorImageValue.h:
+ * css/CSSFilterImageValue.cpp:
+ (WebCore::CSSFilterImageValue::isPending):
+ * css/CSSFilterImageValue.h:
+ * css/CSSImageGeneratorValue.cpp:
+ (WebCore::CSSImageGeneratorValue::isPending):
+ (WebCore::CSSImageGeneratorValue::subimageIsPending):
+ (WebCore::CSSImageGeneratorValue::cachedImageForCSSValue):
+ * css/CSSImageGeneratorValue.h:
+ * css/CSSImageSetValue.cpp:
+ (WebCore::CSSImageSetValue::~CSSImageSetValue):
+ (WebCore::CSSImageSetValue::loadBestFitImage):
+ (WebCore::CSSImageSetValue::styleImage):
+ (WebCore::CSSImageSetValue::traverseSubresources):
+ (WebCore::CSSImageSetValue::detachPendingImage): Deleted.
+ (WebCore::CSSImageSetValue::bestFitImage): Deleted.
+ (WebCore::CSSImageSetValue::cachedOrPendingImageSet): Deleted.
+ * css/CSSImageSetValue.h:
+ * css/CSSImageValue.cpp:
+ (WebCore::CSSImageValue::CSSImageValue):
+ (WebCore::CSSImageValue::~CSSImageValue):
+ (WebCore::CSSImageValue::isPending):
+ (WebCore::CSSImageValue::styleImage):
+ (WebCore::CSSImageValue::loadImage):
+ (WebCore::CSSImageValue::traverseSubresources):
+ (WebCore::CSSImageValue::detachPendingImage): Deleted.
+ (WebCore::CSSImageValue::cachedOrPendingImage): Deleted.
+ (WebCore::CSSImageValue::cachedImage): Deleted.
+ * css/CSSImageValue.h:
+ (WebCore::CSSImageValue::create):
+ * css/StyleResolver.cpp:
+ (WebCore::StyleResolver::cachedOrPendingFromValue):
+ (WebCore::StyleResolver::generatedOrPendingFromValue):
+ (WebCore::StyleResolver::setOrPendingFromValue):
+ (WebCore::StyleResolver::cursorOrPendingFromValue):
+ * page/PageSerializer.cpp:
+ (WebCore::PageSerializer::retrieveResourcesForProperties):
+ * page/animation/CSSPropertyAnimation.cpp:
+ (WebCore::blendFilter):
+ (WebCore::crossfadeBlend):
+ * rendering/RenderImageResourceStyleImage.cpp:
+ (WebCore::RenderImageResourceStyleImage::image):
+ * rendering/shapes/ShapeOutsideInfo.cpp:
+ (WebCore::ShapeOutsideInfo::createShapeForImage):
+ * rendering/style/StyleCachedImage.cpp:
+ (WebCore::StyleCachedImage::StyleCachedImage):
+ (WebCore::StyleCachedImage::~StyleCachedImage):
+ (WebCore::StyleCachedImage::setCachedImage):
+ (WebCore::StyleCachedImage::cssValue):
+ (WebCore::StyleCachedImage::canRender):
+ (WebCore::StyleCachedImage::isPending):
+ (WebCore::StyleCachedImage::isLoaded):
+ (WebCore::StyleCachedImage::errorOccurred):
+ (WebCore::StyleCachedImage::imageSize):
+ (WebCore::StyleCachedImage::imageHasRelativeWidth):
+ (WebCore::StyleCachedImage::imageHasRelativeHeight):
+ (WebCore::StyleCachedImage::computeIntrinsicDimensions):
+ (WebCore::StyleCachedImage::usesImageContainerSize):
+ (WebCore::StyleCachedImage::setContainerSizeForRenderer):
+ (WebCore::StyleCachedImage::addClient):
+ (WebCore::StyleCachedImage::removeClient):
+ (WebCore::StyleCachedImage::image):
+ (WebCore::StyleCachedImage::knownToBeOpaque):
+ * rendering/style/StyleCachedImage.h:
+ * rendering/style/StyleGeneratedImage.cpp:
+ (WebCore::StyleGeneratedImage::isPending):
+ * rendering/style/StyleGeneratedImage.h:
+ * rendering/style/StyleImage.h:
+ (WebCore::StyleImage::isCachedImage):
+ (WebCore::StyleImage::StyleImage):
+ (WebCore::StyleImage::canRender): Deleted.
+ (WebCore::StyleImage::isPendingImage): Deleted.
+ * rendering/style/StylePendingImage.h: Removed.
+ * style/StylePendingResources.cpp:
+ (WebCore::Style::loadPendingImage):
+ (WebCore::Style::loadPendingImages):
+
</ins><span class="cx"> 2016-08-29 Alex Christensen <achristensen@webkit.org>
</span><span class="cx">
</span><span class="cx"> Fix Windows build after r205161
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCoreWebCorexcodeprojprojectpbxproj"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/WebCore.xcodeproj/project.pbxproj (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/WebCore.xcodeproj/project.pbxproj 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -503,7 +503,6 @@
</span><span class="cx"> 0F97A658155DA81E00FADD4C /* DisplayRefreshMonitorIOS.mm in Sources */ = {isa = PBXBuildFile; fileRef = 0F97A657155DA81E00FADD4C /* DisplayRefreshMonitorIOS.mm */; };
</span><span class="cx"> 0FA24D79162DF91900A3F4C0 /* GraphicsLayerUpdater.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FA24D77162DF91900A3F4C0 /* GraphicsLayerUpdater.cpp */; };
</span><span class="cx"> 0FA24D7A162DF91900A3F4C0 /* GraphicsLayerUpdater.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FA24D78162DF91900A3F4C0 /* GraphicsLayerUpdater.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><del>- 0FA6F96F1BCF614A0031EE70 /* StylePendingImage.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FA6F96E1BCF614A0031EE70 /* StylePendingImage.h */; };
</del><span class="cx"> 0FA88EBD16A8D1BD00F99984 /* ScrollingStateFrameScrollingNodeMac.mm in Sources */ = {isa = PBXBuildFile; fileRef = 0FA88EBC16A8D1BD00F99984 /* ScrollingStateFrameScrollingNodeMac.mm */; };
</span><span class="cx"> 0FB6252E18DE1B1500A07C05 /* GeometryUtilities.cpp in Sources */ = {isa = PBXBuildFile; fileRef = 0FB6252C18DE1B1500A07C05 /* GeometryUtilities.cpp */; };
</span><span class="cx"> 0FB6252F18DE1B1500A07C05 /* GeometryUtilities.h in Headers */ = {isa = PBXBuildFile; fileRef = 0FB6252D18DE1B1500A07C05 /* GeometryUtilities.h */; settings = {ATTRIBUTES = (Private, ); }; };
</span><span class="lines">@@ -7276,7 +7275,6 @@
</span><span class="cx"> 0F97A657155DA81E00FADD4C /* DisplayRefreshMonitorIOS.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = DisplayRefreshMonitorIOS.mm; sourceTree = "<group>"; };
</span><span class="cx"> 0FA24D77162DF91900A3F4C0 /* GraphicsLayerUpdater.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = GraphicsLayerUpdater.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 0FA24D78162DF91900A3F4C0 /* GraphicsLayerUpdater.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = GraphicsLayerUpdater.h; sourceTree = "<group>"; };
</span><del>- 0FA6F96E1BCF614A0031EE70 /* StylePendingImage.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = StylePendingImage.h; sourceTree = "<group>"; };
</del><span class="cx"> 0FA88EBC16A8D1BD00F99984 /* ScrollingStateFrameScrollingNodeMac.mm */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.objcpp; path = ScrollingStateFrameScrollingNodeMac.mm; sourceTree = "<group>"; };
</span><span class="cx"> 0FB6252C18DE1B1500A07C05 /* GeometryUtilities.cpp */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.cpp.cpp; path = GeometryUtilities.cpp; sourceTree = "<group>"; };
</span><span class="cx"> 0FB6252D18DE1B1500A07C05 /* GeometryUtilities.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = GeometryUtilities.h; sourceTree = "<group>"; };
</span><span class="lines">@@ -21192,7 +21190,6 @@
</span><span class="cx"> BC5EB7280E81DE8100B25965 /* StyleMarqueeData.h */,
</span><span class="cx"> BC5EB74B0E81E06700B25965 /* StyleMultiColData.cpp */,
</span><span class="cx"> BC5EB74C0E81E06700B25965 /* StyleMultiColData.h */,
</span><del>- 0FA6F96E1BCF614A0031EE70 /* StylePendingImage.h */,
</del><span class="cx"> BC2272E10E82EE9B00E7F975 /* StyleRareInheritedData.cpp */,
</span><span class="cx"> BC2272E20E82EE9B00E7F975 /* StyleRareInheritedData.h */,
</span><span class="cx"> BC2272BA0E82EAAE00E7F975 /* StyleRareNonInheritedData.cpp */,
</span><span class="lines">@@ -26361,7 +26358,6 @@
</span><span class="cx"> BC5EB72A0E81DE8100B25965 /* StyleMarqueeData.h in Headers */,
</span><span class="cx"> 0FF50272102BA96A0066F39A /* StyleMedia.h in Headers */,
</span><span class="cx"> BC5EB74E0E81E06700B25965 /* StyleMultiColData.h in Headers */,
</span><del>- 0FA6F96F1BCF614A0031EE70 /* StylePendingImage.h in Headers */,
</del><span class="cx"> E4DACE6A1D12E10B0075980F /* StylePendingResources.h in Headers */,
</span><span class="cx"> A80E6DFC0A199067007FB8C5 /* StyleProperties.h in Headers */,
</span><span class="cx"> E1B25107152A0BB00069B779 /* StylePropertyShorthand.h in Headers */,
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSCrossfadeValuecpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCrossfadeValue.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCrossfadeValue.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCrossfadeValue.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -110,7 +110,7 @@
</span><span class="cx"> fromImageSize.height() * inversePercentage + toImageSize.height() * percentage);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-bool CSSCrossfadeValue::isPending()
</del><ins>+bool CSSCrossfadeValue::isPending() const
</ins><span class="cx"> {
</span><span class="cx"> return CSSImageGeneratorValue::subimageIsPending(m_fromValue)
</span><span class="cx"> || CSSImageGeneratorValue::subimageIsPending(m_toValue);
</span><span class="lines">@@ -199,12 +199,9 @@
</span><span class="cx"> ASSERT(equalInputImages(from));
</span><span class="cx"> if (!m_cachedToImage || !m_cachedFromImage)
</span><span class="cx"> return nullptr;
</span><del>- RefPtr<StyleCachedImage> toStyledImage = StyleCachedImage::create(m_cachedToImage.get());
- RefPtr<StyleCachedImage> fromStyledImage = StyleCachedImage::create(m_cachedFromImage.get());
</del><ins>+ auto fromImageValue = CSSImageValue::create(*m_cachedFromImage);
+ auto toImageValue = CSSImageValue::create(*m_cachedToImage);
</ins><span class="cx">
</span><del>- auto fromImageValue = CSSImageValue::create(m_cachedFromImage->url(), fromStyledImage.get());
- auto toImageValue = CSSImageValue::create(m_cachedToImage->url(), toStyledImage.get());
-
</del><span class="cx"> double fromPercentage = from.m_percentageValue->getDoubleValue();
</span><span class="cx"> if (from.m_percentageValue->isPercentage())
</span><span class="cx"> fromPercentage /= 100.0;
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSCrossfadeValueh"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCrossfadeValue.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCrossfadeValue.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCrossfadeValue.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -57,7 +57,7 @@
</span><span class="cx"> FloatSize fixedSize(const RenderElement*);
</span><span class="cx">
</span><span class="cx"> bool isPrefixed() const { return m_isPrefixed; }
</span><del>- bool isPending();
</del><ins>+ bool isPending() const;
</ins><span class="cx"> bool knownToBeOpaque(const RenderElement*) const;
</span><span class="cx">
</span><span class="cx"> void loadSubimages(CachedResourceLoader&, const ResourceLoaderOptions&);
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSCursorImageValuecpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCursorImageValue.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCursorImageValue.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCursorImageValue.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -28,7 +28,6 @@
</span><span class="cx"> #include "CachedResourceLoader.h"
</span><span class="cx"> #include "StyleCachedImage.h"
</span><span class="cx"> #include "StyleImage.h"
</span><del>-#include "StylePendingImage.h"
</del><span class="cx"> #include "SVGCursorElement.h"
</span><span class="cx"> #include "SVGLengthContext.h"
</span><span class="cx"> #include "SVGNames.h"
</span><span class="lines">@@ -46,18 +45,12 @@
</span><span class="cx"> , m_hasHotSpot(hasHotSpot)
</span><span class="cx"> , m_hotSpot(hotSpot)
</span><span class="cx"> {
</span><ins>+ if (is<CSSImageValue>(m_imageValue.get()))
+ m_originalURL = { ParsedURLString, downcast<CSSImageValue>(m_imageValue.get()).url() };
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-inline void CSSCursorImageValue::detachPendingImage()
-{
- if (is<StylePendingImage>(m_image.get()))
- downcast<StylePendingImage>(*m_image).detachFromCSSValue();
-}
-
</del><span class="cx"> CSSCursorImageValue::~CSSCursorImageValue()
</span><span class="cx"> {
</span><del>- detachPendingImage();
-
</del><span class="cx"> for (auto* element : m_cursorElements)
</span><span class="cx"> element->removeClient(*this);
</span><span class="cx"> }
</span><span class="lines">@@ -77,10 +70,10 @@
</span><span class="cx">
</span><span class="cx"> SVGCursorElement* CSSCursorImageValue::updateCursorElement(const Document& document)
</span><span class="cx"> {
</span><del>- if (!isSVGCursor())
</del><ins>+ if (!m_originalURL.hasFragmentIdentifier())
</ins><span class="cx"> return nullptr;
</span><span class="cx">
</span><del>- auto* element = SVGURIReference::targetElementFromIRIString(downcast<CSSImageValue>(m_imageValue.get()).url(), document);
</del><ins>+ auto* element = SVGURIReference::targetElementFromIRIString(m_originalURL, document);
</ins><span class="cx"> if (!is<SVGCursorElement>(element))
</span><span class="cx"> return nullptr;
</span><span class="cx">
</span><span class="lines">@@ -95,13 +88,10 @@
</span><span class="cx"> void CSSCursorImageValue::cursorElementRemoved(SVGCursorElement& cursorElement)
</span><span class="cx"> {
</span><span class="cx"> m_cursorElements.remove(&cursorElement);
</span><del>- clearCachedImage();
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void CSSCursorImageValue::cursorElementChanged(SVGCursorElement& cursorElement)
</span><span class="cx"> {
</span><del>- clearCachedImage();
-
</del><span class="cx"> // FIXME: This will override hot spot specified in CSS, which is probably incorrect.
</span><span class="cx"> SVGLengthContext lengthContext(nullptr);
</span><span class="cx"> m_hasHotSpot = true;
</span><span class="lines">@@ -112,73 +102,34 @@
</span><span class="cx"> m_hotSpot.setY(static_cast<int>(y));
</span><span class="cx"> }
</span><span class="cx">
</span><del>-StyleImage* CSSCursorImageValue::cachedImage(CachedResourceLoader& loader, const ResourceLoaderOptions& options)
</del><ins>+void CSSCursorImageValue::loadImage(CachedResourceLoader& loader, const ResourceLoaderOptions& options)
</ins><span class="cx"> {
</span><del>- if (is<CSSImageSetValue>(m_imageValue.get()))
- return downcast<CSSImageSetValue>(m_imageValue.get()).bestFitImage(loader, options);
-
- auto* cursorElement = loader.document() ? updateCursorElement(*loader.document()) : nullptr;
-
- if (!m_isImageValid) {
- m_isImageValid = true;
- // For SVG images we need to lazily substitute in the correct URL. Rather than attempt
- // to change the URL of the CSSImageValue (which would then change behavior like cssText),
- // we create an alternate CSSImageValue to use.
- if (cursorElement) {
- detachPendingImage();
- Ref<CSSImageValue> svgImageValue(CSSImageValue::create(cursorElement->href()));
- StyleCachedImage* cachedImage = svgImageValue->cachedImage(loader, options);
- m_image = cachedImage;
- return cachedImage;
- }
-
- if (is<CSSImageValue>(m_imageValue.get())) {
- detachPendingImage();
- m_image = downcast<CSSImageValue>(m_imageValue.get()).cachedImage(loader, options);
- }
</del><ins>+ if (is<CSSImageSetValue>(m_imageValue.get())) {
+ downcast<CSSImageSetValue>(m_imageValue.get()).loadBestFitImage(loader, options);
+ return;
</ins><span class="cx"> }
</span><span class="cx">
</span><del>- if (is<StyleCachedImage>(m_image.get()))
- return downcast<StyleCachedImage>(m_image.get());
-
- return nullptr;
</del><ins>+ downcast<CSSImageValue>(m_imageValue.get()).loadImage(loader, options);
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-StyleImage* CSSCursorImageValue::cachedOrPendingImage(const Document& document)
</del><ins>+StyleCachedImage& CSSCursorImageValue::styleImage(const Document& document)
</ins><span class="cx"> {
</span><span class="cx"> // Need to delegate completely so that changes in device scale factor can be handled appropriately.
</span><ins>+ StyleCachedImage* styleImage;
</ins><span class="cx"> if (is<CSSImageSetValue>(m_imageValue.get()))
</span><del>- return downcast<CSSImageSetValue>(m_imageValue.get()).cachedOrPendingImageSet(document);
-
- if (!m_image)
- m_image = StylePendingImage::create(this);
-
- return m_image.get();
-}
-
-bool CSSCursorImageValue::isSVGCursor() const
-{
- if (is<CSSImageValue>(m_imageValue.get())) {
- URL kurl(ParsedURLString, downcast<CSSImageValue>(m_imageValue.get()).url());
- return kurl.hasFragmentIdentifier();
</del><ins>+ styleImage = &downcast<CSSImageSetValue>(m_imageValue.get()).styleImage(document);
+ else {
+ if (auto* cursorElement = updateCursorElement(document)) {
+ if (cursorElement->href() != downcast<CSSImageValue>(m_imageValue.get()).url())
+ m_imageValue = CSSImageValue::create(cursorElement->href());
+ }
+ styleImage = &downcast<CSSImageValue>(m_imageValue.get()).styleImage();
</ins><span class="cx"> }
</span><del>- return false;
-}
</del><ins>+ styleImage->setCSSValue(*this);
</ins><span class="cx">
</span><del>-String CSSCursorImageValue::cachedImageURL()
-{
- if (!is<StyleCachedImage>(m_image.get()))
- return String();
- return downcast<StyleCachedImage>(*m_image).cachedImage()->url();
</del><ins>+ return *styleImage;
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-void CSSCursorImageValue::clearCachedImage()
-{
- detachPendingImage();
- m_image = nullptr;
- m_isImageValid = false;
-}
-
</del><span class="cx"> bool CSSCursorImageValue::equals(const CSSCursorImageValue& other) const
</span><span class="cx"> {
</span><span class="cx"> return m_hasHotSpot ? other.m_hasHotSpot && m_hotSpot == other.m_hotSpot : !other.m_hasHotSpot
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSCursorImageValueh"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCursorImageValue.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCursorImageValue.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSCursorImageValue.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -31,6 +31,7 @@
</span><span class="cx"> class Element;
</span><span class="cx"> class SVGCursorElement;
</span><span class="cx"> class SVGElement;
</span><ins>+class StyleCachedImage;
</ins><span class="cx">
</span><span class="cx"> class CSSCursorImageValue final : public CSSValue {
</span><span class="cx"> public:
</span><span class="lines">@@ -52,9 +53,8 @@
</span><span class="cx">
</span><span class="cx"> String customCSSText() const;
</span><span class="cx">
</span><del>- SVGCursorElement* updateCursorElement(const Document&);
- StyleImage* cachedImage(CachedResourceLoader&, const ResourceLoaderOptions&);
- StyleImage* cachedOrPendingImage(const Document&);
</del><ins>+ void loadImage(CachedResourceLoader&, const ResourceLoaderOptions&);
+ StyleCachedImage& styleImage(const Document&);
</ins><span class="cx">
</span><span class="cx"> void removeReferencedElement(SVGElement*);
</span><span class="cx">
</span><span class="lines">@@ -66,18 +66,13 @@
</span><span class="cx"> private:
</span><span class="cx"> CSSCursorImageValue(Ref<CSSValue>&& imageValue, bool hasHotSpot, const IntPoint& hotSpot);
</span><span class="cx">
</span><del>- void detachPendingImage();
</del><ins>+ SVGCursorElement* updateCursorElement(const Document&);
</ins><span class="cx">
</span><del>- bool isSVGCursor() const;
- String cachedImageURL();
- void clearCachedImage();
-
</del><ins>+ URL m_originalURL;
</ins><span class="cx"> Ref<CSSValue> m_imageValue;
</span><span class="cx">
</span><span class="cx"> bool m_hasHotSpot;
</span><span class="cx"> IntPoint m_hotSpot;
</span><del>- RefPtr<StyleImage> m_image;
- bool m_isImageValid { false };
</del><span class="cx"> HashSet<SVGCursorElement*> m_cursorElements;
</span><span class="cx"> };
</span><span class="cx">
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSFilterImageValuecpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSFilterImageValue.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSFilterImageValue.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSFilterImageValue.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -73,7 +73,7 @@
</span><span class="cx"> return cachedImage->imageForRenderer(renderer)->size();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-bool CSSFilterImageValue::isPending()
</del><ins>+bool CSSFilterImageValue::isPending() const
</ins><span class="cx"> {
</span><span class="cx"> return CSSImageGeneratorValue::subimageIsPending(m_imageValue);
</span><span class="cx"> }
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSFilterImageValueh"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSFilterImageValue.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSFilterImageValue.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSFilterImageValue.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -59,7 +59,7 @@
</span><span class="cx"> bool isFixedSize() const { return true; }
</span><span class="cx"> FloatSize fixedSize(const RenderElement*);
</span><span class="cx">
</span><del>- bool isPending();
</del><ins>+ bool isPending() const;
</ins><span class="cx"> bool knownToBeOpaque(const RenderElement*) const;
</span><span class="cx">
</span><span class="cx"> void loadSubimages(CachedResourceLoader&, const ResourceLoaderOptions&);
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSImageGeneratorValuecpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageGeneratorValue.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageGeneratorValue.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageGeneratorValue.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -167,7 +167,7 @@
</span><span class="cx"> return FloatSize();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-bool CSSImageGeneratorValue::isPending()
</del><ins>+bool CSSImageGeneratorValue::isPending() const
</ins><span class="cx"> {
</span><span class="cx"> switch (classType()) {
</span><span class="cx"> case CrossfadeClass:
</span><span class="lines">@@ -232,10 +232,10 @@
</span><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span><del>-bool CSSImageGeneratorValue::subimageIsPending(CSSValue& value)
</del><ins>+bool CSSImageGeneratorValue::subimageIsPending(const CSSValue& value)
</ins><span class="cx"> {
</span><span class="cx"> if (is<CSSImageValue>(value))
</span><del>- return downcast<CSSImageValue>(value).cachedOrPendingImage().isPendingImage();
</del><ins>+ return downcast<CSSImageValue>(value).isPending();
</ins><span class="cx">
</span><span class="cx"> if (is<CSSImageGeneratorValue>(value))
</span><span class="cx"> return downcast<CSSImageGeneratorValue>(value).isPending();
</span><span class="lines">@@ -251,11 +251,9 @@
</span><span class="cx"> CachedImage* CSSImageGeneratorValue::cachedImageForCSSValue(CSSValue& value, CachedResourceLoader& cachedResourceLoader, const ResourceLoaderOptions& options)
</span><span class="cx"> {
</span><span class="cx"> if (is<CSSImageValue>(value)) {
</span><del>- StyleCachedImage* styleCachedImage = downcast<CSSImageValue>(value).cachedImage(cachedResourceLoader, options);
- if (!styleCachedImage)
- return nullptr;
-
- return styleCachedImage->cachedImage();
</del><ins>+ auto& imageValue = downcast<CSSImageValue>(value);
+ imageValue.loadImage(cachedResourceLoader, options);
+ return imageValue.styleImage().cachedImage();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> if (is<CSSImageGeneratorValue>(value)) {
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSImageGeneratorValueh"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageGeneratorValue.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageGeneratorValue.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageGeneratorValue.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -54,7 +54,7 @@
</span><span class="cx"> bool isFixedSize() const;
</span><span class="cx"> FloatSize fixedSize(const RenderElement*);
</span><span class="cx">
</span><del>- bool isPending();
</del><ins>+ bool isPending() const;
</ins><span class="cx"> bool knownToBeOpaque(const RenderElement*) const;
</span><span class="cx">
</span><span class="cx"> void loadSubimages(CachedResourceLoader&, const ResourceLoaderOptions&);
</span><span class="lines">@@ -68,7 +68,7 @@
</span><span class="cx">
</span><span class="cx"> // Helper functions for Crossfade and Filter.
</span><span class="cx"> static CachedImage* cachedImageForCSSValue(CSSValue&, CachedResourceLoader&, const ResourceLoaderOptions&);
</span><del>- static bool subimageIsPending(CSSValue&);
</del><ins>+ static bool subimageIsPending(const CSSValue&);
</ins><span class="cx">
</span><span class="cx"> private:
</span><span class="cx"> class CachedGeneratedImage {
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSImageSetValuecpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageSetValue.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageSetValue.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageSetValue.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -36,7 +36,6 @@
</span><span class="cx"> #include "Document.h"
</span><span class="cx"> #include "Page.h"
</span><span class="cx"> #include "StyleCachedImage.h"
</span><del>-#include "StylePendingImage.h"
</del><span class="cx"> #include <wtf/text/StringBuilder.h>
</span><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="lines">@@ -48,18 +47,10 @@
</span><span class="cx"> {
</span><span class="cx"> }
</span><span class="cx">
</span><del>-inline void CSSImageSetValue::detachPendingImage()
-{
- if (is<StylePendingImage>(m_image.get()))
- downcast<StylePendingImage>(*m_image).detachFromCSSValue();
-}
-
</del><span class="cx"> CSSImageSetValue::~CSSImageSetValue()
</span><span class="cx"> {
</span><del>- detachPendingImage();
-
- if (is<StyleCachedImage>(m_image.get()))
- downcast<StyleCachedImage>(*m_image).detachFromCSSValue();
</del><ins>+ if (m_image)
+ m_image->detachFromCSSValue();
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void CSSImageSetValue::fillImageSet()
</span><span class="lines">@@ -98,7 +89,7 @@
</span><span class="cx"> return image;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-StyleCachedImage* CSSImageSetValue::bestFitImage(CachedResourceLoader& loader, const ResourceLoaderOptions& options)
</del><ins>+void CSSImageSetValue::loadBestFitImage(CachedResourceLoader& loader, const ResourceLoaderOptions& options)
</ins><span class="cx"> {
</span><span class="cx"> Document* document = loader.document();
</span><span class="cx"> if (Page* page = document->page())
</span><span class="lines">@@ -109,32 +100,29 @@
</span><span class="cx"> if (!m_imagesInSet.size())
</span><span class="cx"> fillImageSet();
</span><span class="cx">
</span><del>- if (!m_accessedBestFitImage) {
- // FIXME: In the future, we want to take much more than deviceScaleFactor into acount here.
- // All forms of scale should be included: Page::pageScaleFactor(), Frame::pageZoomFactor(),
- // and any CSS transforms. https://bugs.webkit.org/show_bug.cgi?id=81698
- ImageWithScale image = bestImageForScaleFactor();
- CachedResourceRequest request(ResourceRequest(document->completeURL(image.imageURL)), options);
- request.setInitiator(cachedResourceRequestInitiators().css);
- if (options.mode == FetchOptions::Mode::Cors) {
- ASSERT(document->securityOrigin());
- updateRequestForAccessControl(request.mutableResourceRequest(), *document->securityOrigin(), options.allowCredentials);
- }
- if (CachedResourceHandle<CachedImage> cachedImage = loader.requestImage(request)) {
- detachPendingImage();
- m_image = StyleCachedImage::createForImageSet(cachedImage.get(), image.scaleFactor, *this);
- m_accessedBestFitImage = true;
- }
</del><ins>+ if (m_accessedBestFitImage)
+ return;
+ // FIXME: In the future, we want to take much more than deviceScaleFactor into acount here.
+ // All forms of scale should be included: Page::pageScaleFactor(), Frame::pageZoomFactor(),
+ // and any CSS transforms. https://bugs.webkit.org/show_bug.cgi?id=81698
+ ImageWithScale image = bestImageForScaleFactor();
+ CachedResourceRequest request(ResourceRequest(document->completeURL(image.imageURL)), options);
+ request.setInitiator(cachedResourceRequestInitiators().css);
+ if (options.mode == FetchOptions::Mode::Cors) {
+ ASSERT(document->securityOrigin());
+ updateRequestForAccessControl(request.mutableResourceRequest(), *document->securityOrigin(), options.allowCredentials);
</ins><span class="cx"> }
</span><del>-
- return is<StyleCachedImage>(m_image.get()) ? downcast<StyleCachedImage>(m_image.get()) : nullptr;
</del><ins>+ if (CachedResourceHandle<CachedImage> cachedImage = loader.requestImage(request)) {
+ styleImage(*document).setCachedImage(*cachedImage, image.scaleFactor);
+ m_accessedBestFitImage = true;
+ }
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-StyleImage* CSSImageSetValue::cachedOrPendingImageSet(const Document& document)
</del><ins>+StyleCachedImage& CSSImageSetValue::styleImage(const Document& document)
</ins><span class="cx"> {
</span><span class="cx"> if (!m_image)
</span><del>- m_image = StylePendingImage::create(this);
- else if (!m_image->isPendingImage()) {
</del><ins>+ m_image = StyleCachedImage::create(*this);
+ else if (!m_image->isPending()) {
</ins><span class="cx"> float deviceScaleFactor = 1;
</span><span class="cx"> if (Page* page = document.page())
</span><span class="cx"> deviceScaleFactor = page->deviceScaleFactor();
</span><span class="lines">@@ -142,11 +130,12 @@
</span><span class="cx"> // If the deviceScaleFactor has changed, we may not have the best image loaded, so we have to re-assess.
</span><span class="cx"> if (deviceScaleFactor != m_scaleFactor) {
</span><span class="cx"> m_accessedBestFitImage = false;
</span><del>- m_image = StylePendingImage::create(this);
</del><ins>+ m_image->detachFromCSSValue();
+ m_image = StyleCachedImage::create(*this);
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx">
</span><del>- return m_image.get();
</del><ins>+ return *m_image;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> String CSSImageSetValue::customCSSText() const
</span><span class="lines">@@ -181,10 +170,11 @@
</span><span class="cx">
</span><span class="cx"> bool CSSImageSetValue::traverseSubresources(const std::function<bool (const CachedResource&)>& handler) const
</span><span class="cx"> {
</span><del>- if (!is<StyleCachedImage>(m_image.get()))
</del><ins>+ if (!m_image)
</ins><span class="cx"> return false;
</span><del>- CachedImage* cachedResource = downcast<StyleCachedImage>(*m_image).cachedImage();
- ASSERT(cachedResource);
</del><ins>+ CachedImage* cachedResource = m_image->cachedImage();
+ if (!cachedResource)
+ return false;
</ins><span class="cx"> return handler(*cachedResource);
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSImageSetValueh"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageSetValue.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageSetValue.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageSetValue.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -45,11 +45,9 @@
</span><span class="cx"> }
</span><span class="cx"> ~CSSImageSetValue();
</span><span class="cx">
</span><del>- StyleCachedImage* bestFitImage(CachedResourceLoader&, const ResourceLoaderOptions&);
</del><ins>+ void loadBestFitImage(CachedResourceLoader&, const ResourceLoaderOptions&);
+ StyleCachedImage& styleImage(const Document&);
</ins><span class="cx">
</span><del>- // Returns a StyleCachedImage if the best fit image has been cached already, otherwise a StylePendingImage.
- StyleImage* cachedOrPendingImageSet(const Document&);
-
</del><span class="cx"> String customCSSText() const;
</span><span class="cx">
</span><span class="cx"> bool isPending() const { return !m_accessedBestFitImage; }
</span><span class="lines">@@ -70,11 +68,10 @@
</span><span class="cx"> CSSImageSetValue();
</span><span class="cx"> CSSImageSetValue(const CSSImageSetValue& cloneFrom);
</span><span class="cx">
</span><del>- void detachPendingImage();
</del><span class="cx"> void fillImageSet();
</span><span class="cx"> static inline bool compareByScaleFactor(ImageWithScale first, ImageWithScale second) { return first.scaleFactor < second.scaleFactor; }
</span><span class="cx">
</span><del>- RefPtr<StyleImage> m_image;
</del><ins>+ RefPtr<StyleCachedImage> m_image;
</ins><span class="cx"> bool m_accessedBestFitImage;
</span><span class="cx">
</span><span class="cx"> // This represents the scale factor that we used to find the best fit image. It does not necessarily
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSImageValuecpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageValue.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageValue.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageValue.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -33,7 +33,6 @@
</span><span class="cx"> #include "Element.h"
</span><span class="cx"> #include "MemoryCache.h"
</span><span class="cx"> #include "StyleCachedImage.h"
</span><del>-#include "StylePendingImage.h"
</del><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><span class="lines">@@ -44,63 +43,62 @@
</span><span class="cx"> {
</span><span class="cx"> }
</span><span class="cx">
</span><del>-CSSImageValue::CSSImageValue(const String& url, StyleImage* image)
</del><ins>+CSSImageValue::CSSImageValue(CachedImage& image)
</ins><span class="cx"> : CSSValue(ImageClass)
</span><del>- , m_url(url)
- , m_image(image)
</del><ins>+ , m_url(image.url())
+ , m_image(StyleCachedImage::create(*this))
</ins><span class="cx"> , m_accessedImage(true)
</span><span class="cx"> {
</span><ins>+ m_image->setCachedImage(image);
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-inline void CSSImageValue::detachPendingImage()
</del><ins>+
+CSSImageValue::~CSSImageValue()
</ins><span class="cx"> {
</span><del>- if (is<StylePendingImage>(m_image.get()))
- downcast<StylePendingImage>(*m_image).detachFromCSSValue();
</del><ins>+ if (m_image)
+ m_image->detachFromCSSValue();
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-CSSImageValue::~CSSImageValue()
</del><ins>+bool CSSImageValue::isPending() const
</ins><span class="cx"> {
</span><del>- detachPendingImage();
</del><ins>+ return !m_image || !m_image->cachedImage();
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-StyleImage& CSSImageValue::cachedOrPendingImage()
</del><ins>+StyleCachedImage& CSSImageValue::styleImage()
</ins><span class="cx"> {
</span><span class="cx"> if (!m_image)
</span><del>- m_image = StylePendingImage::create(this);
</del><ins>+ m_image = StyleCachedImage::create(*this);
</ins><span class="cx">
</span><span class="cx"> return *m_image;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-StyleCachedImage* CSSImageValue::cachedImage(CachedResourceLoader& loader, const ResourceLoaderOptions& options)
</del><ins>+void CSSImageValue::loadImage(CachedResourceLoader& loader, const ResourceLoaderOptions& options)
</ins><span class="cx"> {
</span><del>- if (!m_accessedImage) {
- m_accessedImage = true;
</del><ins>+ if (m_accessedImage)
+ return;
+ m_accessedImage = true;
</ins><span class="cx">
</span><del>- CachedResourceRequest request(ResourceRequest(loader.document()->completeURL(m_url)), options);
- if (m_initiatorName.isEmpty())
- request.setInitiator(cachedResourceRequestInitiators().css);
- else
- request.setInitiator(m_initiatorName);
</del><ins>+ CachedResourceRequest request(ResourceRequest(loader.document()->completeURL(m_url)), options);
+ if (m_initiatorName.isEmpty())
+ request.setInitiator(cachedResourceRequestInitiators().css);
+ else
+ request.setInitiator(m_initiatorName);
</ins><span class="cx">
</span><del>- if (options.mode == FetchOptions::Mode::Cors) {
- ASSERT(loader.document()->securityOrigin());
- updateRequestForAccessControl(request.mutableResourceRequest(), *loader.document()->securityOrigin(), options.allowCredentials);
- }
- if (CachedResourceHandle<CachedImage> cachedImage = loader.requestImage(request)) {
- detachPendingImage();
- m_image = StyleCachedImage::create(cachedImage.get());
- }
</del><ins>+ if (options.mode == FetchOptions::Mode::Cors) {
+ ASSERT(loader.document()->securityOrigin());
+ updateRequestForAccessControl(request.mutableResourceRequest(), *loader.document()->securityOrigin(), options.allowCredentials);
</ins><span class="cx"> }
</span><del>-
- return is<StyleCachedImage>(m_image.get()) ? downcast<StyleCachedImage>(m_image.get()) : nullptr;
</del><ins>+ if (CachedResourceHandle<CachedImage> cachedImage = loader.requestImage(request))
+ styleImage().setCachedImage(*cachedImage);
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool CSSImageValue::traverseSubresources(const std::function<bool (const CachedResource&)>& handler) const
</span><span class="cx"> {
</span><del>- if (!is<StyleCachedImage>(m_image.get()))
</del><ins>+ if (!m_image)
</ins><span class="cx"> return false;
</span><del>- CachedResource* cachedResource = downcast<StyleCachedImage>(*m_image).cachedImage();
- ASSERT(cachedResource);
</del><ins>+ CachedResource* cachedResource = m_image->cachedImage();
+ if (!cachedResource)
+ return false;
</ins><span class="cx"> return handler(*cachedResource);
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssCSSImageValueh"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageValue.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageValue.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/CSSImageValue.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -26,10 +26,10 @@
</span><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><ins>+class CachedImage;
</ins><span class="cx"> class CachedResourceLoader;
</span><span class="cx"> class Element;
</span><span class="cx"> class StyleCachedImage;
</span><del>-class StyleImage;
</del><span class="cx"> class RenderElement;
</span><span class="cx"> struct ResourceLoaderOptions;
</span><span class="cx">
</span><span class="lines">@@ -36,12 +36,12 @@
</span><span class="cx"> class CSSImageValue final : public CSSValue {
</span><span class="cx"> public:
</span><span class="cx"> static Ref<CSSImageValue> create(const String& url) { return adoptRef(*new CSSImageValue(url)); }
</span><del>- static Ref<CSSImageValue> create(const String& url, StyleImage* image) { return adoptRef(*new CSSImageValue(url, image)); }
</del><ins>+ static Ref<CSSImageValue> create(CachedImage& image) { return adoptRef(*new CSSImageValue(image)); }
</ins><span class="cx"> ~CSSImageValue();
</span><span class="cx">
</span><del>- StyleCachedImage* cachedImage(CachedResourceLoader&, const ResourceLoaderOptions&);
- // Returns a StyleCachedImage if the image is cached already, otherwise a StylePendingImage.
- StyleImage& cachedOrPendingImage();
</del><ins>+ bool isPending() const;
+ void loadImage(CachedResourceLoader&, const ResourceLoaderOptions&);
+ StyleCachedImage& styleImage();
</ins><span class="cx">
</span><span class="cx"> const String& url() const { return m_url; }
</span><span class="cx">
</span><span class="lines">@@ -59,11 +59,10 @@
</span><span class="cx">
</span><span class="cx"> private:
</span><span class="cx"> explicit CSSImageValue(const String& url);
</span><del>- CSSImageValue(const String& url, StyleImage*);
- void detachPendingImage();
</del><ins>+ explicit CSSImageValue(CachedImage&);
</ins><span class="cx">
</span><span class="cx"> String m_url;
</span><del>- RefPtr<StyleImage> m_image;
</del><ins>+ RefPtr<StyleCachedImage> m_image;
</ins><span class="cx"> bool m_accessedImage;
</span><span class="cx"> AtomicString m_initiatorName;
</span><span class="cx"> };
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorecssStyleResolvercpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/css/StyleResolver.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/css/StyleResolver.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/css/StyleResolver.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -123,7 +123,6 @@
</span><span class="cx"> #include "StyleCachedImage.h"
</span><span class="cx"> #include "StyleFontSizeFunctions.h"
</span><span class="cx"> #include "StyleGeneratedImage.h"
</span><del>-#include "StylePendingImage.h"
</del><span class="cx"> #include "StyleProperties.h"
</span><span class="cx"> #include "StylePropertyShorthand.h"
</span><span class="cx"> #include "StyleRule.h"
</span><span class="lines">@@ -1723,8 +1722,8 @@
</span><span class="cx">
</span><span class="cx"> Ref<StyleImage> StyleResolver::cachedOrPendingFromValue(CSSPropertyID property, CSSImageValue& value)
</span><span class="cx"> {
</span><del>- Ref<StyleImage> image = value.cachedOrPendingImage();
- if (image->isPendingImage())
</del><ins>+ Ref<StyleImage> image = value.styleImage();
+ if (image->isPending())
</ins><span class="cx"> m_state.ensurePendingResources().pendingImages.set(property, &value);
</span><span class="cx"> return image;
</span><span class="cx"> }
</span><span class="lines">@@ -1735,28 +1734,26 @@
</span><span class="cx"> // FilterImage needs to calculate FilterOperations.
</span><span class="cx"> downcast<CSSFilterImageValue>(value).createFilterOperations(this);
</span><span class="cx"> }
</span><ins>+ if (value.isPending())
+ m_state.ensurePendingResources().pendingImages.set(property, &value);
</ins><span class="cx">
</span><del>- if (value.isPending()) {
- m_state.ensurePendingResources().pendingImages.set(property, &value);
- return StylePendingImage::create(&value);
- }
</del><span class="cx"> return StyleGeneratedImage::create(value);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> RefPtr<StyleImage> StyleResolver::setOrPendingFromValue(CSSPropertyID property, CSSImageSetValue& value)
</span><span class="cx"> {
</span><del>- RefPtr<StyleImage> image = value.cachedOrPendingImageSet(document());
- if (image && image->isPendingImage())
</del><ins>+ auto& image = value.styleImage(document());
+ if (image.isPending())
</ins><span class="cx"> m_state.ensurePendingResources().pendingImages.set(property, &value);
</span><del>- return image;
</del><ins>+ return &image;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> RefPtr<StyleImage> StyleResolver::cursorOrPendingFromValue(CSSPropertyID property, CSSCursorImageValue& value)
</span><span class="cx"> {
</span><del>- RefPtr<StyleImage> image = value.cachedOrPendingImage(document());
- if (image && image->isPendingImage())
</del><ins>+ auto& image = value.styleImage(document());
+ if (image.isPending())
</ins><span class="cx"> m_state.ensurePendingResources().pendingImages.set(property, &value);
</span><del>- return image;
</del><ins>+ return &image;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> #if ENABLE(IOS_TEXT_AUTOSIZING)
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorepagePageSerializercpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/page/PageSerializer.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/page/PageSerializer.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/page/PageSerializer.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -331,13 +331,12 @@
</span><span class="cx"> if (!is<CSSImageValue>(*cssValue))
</span><span class="cx"> continue;
</span><span class="cx">
</span><del>- auto& styleImage = downcast<CSSImageValue>(*cssValue).cachedOrPendingImage();
- // Non cached-images are just place-holders and do not contain data.
- if (!is<StyleCachedImage>(styleImage))
</del><ins>+ auto& styleImage = downcast<CSSImageValue>(*cssValue).styleImage();
+
+ auto* image = styleImage.cachedImage();
+ if (!image)
</ins><span class="cx"> continue;
</span><span class="cx">
</span><del>- CachedImage* image = downcast<StyleCachedImage>(styleImage).cachedImage();
-
</del><span class="cx"> URL url = document->completeURL(image->url());
</span><span class="cx"> addImageToResources(image, nullptr, url);
</span><span class="cx"> }
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorepageanimationCSSPropertyAnimationcpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/page/animation/CSSPropertyAnimation.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/page/animation/CSSPropertyAnimation.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/page/animation/CSSPropertyAnimation.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -225,8 +225,7 @@
</span><span class="cx"> ASSERT(image);
</span><span class="cx"> FilterOperations filterResult = blendFilterOperations(anim, from, to, progress);
</span><span class="cx">
</span><del>- RefPtr<StyleCachedImage> styledImage = StyleCachedImage::create(image);
- auto imageValue = CSSImageValue::create(image->url(), styledImage.get());
</del><ins>+ auto imageValue = CSSImageValue::create(*image);
</ins><span class="cx"> auto filterValue = ComputedStyleExtractor::valueForFilter(anim->renderer()->style(), filterResult, DoNotAdjustPixelValues);
</span><span class="cx">
</span><span class="cx"> auto result = CSSFilterImageValue::create(WTFMove(imageValue), WTFMove(filterValue));
</span><span class="lines">@@ -290,8 +289,8 @@
</span><span class="cx"> if (progress == 1)
</span><span class="cx"> return toStyleImage;
</span><span class="cx">
</span><del>- auto fromImageValue = CSSImageValue::create(fromStyleImage->cachedImage()->url(), fromStyleImage);
- auto toImageValue = CSSImageValue::create(toStyleImage->cachedImage()->url(), toStyleImage);
</del><ins>+ auto fromImageValue = CSSImageValue::create(*fromStyleImage->cachedImage());
+ auto toImageValue = CSSImageValue::create(*toStyleImage->cachedImage());
</ins><span class="cx"> auto percentageValue = CSSPrimitiveValue::create(progress, CSSPrimitiveValue::CSS_NUMBER);
</span><span class="cx">
</span><span class="cx"> auto crossfadeValue = CSSCrossfadeValue::create(WTFMove(fromImageValue), WTFMove(toImageValue), WTFMove(percentageValue));
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorerenderingRenderImageResourceStyleImagecpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/RenderImageResourceStyleImage.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/RenderImageResourceStyleImage.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/RenderImageResourceStyleImage.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -63,7 +63,7 @@
</span><span class="cx"> RefPtr<Image> RenderImageResourceStyleImage::image(int width, int height) const
</span><span class="cx"> {
</span><span class="cx"> // Generated content may trigger calls to image() while we're still pending, don't assert but gracefully exit.
</span><del>- if (m_styleImage->isPendingImage())
</del><ins>+ if (m_styleImage->isPending())
</ins><span class="cx"> return nullptr;
</span><span class="cx"> return m_styleImage->image(m_renderer, IntSize(width, height));
</span><span class="cx"> }
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorerenderingshapesShapeOutsideInfocpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/shapes/ShapeOutsideInfo.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/shapes/ShapeOutsideInfo.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/shapes/ShapeOutsideInfo.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -156,7 +156,7 @@
</span><span class="cx"> ? downcast<RenderImage>(m_renderer).replacedContentRect(m_renderer.intrinsicSize())
</span><span class="cx"> : LayoutRect(LayoutPoint(), imageSize);
</span><span class="cx">
</span><del>- ASSERT(!styleImage->isPendingImage());
</del><ins>+ ASSERT(!styleImage->isPending());
</ins><span class="cx"> RefPtr<Image> image = styleImage->image(const_cast<RenderBox*>(&m_renderer), imageSize);
</span><span class="cx"> return Shape::createRasterShape(image.get(), shapeImageThreshold, imageRect, marginRect, writingMode, margin);
</span><span class="cx"> }
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleCachedImagecpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleCachedImage.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleCachedImage.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleCachedImage.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -30,44 +30,63 @@
</span><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><del>-StyleCachedImage::StyleCachedImage(CachedImage* image, float scaleFactor, CSSImageSetValue* cssValue)
- : m_image(image)
- , m_scaleFactor(scaleFactor)
- , m_cssImageSetValue(cssValue)
</del><ins>+StyleCachedImage::StyleCachedImage(CSSValue& cssValue)
+ : m_cssValue(&cssValue)
</ins><span class="cx"> {
</span><span class="cx"> m_isCachedImage = true;
</span><del>- m_image->addClient(this);
</del><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> StyleCachedImage::~StyleCachedImage()
</span><span class="cx"> {
</span><del>- m_image->removeClient(this);
</del><ins>+ if (m_image)
+ m_image->removeClient(this);
</ins><span class="cx"> }
</span><span class="cx">
</span><ins>+void StyleCachedImage::setCachedImage(CachedImage& image, float scaleFactor)
+{
+ ASSERT(!m_image);
+ m_image = &image;
+ m_image->addClient(this);
+ m_scaleFactor = scaleFactor;
+}
+
</ins><span class="cx"> PassRefPtr<CSSValue> StyleCachedImage::cssValue() const
</span><span class="cx"> {
</span><del>- if (m_cssImageSetValue)
- return const_cast<CSSImageSetValue*>(m_cssImageSetValue);
</del><ins>+ if (m_cssValue)
+ return m_cssValue;
</ins><span class="cx"> return CSSPrimitiveValue::create(m_image->url(), CSSPrimitiveValue::CSS_URI);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool StyleCachedImage::canRender(const RenderObject* renderer, float multiplier) const
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return false;
</ins><span class="cx"> return m_image->canRender(renderer, multiplier);
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+bool StyleCachedImage::isPending() const
+{
+ return !m_image;
+}
+
</ins><span class="cx"> bool StyleCachedImage::isLoaded() const
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return false;
</ins><span class="cx"> return m_image->isLoaded();
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool StyleCachedImage::errorOccurred() const
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return false;
</ins><span class="cx"> return m_image->errorOccurred();
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> FloatSize StyleCachedImage::imageSize(const RenderElement* renderer, float multiplier) const
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return { };
</ins><span class="cx"> FloatSize size = m_image->imageSizeForRenderer(renderer, multiplier);
</span><span class="cx"> size.scale(1 / m_scaleFactor);
</span><span class="cx"> return size;
</span><span class="lines">@@ -75,41 +94,57 @@
</span><span class="cx">
</span><span class="cx"> bool StyleCachedImage::imageHasRelativeWidth() const
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return false;
</ins><span class="cx"> return m_image->imageHasRelativeWidth();
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool StyleCachedImage::imageHasRelativeHeight() const
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return false;
</ins><span class="cx"> return m_image->imageHasRelativeHeight();
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void StyleCachedImage::computeIntrinsicDimensions(const RenderElement*, Length& intrinsicWidth, Length& intrinsicHeight, FloatSize& intrinsicRatio)
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return;
</ins><span class="cx"> m_image->computeIntrinsicDimensions(intrinsicWidth, intrinsicHeight, intrinsicRatio);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> bool StyleCachedImage::usesImageContainerSize() const
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return false;
</ins><span class="cx"> return m_image->usesImageContainerSize();
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void StyleCachedImage::setContainerSizeForRenderer(const RenderElement* renderer, const FloatSize& imageContainerSize, float imageContainerZoomFactor)
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return;
</ins><span class="cx"> m_image->setContainerSizeForRenderer(renderer, LayoutSize(imageContainerSize), imageContainerZoomFactor);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void StyleCachedImage::addClient(RenderElement* renderer)
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return;
</ins><span class="cx"> m_image->addClient(renderer);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> void StyleCachedImage::removeClient(RenderElement* renderer)
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return;
</ins><span class="cx"> m_image->removeClient(renderer);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> RefPtr<Image> StyleCachedImage::image(RenderElement* renderer, const FloatSize&) const
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return nullptr;
</ins><span class="cx"> return m_image->imageForRenderer(renderer);
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -120,6 +155,8 @@
</span><span class="cx">
</span><span class="cx"> bool StyleCachedImage::knownToBeOpaque(const RenderElement* renderer) const
</span><span class="cx"> {
</span><ins>+ if (!m_image)
+ return false;
</ins><span class="cx"> return m_image->currentFrameKnownToBeOpaque(renderer);
</span><span class="cx"> }
</span><span class="cx">
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleCachedImageh"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleCachedImage.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleCachedImage.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleCachedImage.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -30,26 +30,28 @@
</span><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><del>-class CSSImageSetValue;
</del><ins>+class CSSValue;
</ins><span class="cx"> class CachedImage;
</span><span class="cx">
</span><span class="cx"> class StyleCachedImage final : public StyleImage, private CachedImageClient {
</span><span class="cx"> WTF_MAKE_FAST_ALLOCATED;
</span><span class="cx"> public:
</span><del>- static Ref<StyleCachedImage> create(CachedImage* image) { return adoptRef(*new StyleCachedImage(image, 1, nullptr)); }
- static Ref<StyleCachedImage> createForImageSet(CachedImage* image, float scaleFactor, CSSImageSetValue& cssValue) { return adoptRef(*new StyleCachedImage(image, scaleFactor, &cssValue)); }
</del><ins>+ static Ref<StyleCachedImage> create(CSSValue& cssValue) { return adoptRef(*new StyleCachedImage(cssValue)); }
</ins><span class="cx"> virtual ~StyleCachedImage();
</span><span class="cx">
</span><span class="cx"> CachedImage* cachedImage() const override { return m_image.get(); }
</span><span class="cx">
</span><del>- void detachFromCSSValue() { m_cssImageSetValue = nullptr; }
</del><ins>+ void detachFromCSSValue() { m_cssValue = nullptr; }
+ void setCSSValue(CSSValue& value) { m_cssValue = &value; }
</ins><span class="cx">
</span><del>-private:
</del><ins>+ void setCachedImage(CachedImage&, float scaleFactor = 1);
+
</ins><span class="cx"> WrappedImagePtr data() const override { return m_image.get(); }
</span><span class="cx">
</span><span class="cx"> PassRefPtr<CSSValue> cssValue() const override;
</span><span class="cx">
</span><span class="cx"> bool canRender(const RenderObject*, float multiplier) const override;
</span><ins>+ bool isPending() const override;
</ins><span class="cx"> bool isLoaded() const override;
</span><span class="cx"> bool errorOccurred() const override;
</span><span class="cx"> FloatSize imageSize(const RenderElement*, float multiplier) const override;
</span><span class="lines">@@ -64,11 +66,12 @@
</span><span class="cx"> float imageScaleFactor() const override;
</span><span class="cx"> bool knownToBeOpaque(const RenderElement*) const override;
</span><span class="cx">
</span><del>- StyleCachedImage(CachedImage*, float scaleFactor, CSSImageSetValue*);
</del><ins>+private:
+ StyleCachedImage(CSSValue&);
</ins><span class="cx">
</span><ins>+ CSSValue* m_cssValue;
+ float m_scaleFactor { 1 };
</ins><span class="cx"> CachedResourceHandle<CachedImage> m_image;
</span><del>- float m_scaleFactor;
- CSSImageSetValue* m_cssImageSetValue; // Not retained; it owns us.
</del><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleGeneratedImagecpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleGeneratedImage.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleGeneratedImage.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleGeneratedImage.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -42,6 +42,11 @@
</span><span class="cx"> return const_cast<CSSImageGeneratorValue*>(m_imageGeneratorValue.ptr());
</span><span class="cx"> }
</span><span class="cx">
</span><ins>+bool StyleGeneratedImage::isPending() const
+{
+ return m_imageGeneratorValue->isPending();
+}
+
</ins><span class="cx"> FloatSize StyleGeneratedImage::imageSize(const RenderElement* renderer, float multiplier) const
</span><span class="cx"> {
</span><span class="cx"> if (m_fixedSize) {
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleGeneratedImageh"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleGeneratedImage.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleGeneratedImage.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleGeneratedImage.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -45,6 +45,7 @@
</span><span class="cx">
</span><span class="cx"> PassRefPtr<CSSValue> cssValue() const override;
</span><span class="cx">
</span><ins>+ bool isPending() const override;
</ins><span class="cx"> FloatSize imageSize(const RenderElement*, float multiplier) const override;
</span><span class="cx"> bool imageHasRelativeWidth() const override { return !m_fixedSize; }
</span><span class="cx"> bool imageHasRelativeHeight() const override { return !m_fixedSize; }
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStyleImageh"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleImage.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleImage.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StyleImage.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -47,12 +47,13 @@
</span><span class="cx">
</span><span class="cx"> bool operator==(const StyleImage& other) const
</span><span class="cx"> {
</span><del>- return data() == other.data();
</del><ins>+ return &other == this || (data() && data() == other.data());
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> virtual PassRefPtr<CSSValue> cssValue() const = 0;
</span><span class="cx">
</span><span class="cx"> virtual bool canRender(const RenderObject*, float /*multiplier*/) const { return true; }
</span><ins>+ virtual bool isPending() const = 0;
</ins><span class="cx"> virtual bool isLoaded() const { return true; }
</span><span class="cx"> virtual bool errorOccurred() const { return false; }
</span><span class="cx"> virtual FloatSize imageSize(const RenderElement*, float multiplier) const = 0;
</span><span class="lines">@@ -70,18 +71,15 @@
</span><span class="cx"> virtual CachedImage* cachedImage() const { return 0; }
</span><span class="cx">
</span><span class="cx"> ALWAYS_INLINE bool isCachedImage() const { return m_isCachedImage; }
</span><del>- ALWAYS_INLINE bool isPendingImage() const { return m_isPendingImage; }
</del><span class="cx"> ALWAYS_INLINE bool isGeneratedImage() const { return m_isGeneratedImage; }
</span><span class="cx">
</span><span class="cx"> protected:
</span><span class="cx"> StyleImage()
</span><span class="cx"> : m_isCachedImage(false)
</span><del>- , m_isPendingImage(false)
</del><span class="cx"> , m_isGeneratedImage(false)
</span><span class="cx"> {
</span><span class="cx"> }
</span><span class="cx"> bool m_isCachedImage : 1;
</span><del>- bool m_isPendingImage : 1;
</del><span class="cx"> bool m_isGeneratedImage : 1;
</span><span class="cx"> };
</span><span class="cx">
</span></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorerenderingstyleStylePendingImageh"></a>
<div class="delfile"><h4>Deleted: releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StylePendingImage.h (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StylePendingImage.h 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/rendering/style/StylePendingImage.h 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -1,87 +0,0 @@
</span><del>-/*
- * Copyright (C) 2010 Apple Inc. All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- * notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- * notice, this list of conditions and the following disclaimer in the
- * documentation and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY APPLE INC. ``AS IS'' AND ANY
- * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
- * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL APPLE INC. OR
- * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
- * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
- * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
- * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
- * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- */
-
-#ifndef StylePendingImage_h
-#define StylePendingImage_h
-
-#include "CSSCursorImageValue.h"
-#include "CSSImageGeneratorValue.h"
-#include "CSSImageSetValue.h"
-#include "CSSImageValue.h"
-#include "StyleImage.h"
-
-namespace WebCore {
-
-// StylePendingImage is a placeholder StyleImage that is entered into the RenderStyle during
-// style resolution, in order to avoid loading images that are not referenced by the final style.
-// They should never exist in a RenderStyle after it has been returned from the style selector.
-
-class StylePendingImage final : public StyleImage {
-public:
- static Ref<StylePendingImage> create(CSSValue* value) { return adoptRef(*new StylePendingImage(value)); }
-
- CSSImageValue* cssImageValue() const { return is<CSSImageValue>(m_value) ? downcast<CSSImageValue>(m_value) : nullptr; }
- CSSImageGeneratorValue* cssImageGeneratorValue() const { return is<CSSImageGeneratorValue>(m_value) ? static_cast<CSSImageGeneratorValue*>(m_value) : nullptr; }
- CSSCursorImageValue* cssCursorImageValue() const { return is<CSSCursorImageValue>(m_value) ? downcast<CSSCursorImageValue>(m_value) : nullptr; }
- CSSImageSetValue* cssImageSetValue() const { return is<CSSImageSetValue>(m_value) ? downcast<CSSImageSetValue>(m_value) : nullptr; }
-
- void detachFromCSSValue() { m_value = nullptr; }
-
-private:
- WrappedImagePtr data() const override { return const_cast<StylePendingImage*>(this); }
-
- PassRefPtr<CSSValue> cssValue() const override { return m_value; }
-
- FloatSize imageSize(const RenderElement*, float /*multiplier*/) const override { return FloatSize(); }
- bool imageHasRelativeWidth() const override { return false; }
- bool imageHasRelativeHeight() const override { return false; }
- void computeIntrinsicDimensions(const RenderElement*, Length& /* intrinsicWidth */ , Length& /* intrinsicHeight */, FloatSize& /* intrinsicRatio */) override { }
- bool usesImageContainerSize() const override { return false; }
- void setContainerSizeForRenderer(const RenderElement*, const FloatSize&, float) override { }
- void addClient(RenderElement*) override { }
- void removeClient(RenderElement*) override { }
-
- RefPtr<Image> image(RenderElement*, const FloatSize&) const override
- {
- ASSERT_NOT_REACHED();
- return nullptr;
- }
-
- bool knownToBeOpaque(const RenderElement*) const override { return false; }
-
- StylePendingImage(CSSValue* value)
- : m_value(value)
- {
- m_isPendingImage = true;
- }
-
- CSSValue* m_value; // Not retained; it owns us.
-};
-
-} // namespace WebCore
-
-SPECIALIZE_TYPE_TRAITS_STYLE_IMAGE(StylePendingImage, isPendingImage)
-
-#endif // StylePendingImage_h
</del></span></pre></div>
<a id="releasesWebKitGTKwebkit214SourceWebCorestyleStylePendingResourcescpp"></a>
<div class="modfile"><h4>Modified: releases/WebKitGTK/webkit-2.14/Source/WebCore/style/StylePendingResources.cpp (205585 => 205586)</h4>
<pre class="diff"><span>
<span class="info">--- releases/WebKitGTK/webkit-2.14/Source/WebCore/style/StylePendingResources.cpp 2016-09-08 06:02:15 UTC (rev 205585)
+++ releases/WebKitGTK/webkit-2.14/Source/WebCore/style/StylePendingResources.cpp 2016-09-08 06:31:44 UTC (rev 205586)
</span><span class="lines">@@ -26,6 +26,10 @@
</span><span class="cx"> #include "config.h"
</span><span class="cx"> #include "StylePendingResources.h"
</span><span class="cx">
</span><ins>+#include "CSSCursorImageValue.h"
+#include "CSSImageGeneratorValue.h"
+#include "CSSImageSetValue.h"
+#include "CSSImageValue.h"
</ins><span class="cx"> #include "CachedResourceLoader.h"
</span><span class="cx"> #include "CachedSVGDocumentReference.h"
</span><span class="cx"> #include "ContentData.h"
</span><span class="lines">@@ -36,7 +40,6 @@
</span><span class="cx"> #include "SVGURIReference.h"
</span><span class="cx"> #include "StyleCachedImage.h"
</span><span class="cx"> #include "StyleGeneratedImage.h"
</span><del>-#include "StylePendingImage.h"
</del><span class="cx"> #include "TransformFunctions.h"
</span><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="lines">@@ -43,9 +46,11 @@
</span><span class="cx"> namespace Style {
</span><span class="cx">
</span><span class="cx"> enum class LoadPolicy { Normal, ShapeOutside };
</span><del>-static RefPtr<StyleImage> loadPendingImage(Document& document, const StyleImage& image, const Element* element, LoadPolicy loadPolicy = LoadPolicy::Normal)
</del><ins>+static void loadPendingImage(Document& document, const StyleImage* styleImage, const Element* element, LoadPolicy loadPolicy = LoadPolicy::Normal)
</ins><span class="cx"> {
</span><del>- auto& pendingImage = downcast<StylePendingImage>(image);
</del><ins>+ if (!styleImage || !styleImage->isPending())
+ return;
+
</ins><span class="cx"> ResourceLoaderOptions options = CachedResourceLoader::defaultCachedResourceOptions();
</span><span class="cx"> options.contentSecurityPolicyImposition = element && element->isInUserAgentShadowTree() ? ContentSecurityPolicyImposition::SkipPolicyCheck : ContentSecurityPolicyImposition::DoPolicyCheck;
</span><span class="cx">
</span><span class="lines">@@ -55,21 +60,26 @@
</span><span class="cx"> options.allowCredentials = DoNotAllowStoredCredentials;
</span><span class="cx"> }
</span><span class="cx">
</span><del>- if (auto imageValue = pendingImage.cssImageValue())
- return imageValue->cachedImage(document.cachedResourceLoader(), options);
</del><ins>+ auto cssValue = const_cast<StyleImage*>(styleImage)->cssValue();
+ if (is<CSSImageValue>(cssValue.get())) {
+ downcast<CSSImageValue>(*cssValue).loadImage(document.cachedResourceLoader(), options);
+ return;
+ };
</ins><span class="cx">
</span><del>- if (auto imageGeneratorValue = pendingImage.cssImageGeneratorValue()) {
- imageGeneratorValue->loadSubimages(document.cachedResourceLoader(), options);
- return StyleGeneratedImage::create(*imageGeneratorValue);
- }
</del><ins>+ if (is<CSSImageSetValue>(cssValue.get())) {
+ downcast<CSSImageSetValue>(*cssValue).loadBestFitImage(document.cachedResourceLoader(), options);
+ return;
+ };
</ins><span class="cx">
</span><del>- if (auto cursorImageValue = pendingImage.cssCursorImageValue())
- return cursorImageValue->cachedImage(document.cachedResourceLoader(), options);
</del><ins>+ if (is<CSSImageGeneratorValue>(cssValue.get())) {
+ downcast<CSSImageGeneratorValue>(*cssValue).loadSubimages(document.cachedResourceLoader(), options);
+ return;
+ };
</ins><span class="cx">
</span><del>- if (auto imageSetValue = pendingImage.cssImageSetValue())
- return imageSetValue->bestFitImage(document.cachedResourceLoader(), options);
-
- return nullptr;
</del><ins>+ if (is<CSSCursorImageValue>(cssValue.get())) {
+ downcast<CSSCursorImageValue>(*cssValue).loadImage(document.cachedResourceLoader(), options);
+ return;
+ };
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> static void loadPendingImages(const PendingResources& pendingResources, Document& document, RenderStyle& style, const Element* element)
</span><span class="lines">@@ -77,82 +87,52 @@
</span><span class="cx"> for (auto currentProperty : pendingResources.pendingImages.keys()) {
</span><span class="cx"> switch (currentProperty) {
</span><span class="cx"> case CSSPropertyBackgroundImage: {
</span><del>- for (FillLayer* backgroundLayer = &style.ensureBackgroundLayers(); backgroundLayer; backgroundLayer = backgroundLayer->next()) {
- auto* styleImage = backgroundLayer->image();
- if (is<StylePendingImage>(styleImage))
- backgroundLayer->setImage(loadPendingImage(document, *styleImage, element));
- }
</del><ins>+ for (auto* backgroundLayer = &style.ensureBackgroundLayers(); backgroundLayer; backgroundLayer = backgroundLayer->next())
+ loadPendingImage(document, backgroundLayer->image(), element);
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> case CSSPropertyContent: {
</span><del>- for (ContentData* contentData = const_cast<ContentData*>(style.contentData()); contentData; contentData = contentData->next()) {
</del><ins>+ for (auto* contentData = const_cast<ContentData*>(style.contentData()); contentData; contentData = contentData->next()) {
</ins><span class="cx"> if (is<ImageContentData>(*contentData)) {
</span><span class="cx"> auto& styleImage = downcast<ImageContentData>(*contentData).image();
</span><del>- if (is<StylePendingImage>(styleImage)) {
- if (auto loadedImage = loadPendingImage(document, styleImage, element))
- downcast<ImageContentData>(*contentData).setImage(WTFMove(loadedImage));
- }
</del><ins>+ loadPendingImage(document, &styleImage, element);
</ins><span class="cx"> }
</span><span class="cx"> }
</span><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> case CSSPropertyCursor: {
</span><del>- if (CursorList* cursorList = style.cursors()) {
- for (size_t i = 0; i < cursorList->size(); ++i) {
- CursorData& currentCursor = cursorList->at(i);
- auto* styleImage = currentCursor.image();
- if (is<StylePendingImage>(styleImage))
- currentCursor.setImage(loadPendingImage(document, *styleImage, element));
- }
</del><ins>+ if (auto* cursorList = style.cursors()) {
+ for (size_t i = 0; i < cursorList->size(); ++i)
+ loadPendingImage(document, cursorList->at(i).image(), element);
</ins><span class="cx"> }
</span><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> case CSSPropertyListStyleImage: {
</span><del>- auto* styleImage = style.listStyleImage();
- if (is<StylePendingImage>(styleImage))
- style.setListStyleImage(loadPendingImage(document, *styleImage, element));
</del><ins>+ loadPendingImage(document, style.listStyleImage(), element);
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> case CSSPropertyBorderImageSource: {
</span><del>- auto* styleImage = style.borderImageSource();
- if (is<StylePendingImage>(styleImage))
- style.setBorderImageSource(loadPendingImage(document, *styleImage, element));
</del><ins>+ loadPendingImage(document, style.borderImageSource(), element);
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> case CSSPropertyWebkitBoxReflect: {
</span><del>- if (StyleReflection* reflection = style.boxReflect()) {
- const NinePieceImage& maskImage = reflection->mask();
- auto* styleImage = maskImage.image();
- if (is<StylePendingImage>(styleImage)) {
- auto loadedImage = loadPendingImage(document, *styleImage, element);
- reflection->setMask(NinePieceImage(WTFMove(loadedImage), maskImage.imageSlices(), maskImage.fill(), maskImage.borderSlices(), maskImage.outset(), maskImage.horizontalRule(), maskImage.verticalRule()));
- }
- }
</del><ins>+ if (auto* reflection = style.boxReflect())
+ loadPendingImage(document, reflection->mask().image(), element);
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> case CSSPropertyWebkitMaskBoxImageSource: {
</span><del>- auto* styleImage = style.maskBoxImageSource();
- if (is<StylePendingImage>(styleImage))
- style.setMaskBoxImageSource(loadPendingImage(document, *styleImage, element));
</del><ins>+ loadPendingImage(document, style.maskBoxImageSource(), element);
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> case CSSPropertyWebkitMaskImage: {
</span><del>- for (FillLayer* maskLayer = &style.ensureMaskLayers(); maskLayer; maskLayer = maskLayer->next()) {
- auto* styleImage = maskLayer->image();
- if (is<StylePendingImage>(styleImage))
- maskLayer->setImage(loadPendingImage(document, *styleImage, element));
- }
</del><ins>+ for (auto* maskLayer = &style.ensureMaskLayers(); maskLayer; maskLayer = maskLayer->next())
+ loadPendingImage(document, maskLayer->image(), element);
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> #if ENABLE(CSS_SHAPES)
</span><span class="cx"> case CSSPropertyWebkitShapeOutside: {
</span><del>- if (!style.shapeOutside())
- return;
-
- StyleImage* image = style.shapeOutside()->image();
- if (is<StylePendingImage>(image))
- style.shapeOutside()->setImage(loadPendingImage(document, *image, element, LoadPolicy::ShapeOutside));
-
</del><ins>+ if (style.shapeOutside())
+ loadPendingImage(document, style.shapeOutside()->image(), element, LoadPolicy::ShapeOutside);
</ins><span class="cx"> break;
</span><span class="cx"> }
</span><span class="cx"> #endif
</span></span></pre>
</div>
</div>
</body>
</html>