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

Unified Diff: core/fxge/win32/fx_win32_device.cpp

Issue 2395883002: Use another vector for installed fonts in CFX_FontMapper (Closed)
Patch Set: Nits Created 4 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
« no previous file with comments | « core/fxge/ge/cfx_fontmapper.cpp ('k') | fpdfsdk/cfx_systemhandler.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: core/fxge/win32/fx_win32_device.cpp
diff --git a/core/fxge/win32/fx_win32_device.cpp b/core/fxge/win32/fx_win32_device.cpp
index bc6817da63cd6ad2bd5d7f69b49cb988da7d6451..0ef78295493153d00f2e3d8b54af46a185269e20 100644
--- a/core/fxge/win32/fx_win32_device.cpp
+++ b/core/fxge/win32/fx_win32_device.cpp
@@ -471,13 +471,13 @@ CFX_ByteString CFX_Win32FontInfo::FindFont(const CFX_ByteString& name) {
for (size_t i = 0; i < m_pMapper->m_InstalledTTFonts.size(); ++i) {
CFX_ByteString thisname = m_pMapper->m_InstalledTTFonts[i];
- if (thisname[0] == ' ') {
- if (thisname.Mid(1, name.GetLength()) == name) {
- return m_pMapper->m_InstalledTTFonts[i + 1];
- }
- } else if (thisname.Left(name.GetLength()) == name) {
+ if (thisname.Left(name.GetLength()) == name)
return m_pMapper->m_InstalledTTFonts[i];
- }
+ }
+ for (size_t i = 0; i < m_pMapper->m_LocalizedTTFonts.size(); ++i) {
+ CFX_ByteString thisname = m_pMapper->m_LocalizedTTFonts[i].first;
+ if (thisname.Left(name.GetLength()) == name)
+ return m_pMapper->m_LocalizedTTFonts[i].second;
}
return CFX_ByteString();
}
« no previous file with comments | « core/fxge/ge/cfx_fontmapper.cpp ('k') | fpdfsdk/cfx_systemhandler.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698