124653 - Crash on ASUS site (download area)

Status: VERIFIED WORKSFORME

(Core :: Internationalization, defect)

Description

[Dimitrios]

Build 2002020803, Win98. Entering the above download area in ASUS site crashes
Mozilla. Improper javascript syntax? Anyway, we shouldn't crash.
Reproducible: Always.
Talkback ID : 2708482G and 2708520Y

Comment 1

[Dimitrios]

An evidence for syntax error: saving the complete page with IE and running the
result on Mozilla doesn't crash. Question is, how can you debug javascript on a
page when the latter crashes Mozilla on loading?

Comment 2

[Stefan [:stefanh]]

WFM Mac OS 9.1 build 2002020405

Comment 3

[Dimitrios]

Tested today, using same build (2002020803). the first time I visited the link,
it didn't crash but, along with the normal window, it opened a new window
containing the following:
"XML Parsing Error:
Location: http://www.mozilla.org/Projects/intl/fonts/win/en/package_zh_TW.xul
Line Number 25, Column 69:"

One more attempt to visit the link, later, and it crashed. Doesn't seem to crash
on Win2K, though it behaves strangely until finishing the page loading. Changing
component, based on the above message (wild guess).

Comment 4

[timeless]

that's the font downloader page

Comment 5

[Roy Yokoyama]

Attachment: Assertion on /layout/xul/base/src/nsRootBoxFrame.cpp

I haven't got the XML parser error; but instead I got 
Assertion.  (see attached)
Talkback is down. I'll take a look of the stack trace once it's up.

Comment 6

[Roy Yokoyama]

== cc'ing hyatt for nsRootBoxFrame.cpp assertion

Comment 7

[Frank Tang]

Does this happen only on Win 98? how about WinXP win NT?

Comment 8

[Vadim Berezniker]

I don't see this on Windows XP.
Build #2002020208

Comment 9

[Yuying Long]

crash, -> nsbeta1

Comment 10

[Frank Tang]

crash is bad, nsbeta1+

Comment 11

[Roy Yokoyama]

I tried in Win98; but I can't reproduce this bug.

However, both Talkback (2708482 and 2708520) point to
nsExpatDriver::ParseBuffer
[d:\builds\seamonkey\mozilla\htmlparser\src\nsExpatDriver.cpp,line 634] 
which makes me to suspect some BytePosition problem.
mBytePosition = (XML_GetCurrentByteIndex(mExpatParser) - mBytesParsed);

harishd: can you try this bug?  
(you need to debug on NT or 9x. 
fontdownload page doesn't come up in W2k nor WinXP)

Comment 12

[Roy Yokoyama]

Attachment: resetting mBytePosition to 0 if negative

harishd: can we do something to similar to this?

Comment 13

[harishd]

We need to find out why mBytePosition was less than 0.

Comment 14

[Roy Yokoyama]

harishd: the patch is a wild guess for me. 
As comment #11, I failed to reproduce this bug on my Win98 machine.  
I would love to find out WHY mExpatParser is 0 if that's the cause of it. 
It looks like you have dealt with nsExpatDriver::ParseBuffer.
Can you try to see if you can reproduce this?

-- cc'ing nisheeth for XML_GetCurrentByteIndex() --------

Comment 15

[Nisheeth Ranjan]

Oops, wrong email address.

Comment 16

[Nisheeth Ranjan]

CCing heikki on this as well.  Please ignore earlier comment.

Comment 17

[Roy Yokoyama]

ylong: can you try this with NT?
       which OS did you use to have it crashed?

Comment 18

[Yuying Long]

I actually can not reproduce it on both Win98 and WinME neither, the reason I
nominate it was because the crash.

Reporter:
Could you please provide us more detail information for the crash?  where did
you see the crash? by loading the page:
http://download.asus.com.tw/mb_alllist.asp
or during downloading some special driver?

Comment 19

[Dimitrios]

Not seeing this anymore. Test was as follows:
1. Load above page
2. Wait for the download fonts dialog
The dialog text wasn't displayed, instead the dialog window was displayed in
full screen size for a moment, then crash. Now it works as expected and I am
able to cancel the dialog.

Comment 20

[Roy Yokoyama]

as per reporter's comment. Marking as WORKSFORME

Comment 21

[Yuying Long]

Verified.