361654 - configAutoconfigText not used anywhere

Status: VERIFIED FIXED

(Firefox :: Settings UI, defect)

Description

[Marek Stępień [:marcoos, inactive]]

These two entities in connection.dtd aren't used anywhere:

<!ENTITY  configAutoconfigText.label    "Configuration location (URL):">
<!ENTITY  configAutoconfigText.accesskey "u">

They should be removed.

Comment 1

[Marek Stępień [:marcoos, inactive]]

Attachment: Get rid of these entities

Gets rid of these entities from browser's, mail's, calendar's connection.dtd and cck's cckWizard.dtd.

Comment 2

[David :Bienvenu]

Cc'ing mkaply in case he has any insight into these strings. It certainly seems like they're not used.

Comment 3

[Mike Kaply [:mkaply]]

I can't find any history here on this.

My guess was it was going to be an attempt to implement WPAD:

http://www.craigjconsulting.com/setproxy.html

That was never done.

Comment 4

[David :Bienvenu]

Comment on attachment 246396 [details] [diff] [review]
Get rid of these entities

interesting, ok, thx, sr=bienvenu

Comment 5

[Marek Stępień [:marcoos, inactive]]

CC'ing one of the calendar developers, as this patch removes that entity from Sunbird files as well.

Do I need any more reviews for this?

Comment 6

[Joey Minta]

(In reply to comment #5)
No objection here.

Comment 7

[Marek Stępień [:marcoos, inactive]]

This is ready for checking in, then.

Comment 8

[Nickolay_Ponomarev]

browser/locales/en-US/chrome/browser/preferences/connection.dtd   1.6
calendar/locales/en-US/chrome/calendar/preferences/connection.dtd 1.4
extensions/cck/browser/locales/en-US/chrome/cckWizard.dtd         1.28
mail/locales/en-US/chrome/messenger/preferences/connection.dtd    1.3

Comment 9

[Marek Stępień [:marcoos, inactive]]

v.