Closed Bug 629893 Opened 9 years ago Closed 8 years ago
.policy .label" [ @ cookie Accept Dialog .dtd ] is unused and should be removed
Entity "props.policy.label" is unused - it should be removed https://mxr.mozilla.org/mozilla-central/source/toolkit/locales/en-US/chrome/cookie/cookieAcceptDialog.dtd#7 Found in cookieAcceptDialog.dtd file only: https://mxr.mozilla.org/mozilla-central/search?string=props.policy.label&find=%2F&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central There is no mentioned entity in cookieAcceptDialog.xul: https://mxr.mozilla.org/mozilla-central/source/toolkit/components/cookie/content/cookieAcceptDialog.xul
Attachment #508117 - Flags: review? → review?(gavin.sharp)
Attachment #508117 - Flags: review?(gavin.sharp) → review+
This seems to be a fall-out from unreviewed parts landing as part of http://hg.mozilla.org/mozilla-central/rev/b19ea8c3d39c from bug 383993. This is looong gone, doesn't hurt to carry it around for a few more weeks. Let's land this post-fx4.
Depends on: 383993
(In reply to Axel Hecht [:Pike] from comment #2) > Let's land this post-fx4. It's time.
Can you create an updated patch following the guidelines on http://blog.bonardo.net/2010/06/22/so-youre-about-to-use-checkin-needed and set the checkin-needed flag here?
I forgot about the comment in patch.
Attachment #588181 - Attachment is obsolete: true
Target Milestone: --- → mozilla12
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.