Closed
Bug 866632
Opened 8 years ago
Closed 8 years ago
[gcli] |cookie list| on about page shows 'no cookies found for host', should use 'No cookies found for this page'
Categories
(DevTools :: Console, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 23
People
(Reporter: aryx, Assigned: jwalker)
Details
(Keywords: l12y)
Attachments
(1 file)
1.12 KB,
patch
|
harth
:
review+
|
Details | Diff | Splinter Review |
Nightly and Aurora, Windows XP SP3 Opening the gcli while having about:home (or about:blank) open and calling |cookie list| shows 'No cookies found for host'. If look at http://mxr.mozilla.org/mozilla-central/source/browser/locales/en-US/chrome/browser/devtools/gclicommands.properties#774 , you can see that this fails gracefully in English because the host value is at end of the string and blank, but this will likely cause trouble for other localizations. cookieListOutNonePage doesn't seem to be used despite the description. # LOCALIZATION NOTE (cookieListOutNonePage) The output of the 'cookie list' # command uses this string for pages like 'about:blank' which can't contain # cookies cookieListOutNonePage=No cookies found for this page # LOCALIZATION NOTE (cookieListOutNoneHost) The output of the 'cookie list' # command uses this string when there are no cookies on a given web page cookieListOutNoneHost=No cookies found for host %1$S
Assignee | ||
Comment 1•8 years ago
|
||
The cookieListOutNonePage string is used here http://mxr.mozilla.org/mozilla-central/source/browser/devtools/commandline/BuiltinCommands.jsm#980 The "cookie list" command allows the user to specify a host (the host of the current page is taken as the default) so it's wrong to assume the current page. We'll have to do something special in the case where the host is null.
Assignee | ||
Comment 2•8 years ago
|
||
Updated•8 years ago
|
Attachment #746962 -
Flags: review?(fayearthur) → review+
Assignee | ||
Comment 3•8 years ago
|
||
Green on try: https://tbpl.mozilla.org/?tree=Try&rev=7ac21da57bbe
Assignee | ||
Comment 4•8 years ago
|
||
https://tbpl.mozilla.org/?tree=Fx-Team&rev=9ae9cc383d0f https://hg.mozilla.org/integration/fx-team/rev/e55e6114c0e4
Whiteboard: [fixed-in-fx-team]
Assignee | ||
Comment 5•8 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e55e6114c0e4
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 23
Updated•3 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•