Commit 911b38ef authored by kouhei@chromium.org's avatar kouhei@chromium.org

Revert of Oilpan: Trace CSSFontFaceSrcValue::m_svgFontFaceElement....

Revert of Oilpan: Trace CSSFontFaceSrcValue::m_svgFontFaceElement. (https://codereview.chromium.org/333403004/)

Reason for revert:
This patch leaks svg-font-face-leak-document test

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20Blink&tests=svg/svg-font-face-leak-document.html

Original issue's description:
> Oilpan: Trace CSSFontFaceSrcValue::m_svgFontFaceElement.
> 
> BUG=357163
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176270

BUG=357163

Review URL: https://codereview.chromium.org/337703004

git-svn-id: svn://svn.chromium.org/blink/trunk@176299 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent ced0d0ef
...@@ -39,14 +39,6 @@ ...@@ -39,14 +39,6 @@
namespace WebCore { namespace WebCore {
void CSSFontFaceSrcValue::traceAfterDispatch(Visitor* visitor)
{
#if ENABLE(SVG_FONTS)
visitor->trace(m_svgFontFaceElement);
#endif
CSSValue::traceAfterDispatch(visitor);
}
#if ENABLE(SVG_FONTS) #if ENABLE(SVG_FONTS)
bool CSSFontFaceSrcValue::isSVGFontFaceSrc() const bool CSSFontFaceSrcValue::isSVGFontFaceSrc() const
{ {
......
...@@ -73,7 +73,7 @@ public: ...@@ -73,7 +73,7 @@ public:
bool equals(const CSSFontFaceSrcValue&) const; bool equals(const CSSFontFaceSrcValue&) const;
void traceAfterDispatch(Visitor*); void traceAfterDispatch(Visitor* visitor) { CSSValue::traceAfterDispatch(visitor); }
private: private:
CSSFontFaceSrcValue(const String& resource, bool local) CSSFontFaceSrcValue(const String& resource, bool local)
...@@ -81,7 +81,7 @@ private: ...@@ -81,7 +81,7 @@ private:
, m_resource(resource) , m_resource(resource)
, m_isLocal(local) , m_isLocal(local)
#if ENABLE(SVG_FONTS) #if ENABLE(SVG_FONTS)
, m_svgFontFaceElement(nullptr) , m_svgFontFaceElement(0)
#endif #endif
{ {
} }
...@@ -97,7 +97,7 @@ private: ...@@ -97,7 +97,7 @@ private:
ResourcePtr<FontResource> m_fetched; ResourcePtr<FontResource> m_fetched;
#if ENABLE(SVG_FONTS) #if ENABLE(SVG_FONTS)
RawPtrWillBeMember<SVGFontFaceElement> m_svgFontFaceElement; SVGFontFaceElement* m_svgFontFaceElement;
#endif #endif
}; };
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment