Remove duplicate string in about:url-classifier (locales/en-US/chrome/global/aboutUrlClassifier.dtd)

RESOLVED FIXED in Firefox 55

Status

()

Toolkit
Safe Browsing
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: flod, Assigned: flod)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

a year ago
(In reply to Francesco Lodolo [:flod] from comment #17)
> (In reply to Ton from comment #16)
> > (In reply to Carsten Book [:Tomcat] from comment #15)
> > > https://hg.mozilla.org/mozilla-central/rev/261fe706dd83
> > > +<!ENTITY aboutUrlClassifier.cacheTitle                  "Cache">
> > 
> > Note that aboutUrlClassifier.cacheTitle was already present in the file so
> > now appears twice.
> 
> We need to get rid quickly of this string, i.e I can't expose strings safely
> to localization tools with a duplicate like this.
> 
> On a side note, can you start looping me in in bugs that land strings for
> this feature? Some of the strings you're landing could benefit from
> localization tools.
(Assignee)

Comment 1

a year ago
> On a side note, can you start looping me in in bugs that land strings for
> this feature? Some of the strings you're landing could benefit from
> localization tools.

I meant "localization notes", but I was also mixing about:url-classifier with about:debugging.
(Assignee)

Comment 2

a year ago
Actually, unassigning for now.

@François
Can you explain why a group of strings landed in bug 1348253 but are completely unused?
https://hg.mozilla.org/mozilla-central/rev/8ce1a738538e

aboutUrlClassifier.cacheTitle is one of them. It landed as a section title, judging by the screenshots, by it's not used.
Assignee: francesco.lodolo → nobody
Flags: needinfo?(francois)
Summary: Remove duplicate string in about:url-classifier, add localization comments → Remove duplicate string in about:url-classifier (locales/en-US/chrome/global/aboutUrlClassifier.dtd)
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 5

a year ago
OK, I've realized just after sending the first version that this is actually a title (also, Nightly updated in the meantime). The patch only removes the duplicate.

I guess titles for sections were pre-landed, but that was missed in bug 1360480?
Assignee: nobody → francesco.lodolo
Flags: needinfo?(francois)
(In reply to Francesco Lodolo [:flod] from comment #5)
> OK, I've realized just after sending the first version that this is actually
> a title (also, Nightly updated in the meantime). The patch only removes the
> duplicate.
> 
> I guess titles for sections were pre-landed, but that was missed in bug
> 1360480?

Yes, those titles for sections were landed in bug 1348253 and they should be removed since we don't use it now.
I am sorry I just forget to remove them in that bug.

Thanks for fixing this issue, and do you want me to create a bug to remove those unused ones?

Comment 7

a year ago
mozreview-review
Comment on attachment 8870415 [details]
Bug 1367000 - Remove duplicate string ID in aboutUrlClassifier.dtd

https://reviewboard.mozilla.org/r/141866/#review145762
Attachment #8870415 - Flags: review?(francois) → review+
(Assignee)

Comment 8

a year ago
(In reply to Dimi Lee[:dimi][:dlee] from comment #6)
> Thanks for fixing this issue, and do you want me to create a bug to remove
> those unused ones?

Yes please, unless you're planning to use them very shortly (like it happened for this one).

Comment 9

a year ago
Pushed by francesco.lodolo@mozillaitalia.org:
https://hg.mozilla.org/integration/autoland/rev/9984c908c90c
Remove duplicate string ID in aboutUrlClassifier.dtd r=francois
(In reply to Francesco Lodolo [:flod] from comment #8)
> Yes please, unless you're planning to use them very shortly (like it
> happened for this one).

Filed Bug 1367324

Comment 11

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9984c908c90c
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.