12105 - [NEWS] use string bundles for news strings

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P3)

Description

[(not reading, please use [e-mail] instead)]

I need to use string bundles, instead of hard coding the news string.

Comment 1

[(not reading, please use [e-mail] instead)]

marking m10. accepting.

Comment 2

[esther]

I worked on this with Seth yesterday and here are the steps to show the
problem and to reproduce this bug:
1. Launch Messenger
2. Open a newsgroup
3. Select messages to view going down the list and clicking with pauses of about
 2 seconds or less in between clicks.
4. After about 6 messages the newly clicked messages don't display. Message view
is displaying one of the previously selected messages and it doesn't chang.

When fixed, message view shouldn't get stuck on one message.

Comment 3

[lchiang]

Esther - did you comment in the wrong bug?

Comment 4

[(not reading, please use [e-mail] instead)]

yes, esther comments are in the wrong bug.

I copied and pasted them into #8515

Comment 5

[Phil Peterson]

Triage to M11

Comment 6

[lchiang]

(target milestone is M11 or M12 - add to mail beta tracking bug)

Comment 7

[Phil Peterson]

M13

Comment 8

[David :Bienvenu]

I can take this, since I've done it before.

Comment 9

[esther]

David, when this is fixed, can you give me a test case so I can verify it.
Thanks

Comment 10

[lchiang]

Esther - don't worry about this.  Suresh/Par - can you verified via the code
when david marks this fixed?

Comment 11

[David :Bienvenu]

converted to using the existing string bundle (that was easy!)

Comment 12

[ppandit]

Examined code changes in 
mozilla/mailnews/news/src/nsNNTPProtocol.h (v 1.51)
mozilla/mailnews/news/src/nsNNTPProtocol.cpp (v 1.128)

VERIFIED

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f4fa76edb2f5
[fenix] For https://github.com/mozilla-mobile/fenix/issues/12105: Follow ux mock on Settings --> Accessibility screen