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

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

Issue 656913006: Remove SVG fonts (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Update tests for landing Created 6 years, 2 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/CSSFontFaceSrcValue.cpp
diff --git a/Source/core/css/CSSFontFaceSrcValue.cpp b/Source/core/css/CSSFontFaceSrcValue.cpp
index d390caac2bb6d637e7213536a911281aecd775d6..bc03ece0ad5e439a5d6f34519a5cce3c5df0aa7a 100644
--- a/Source/core/css/CSSFontFaceSrcValue.cpp
+++ b/Source/core/css/CSSFontFaceSrcValue.cpp
@@ -33,20 +33,12 @@
#include "core/fetch/FetchRequest.h"
#include "core/fetch/FontResource.h"
#include "core/fetch/ResourceFetcher.h"
-#include "core/svg/SVGFontFaceElement.h"
#include "platform/fonts/FontCache.h"
#include "platform/fonts/FontCustomPlatformData.h"
#include "wtf/text/StringBuilder.h"
namespace blink {
-#if ENABLE(SVG_FONTS)
-bool CSSFontFaceSrcValue::isSVGFontFaceSrc() const
-{
- return equalIgnoringCase(m_format, "svg");
-}
-#endif
-
bool CSSFontFaceSrcValue::isSupportedFormat() const
{
// Normally we would just check the format, but in order to avoid conflicts with the old WinIE style of font-face,
@@ -61,16 +53,6 @@ bool CSSFontFaceSrcValue::isSupportedFormat() const
if (FontCustomPlatformData::supportsFormat(m_format))
return true;
- // We have removed SVG font support on non-gdi platforms. For details, see:
- // https://groups.google.com/a/chromium.org/d/msg/blink-dev/pYbbUcYvlYY/LQvFvM8KZZEJ
-#if ENABLE(SVG_FONTS)
- if (RuntimeEnabledFeatures::svgFontsOnNonGDIPlatformsEnabled()
-#if OS(WIN)
- || !FontCache::useDirectWrite()
-#endif
- )
- return isSVGFontFaceSrc();
-#endif
return false;
}

Powered by Google App Engine
This is Rietveld 408576698