Comment #94 on issue 12530 by yusu...@chromium.org: Segfault when  
ResourceBundle::GetFont fails -- more font fallback needed?
http://code.google.com/p/chromium/issues/detail?id=12530

oleguer, thanks for the info!

> oleg...@imhotep:/etc/fonts/conf.d$ grep -r weak `find \( !  
> -regex '.*/\..*' \) -type f`
> ..
> oleg...@imhotep:/etc/fonts/conf.d$ grep -r prepend `find \( !  
> -regex '.*/\..*' \)
-type f`

Maybe "-type f" is not suitable since most of files in the directory are  
symlinks. If you simply use "grep same -r ." or something like that, some  
files should be listed. But anyway, the information you provided was  
sufficient, thanks.

> family: "Arial"(w) "DejaVu Sans Condensed"(w) "Luxi Sans"(w) "Nimbus Sans  
> L"(w)
"Bitstream Vera Sans"(s)

Our font matching function expects that "Bitstream Vera Sans" is on the top  
of the list, but in fact "Arial (weak)" is there, probably due to your  
conf.d settings. And this might be the culprit of the error. I believe the  
matching function should be improved to support this case.

agl:
I think we should treat the A-B-A case as a "good match" as well:  
http://codereview.chromium.org/192062

     requested family: A
     post_config_family: B
     post_match_family: A

> So, how about skipping the strcasecmp check below when we request a font  
> for
> BrowserUI?

This was a bad idea since we also have to handle font requests for web  
content area in the improved way above. Otherwise, oleguer can't use the  
Bitstream font in web content area.

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to