Non-ascii titles are displayed as garbage during session restore

VERIFIED FIXED in Firefox 11

Status

()

Firefox for Android
General
P3
normal
VERIFIED FIXED
6 years ago
5 years ago

People

(Reporter: glandium, Assigned: m_kato)

Tracking

unspecified
Firefox 12
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox11 fixed, firefox12+ fixed, fennec11+)

Details

(URL)

Attachments

(3 attachments)

(Reporter)

Description

6 years ago
Created attachment 585808 [details]
Title as mojibake

STR:
- Go to http://ja.m.wikipedia.org/wiki/%E3%83%9E%E3%83%89%E3%83%AC%E3%83%BC%E3%83%8C%E5%AF%BA%E9%99%A2
- Start other applications such that Nightly is OOM killed
- Start Nightly.

What happens:
- Title is shown in japanese.
- Title is then shown as mojibake.
- Title is shown in japanese again.

I'll be attaching screenshots for 2 and 3.
(Reporter)

Comment 1

6 years ago
Created attachment 585810 [details]
Title in japanese

Updated

6 years ago
Assignee: nobody → bnicholson
tracking-firefox12: --- → +
Priority: -- → P3
tracking-fennec: --- → 11+
(Assignee)

Comment 2

6 years ago
Created attachment 589418 [details] [diff] [review]
fix

readInputStreamToString handles us-ascii.  We should use 3rd option to read as UTF-8.
(Assignee)

Updated

6 years ago
Attachment #589418 - Flags: review?(bnicholson)
Comment on attachment 589418 [details] [diff] [review]
fix

Looks fine to me.
Attachment #589418 - Flags: review?(bnicholson) → review+
Assignee: bnicholson → m_kato
(Assignee)

Comment 4

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/1fd3d6234686
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/1fd3d6234686
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → Firefox 12
Comment on attachment 589418 [details] [diff] [review]
fix

[Approval Request Comment]
Regression caused by (bug #): 
User impact if declined: 
Unicode chars not rendered correctly
Testing completed (on m-c, etc.): 
Risk to taking this patch (and alternatives if risky):
very low, just specifies the encoding
Attachment #589418 - Flags: approval-mozilla-aurora?

Comment 7

6 years ago
Comment on attachment 589418 [details] [diff] [review]
fix

[Triage Comment]
Mobile only - approved for Aurora.
Attachment #589418 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/ba758404a5ad
status-firefox11: --- → fixed
status-firefox12: --- → fixed

Comment 9

5 years ago
Verified fixed on:
-build: Firefox for Android 22.0a1 (2013-02-27)
-device: Samsung Galaxy Nexus
-OS: Android 4.2.2
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.