168733 - nsPSMDetector::HandleData takes about 1.1% of page load time

Status: RESOLVED WONTFIX

(Core :: Internationalization, defect)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I just ran a page load profile on a list of 40 live URLs on the web (see
attachment 88207 [details] for the approximate methods used).  In the profile I ran on a
2002-09-14 build, nsPSMDetector::HandleData (in mozilla/intl/chardet/src/) takes
about 1.1% of page load time.  In the profile I ran on a 2002-08-06 build, it
was not present at all.

There are a number of possibilities:
 * This is a recent regression
 * This is a problem only on uncached loads (and thus closer to 2% of uncached
loads, since I did two loops through my URL list)
 * This is a problem only on a specific page within the list of URLs (and thus
closer to 40% of the time for loading a single URL).

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: profile of time spent within nsPSMDetector::HandleData

This is a segment of a profile that shows only the time spent within
nsPSMDetector::HandleData.

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Hmmm.  So this doesn't show up in a clean profile.  Perhaps the problem is that
I was at one point a few weeks/months ago looking at a Japanese page that
required View | Character Coding | Auto Detect | Japanese, and that's been set
ever since.  Nevertheless, this seems like something that should be fixed even
if it isn't in the default profile.

Comment 3

[Roy Yokoyama]

-> shanjian

Comment 4

[Frank Tang]

shanjian is no longer working on mozilla for 2 years and these bugs are still
here. Mark them won't fix. If you want to reopen it, find a good owner first. 

Comment 5

[Travis Chase]

Mass Re-assigning bugs that Frank Tang Closed on March 1st Spam is his fault

Mass Re-Open to follow

Comment 6

[Travis Chase]

Mass Re-open of Frank Tangs Won't fix debacle. Spam is his responsibility not my own

Comment 7

[Travis Chase]

Reassigning Franks old bugs to Jungshik Shin for triage - Sorry for spam

Comment 8

[Simon Montagu :smontagu]

Source file no longer exists (bug 426271)