98222 - Call to ToLowerCase() seems to be expensive.

Status: VERIFIED DUPLICATE of bug 96529

(Core :: Internationalization, defect)

Description

[harishd]

Sample quantify run loading about:blank.

Function:
nsCharsetAlias2::GetPreferred
Calls:
        414
Function time:	38.83 usec (0.00% of Focus)
F+D time:	840,726.55 usec (0.06% of Focus)
Avg F time:	0.09 usec
Min F time:	0.09 usec
Max F time:	0.18 usec
Module:
y:\mozilla\dist\WIN32_O.OBJ\bin\components\uconv.dll
Source File:	y:\mozilla\intl\uconv\src\nsCharsetAliasImp.cpp
Measurement:
Line
Hidden functions:	(None)

source:
NS_IMETHODIMP nsCharsetAlias2::GetPreferred(const nsString& aAlias, nsString&
oResult)				
{
  nsAutoString aKey;
  aAlias.ToLowerCase(aKey);

Comment 1

[Roy Yokoyama]

ftang: can we remove the ToLowCase()?  What are the implications?

=== assigning to ftang and cc'ing yokoyama ==============

Comment 2

[Frank Tang]

*** This bug has been marked as a duplicate of 96529 ***

Comment 3

[Andreas Becker]

Verified as such. Please reopen in case someone diagrees.