80756 - Font list is empty after selecting a font in Pref->Font dlg

Status: VERIFIED FIXED

(SeaMonkey :: Preferences, defect)

Description

[Roy Yokoyama]

As per nhotta's request, I am file this bug.

========================= email ===============================

Shanjian,

I asked Roy to file a bug for this.
This is very bad (could be a blocker).
Please look at this ASAP.

Thanks,
Naoki

Roy Yokoyama wrote:

Hi Shanjian

I noticed the same while I am working on my bug (69117)

Initially I thought I could fix your bug by
moving the gInitializedGlobalFonts variable
to global scope and make it calls the enumProc() again.
But it didn't work.  Something else is broken.....

Just to let you know.

Roy
x4178

Naoki Hotta wrote:

Shanjian,

Looks like your checkin caused a regression.
Go to Pref->Font and change Western to something else.
Then close the pref dialog and invoke pref dialog again.
You cannot select anything for everything.
Please look at this ASAP.

Naoki

<http://bugzilla.mozilla.org/show_bug.cgi?id=74899>

Comment 1

[nhottanscp]

Confirm and cc to IQA, added 'regresssion'

Comment 2

[Andreas Becker]

Adding keyword intl and nsbeta1.  

This looks critical for international, respectfully requesting a target 
milestone m0.9.1.

Comment 3

[Teruko Kobayashi]

I can reproduce this in 200151494 Win32 build on Winnt 4.0J. Changed QA contact
to [e-mail].

Comment 4

[rbs]

I think the fix is to reset 'gInitializedFontEnumerator = 0' as well, next to:
189     gInitializedFontMaps = 0;

i.e., after shifting this declaration to the beginning of the file:
4404 static int gInitializedFontEnumerator = 0;

Comment 5

[sairuh (rarely reading bugmail)]

->all.

Comment 6

[Shanjian Li]

I am going to back out changes in 74899 to fix this. I think I have a more complete 
fix ready for 74899 without causing this problem. My local tree seems have some 
problem, I need to wait to a cleaner build to test my fix.

Comment 7

[Shanjian Li]

74899 fix has been backed out.

Comment 8

[Yuying Long]

Verified this Font list is empty issue has resolved on 05-18 trunk build.