[Bug 222635]
Michal-novotny
222635 at bugs.launchpad.net
Thu Nov 24 07:32:36 UTC 2011
Comment on attachment 571265
Part 2. "301: <encoding>" works on nsIndexedToHTML correctly v2
> +nsresult
> +nsIndexedToHTML::OnHeaderAvailable(nsIRequest* request, nsISupports *aContext,
> + nsString& aBuffer)
I think the name of the method is confusing. A name like WriteHeader
would be IMO more appropriate. You could also move the check of
mWroteHeader inside the method.
> + bool mWroteHeader;
This member isn't initialized in the constructor.
> + bool failbackCharset = encoding.Equals("ISO-8859-1");
s/failback/fallback/ ?
> if (mExpectAbsLoc &&
> - NS_SUCCEEDED(net_ExtractURLScheme(utf8UnEscapeSpec, nsnull, nsnull, nsnull))) {
> + NS_SUCCEEDED(net_ExtractURLScheme(convertedUnEscapeSpec, nsnull, nsnull, nsnull))) {
> // escape as absolute.
> - escFlags = esc_Forced | esc_OnlyASCII | esc_AlwaysCopy | esc_Minimal;
> + escFlags |= esc_Forced | esc_AlwaysCopy | esc_Minimal;
> }
As far as I can see, this can be removed completely. mExpectAbsLoc was
set to true only in case of gopher and support for this protocol was
removed.
--
You received this bug notification because you are a member of Mozilla
Bugs, which is subscribed to Mozilla.
https://bugs.launchpad.net/bugs/222635
Title:
firefox ftp client don't understand links in cyrillic letters
To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/222635/+subscriptions
More information about the Ubuntu-mozillateam-bugs
mailing list