1308503 - Migrate localization strings away from certManager.dtd

Status: VERIFIED FIXED

(DevTools :: Netmonitor, defect, P1)

Description

[Ricky Chien [:rickychien] (inactive)]

Migrate localization strings as same as bug 1308500, so there should be certManager.properties.

certManager.dtd is different with netmonitor.dtd since it is located in security folder but not devtools. We might need make a copy to not break other references.

Comment 1

[Jan Honza Odvarko [:Honza] (always need-info? me)]

From IRC:

Honza	The Netpanel is using certManager.dtd. As part of netmonitor.html all strings should be moved into *.properties, but this DTD isn't controlled by DevTools. Should we just dup these strings?
jsnajdr	Honza: it seems that Firefox for Android had this problem, too, and ended up duplicating the strings: http://searchfox.org/mozilla-central/source/mobile/android/locales/en-US/chrome/pippki.properties#63
Honza	jsnajdr: ah, good, sounds like we could do the same


Honza

Comment 2

[[e-mail]]

Keeler, the certManager.dtd is used in devtools/netmonitor now, we plan to deXUL netmonitor, any files relying on DTDs for localization should move all the DTD entities to properties files.

Therefore we need create a counterpart for certManager.dtd. Here are two questions:

1. Do you think it's proper to host another cert-manger.properties file in netmonitor or in /security? Since currently this .properties is only used in devtools.
2. We only use small portion of certManager.dtd strings(~8), I'd prefer just copy needed strings rather than migrate whole .dtd if its fine to host another cert-manger.properties file in netmonitor


FYR, to check certManager related string in NetMonitor:

1. open yahoo.com (https)
2. Menu > Developer > Network
3. select a row, switch sidebar tab to `security` tab's certificate section

Comment 3

[[e-mail]]

the question is in comment 2

Comment 4

[Dana Keeler (she/her) (use needinfo) [:keeler] (on leave)]

1. I don't have a lot of experience with l10n, but my intuition is it's best to have the strings located near to where they're used. Going by that, it would be best to put the new file in netmonitor.
2. I think it would be simplest to just use what you need (and remove any strings that aren't being used anywhere else).

Comment 5

[[e-mail]]

Attachment: Bug 1308503 - Migrate localization strings away from certManager.dtd;

Review commit: https://reviewboard.mozilla.org/r/85426/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/85426/

Comment 6

[[e-mail]]

The netmonitor.properties is already exist and used by webconsole, netmonitor-view, and chart.jsm
The patch attached 8 certManager related string from certManager.dtd and ask panel render it with netmonitor.xul.

Comment 7

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Comment on attachment 8800510 [details]
Bug 1308503 - Migrate localization strings away from certManager.dtd;

https://reviewboard.mozilla.org/r/85426/#review84108

Nice!

R+ assuming all tests on Try are green.

Honza

Comment 8

[[e-mail]]

Comment on attachment 8800510 [details]
Bug 1308503 - Migrate localization strings away from certManager.dtd;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/85426/diff/1-2/

Comment 9

[[e-mail]]

treeherder green, thanks!

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/30137ab463a3
Migrate localization strings away from certManager.dtd; r=Honza

Comment 11

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/30137ab463a3

Comment 12

[Ciprian Georgiu, Desktop QA]

This issue is verified fixed on latest Nightly 52.0a1 (2016-10-23) under the following OSes:
- Windows 10 x64
- Ubuntu 16.04 x64
- macOS 10.12.1 Beta

certManager related string functionality is intact and l10n of the Security tab is not broken. Marking here accordingly.