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

Unified Diff: Source/core/testing/InternalSettings.cpp

Issue 82583005: Use removeFontFace to avoid resetting fontSelector. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixed TextAutosizing Created 6 years, 10 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/testing/InternalSettings.cpp
diff --git a/Source/core/testing/InternalSettings.cpp b/Source/core/testing/InternalSettings.cpp
index ae0e04c3af8b7170e06370129e66a6a2983e29cc..71dc30f905c5f81bc61acdbdb48da414ad0d0ee9 100644
--- a/Source/core/testing/InternalSettings.cpp
+++ b/Source/core/testing/InternalSettings.cpp
@@ -232,6 +232,7 @@ void InternalSettings::setStandardFontFamily(const AtomicString& family, const S
if (code == USCRIPT_INVALID_CODE)
return;
settings()->genericFontFamilySettings().setStandard(family, code);
+ settings()->notifyGenericFontFamilyChange();
m_page->setNeedsRecalcStyleInAllFrames();
eseidel 2014/02/14 05:39:15 It seems this can just fold into the notify now?
tasak 2014/02/14 08:27:46 You are right. Removed.
}
@@ -242,6 +243,7 @@ void InternalSettings::setSerifFontFamily(const AtomicString& family, const Stri
if (code == USCRIPT_INVALID_CODE)
return;
settings()->genericFontFamilySettings().setSerif(family, code);
+ settings()->notifyGenericFontFamilyChange();
m_page->setNeedsRecalcStyleInAllFrames();
}
@@ -252,6 +254,7 @@ void InternalSettings::setSansSerifFontFamily(const AtomicString& family, const
if (code == USCRIPT_INVALID_CODE)
return;
settings()->genericFontFamilySettings().setSansSerif(family, code);
+ settings()->notifyGenericFontFamilyChange();
m_page->setNeedsRecalcStyleInAllFrames();
}
@@ -262,6 +265,7 @@ void InternalSettings::setFixedFontFamily(const AtomicString& family, const Stri
if (code == USCRIPT_INVALID_CODE)
return;
settings()->genericFontFamilySettings().setFixed(family, code);
+ settings()->notifyGenericFontFamilyChange();
m_page->setNeedsRecalcStyleInAllFrames();
}
@@ -272,6 +276,7 @@ void InternalSettings::setCursiveFontFamily(const AtomicString& family, const St
if (code == USCRIPT_INVALID_CODE)
return;
settings()->genericFontFamilySettings().setCursive(family, code);
+ settings()->notifyGenericFontFamilyChange();
m_page->setNeedsRecalcStyleInAllFrames();
}
@@ -282,6 +287,7 @@ void InternalSettings::setFantasyFontFamily(const AtomicString& family, const St
if (code == USCRIPT_INVALID_CODE)
return;
settings()->genericFontFamilySettings().setFantasy(family, code);
+ settings()->notifyGenericFontFamilyChange();
m_page->setNeedsRecalcStyleInAllFrames();
}
@@ -292,6 +298,7 @@ void InternalSettings::setPictographFontFamily(const AtomicString& family, const
if (code == USCRIPT_INVALID_CODE)
return;
settings()->genericFontFamilySettings().setPictograph(family, code);
+ settings()->notifyGenericFontFamilyChange();
m_page->setNeedsRecalcStyleInAllFrames();
}

Powered by Google App Engine
This is Rietveld 408576698