Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1546)

Unified Diff: Source/core/layout/svg/SVGResources.cpp

Issue 1302713003: Don't register excessive pending SVG resources (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/layout/svg/LayoutSVGResourceContainer.cpp ('k') | Source/core/layout/svg/SVGResourcesCache.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/layout/svg/SVGResources.cpp
diff --git a/Source/core/layout/svg/SVGResources.cpp b/Source/core/layout/svg/SVGResources.cpp
index 4430187997bc71eeaaba9700341651514ec75cc2..6566b0014de61cb5ecf5027f75f50ad5f1c4606d 100644
--- a/Source/core/layout/svg/SVGResources.cpp
+++ b/Source/core/layout/svg/SVGResources.cpp
@@ -374,56 +374,40 @@ void SVGResources::resourceDestroyed(LayoutSVGResourceContainer* resource)
case MaskerResourceType:
if (!m_clipperFilterMaskerData)
break;
- if (m_clipperFilterMaskerData->masker == resource) {
- m_clipperFilterMaskerData->masker->removeAllClientsFromCache();
+ if (m_clipperFilterMaskerData->masker == resource)
m_clipperFilterMaskerData->masker = nullptr;
- }
break;
case MarkerResourceType:
if (!m_markerData)
break;
- if (m_markerData->markerStart == resource) {
- m_markerData->markerStart->removeAllClientsFromCache();
+ if (m_markerData->markerStart == resource)
m_markerData->markerStart = nullptr;
- }
- if (m_markerData->markerMid == resource) {
- m_markerData->markerMid->removeAllClientsFromCache();
+ if (m_markerData->markerMid == resource)
m_markerData->markerMid = nullptr;
- }
- if (m_markerData->markerEnd == resource) {
- m_markerData->markerEnd->removeAllClientsFromCache();
+ if (m_markerData->markerEnd == resource)
m_markerData->markerEnd = nullptr;
- }
break;
case PatternResourceType:
case LinearGradientResourceType:
case RadialGradientResourceType:
if (!m_fillStrokeData)
break;
- if (m_fillStrokeData->fill == resource) {
- m_fillStrokeData->fill->removeAllClientsFromCache();
+ if (m_fillStrokeData->fill == resource)
m_fillStrokeData->fill = nullptr;
- }
- if (m_fillStrokeData->stroke == resource) {
- m_fillStrokeData->stroke->removeAllClientsFromCache();
+ if (m_fillStrokeData->stroke == resource)
m_fillStrokeData->stroke = nullptr;
- }
break;
case FilterResourceType:
if (!m_clipperFilterMaskerData)
break;
- if (m_clipperFilterMaskerData->filter == resource) {
- m_clipperFilterMaskerData->filter->removeAllClientsFromCache();
+ if (m_clipperFilterMaskerData->filter == resource)
m_clipperFilterMaskerData->filter = nullptr;
- }
break;
case ClipperResourceType:
if (!m_clipperFilterMaskerData)
break;
- if (m_clipperFilterMaskerData->clipper == resource) {
- m_clipperFilterMaskerData->clipper->removeAllClientsFromCache();
+ if (m_clipperFilterMaskerData->clipper == resource)
m_clipperFilterMaskerData->clipper = nullptr;
- }
break;
default:
ASSERT_NOT_REACHED();
« no previous file with comments | « Source/core/layout/svg/LayoutSVGResourceContainer.cpp ('k') | Source/core/layout/svg/SVGResourcesCache.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698