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

Unified Diff: Source/core/css/CSSSegmentedFontFace.cpp

Issue 18882002: [oilpan] Move CSSSegmentedFontFace to the managed heap (Closed) Base URL: svn://svn.chromium.org/blink/branches/oilpan
Patch Set: Created 7 years, 5 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
Index: Source/core/css/CSSSegmentedFontFace.cpp
diff --git a/Source/core/css/CSSSegmentedFontFace.cpp b/Source/core/css/CSSSegmentedFontFace.cpp
index 420368a25a25c494eb59d8f062b729abbc51ba01..bc1e75df2c2eea58bc067818e4486a80221a0a8a 100644
--- a/Source/core/css/CSSSegmentedFontFace.cpp
+++ b/Source/core/css/CSSSegmentedFontFace.cpp
@@ -37,6 +37,8 @@
namespace WebCore {
+DEFINE_GC_TYPE_MARKER(CSSSegmentedFontFace);
+
CSSSegmentedFontFace::CSSSegmentedFontFace(CSSFontSelector* fontSelector)
: m_fontSelector(fontSelector)
{
@@ -47,7 +49,7 @@ CSSSegmentedFontFace::~CSSSegmentedFontFace()
pruneTable();
unsigned size = m_fontFaces.size();
for (unsigned i = 0; i < size; i++)
- m_fontFaces[i]->removedFromSegmentedFontFace(this);
+ m_fontFaces[i]->removedFromSegmentedFontFace(Handle<CSSSegmentedFontFace>(this));
}
void CSSSegmentedFontFace::pruneTable()
@@ -79,9 +81,9 @@ void CSSSegmentedFontFace::fontLoaded(CSSFontFace*)
m_callbacks.swap(callbacks);
for (size_t index = 0; index < callbacks.size(); ++index) {
if (checkFont())
- callbacks[index]->notifyLoaded(this);
+ callbacks[index]->notifyLoaded(Handle<CSSSegmentedFontFace>(this));
else
- callbacks[index]->notifyError(this);
+ callbacks[index]->notifyError(Handle<CSSSegmentedFontFace>(this));
}
}
}
@@ -89,7 +91,7 @@ void CSSSegmentedFontFace::fontLoaded(CSSFontFace*)
void CSSSegmentedFontFace::appendFontFace(PassRefPtr<CSSFontFace> fontFace)
{
pruneTable();
- fontFace->addedToSegmentedFontFace(this);
+ fontFace->addedToSegmentedFontFace(Handle<CSSSegmentedFontFace>(this));
m_fontFaces.append(fontFace);
}
@@ -175,9 +177,9 @@ void CSSSegmentedFontFace::loadFont(const FontDescription& fontDescription, Pass
if (isLoading())
m_callbacks.append(callback);
else if (checkFont())
- callback->notifyLoaded(this);
+ callback->notifyLoaded(Handle<CSSSegmentedFontFace>(this));
else
- callback->notifyError(this);
+ callback->notifyError(Handle<CSSSegmentedFontFace>(this));
}
}

Powered by Google App Engine
This is Rietveld 408576698