Closed
Bug 190167
Opened 22 years ago
Closed 22 years ago
Cookie details empty
Categories
(Core :: Networking: Cookies, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: buecherl, Assigned: darin.moz)
References
()
Details
Attachments
(1 file)
1.83 KB,
patch
|
Biesinger
:
review+
darin.moz
:
superreview+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3b) Gecko/20030122 Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3b) Gecko/20030122 Detailed cookie informations are empty in the current nightly build Reproducible: Always Steps to Reproduce: 1.Go to a cookie site, pref set to 'ask before storing a cookie' 2.click on 'Show details' 3.fields are empty Actual Results: No cookie information Expected Results: Detailed cookie information should be shown COuld reproduce it with a clean profile
Comment 1•22 years ago
|
||
Confirmed with build 2003012210 on windows XP
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Comment 3•22 years ago
|
||
Apparently, the spaces are not allowed in the .properties file. But why did it work in my local build?
Updated•22 years ago
|
Attachment #112339 -
Flags: superreview?(darin)
Attachment #112339 -
Flags: review?(cbiesinger)
Updated•22 years ago
|
Attachment #112339 -
Flags: review?(cbiesinger) → review+
Assignee | ||
Comment 4•22 years ago
|
||
Comment on attachment 112339 [details] [diff] [review] fix properties file sr=darin
Attachment #112339 -
Flags: superreview?(darin) → superreview+
Comment 5•22 years ago
|
||
Comment on attachment 112339 [details] [diff] [review] fix properties file Requesting approval for 1.3b. This is a low risk patch, that fixes a quite visible bug.
Attachment #112339 -
Flags: approval1.3b?
The patch in bug 190256 fixes this.
Comment 7•22 years ago
|
||
to quote bz in bug 190085 comment 10: > KaiRo, just thought you'd like to know that spaces used to be fine in those > places till alecf's recent checkin.... And will be again once bug 190256 is fixed. > > I say still land this patch, but this should make it clear why it worked when > you tested it before checking in... ;) I guess the same applies here...
Depends on: 190256
*** Bug 190315 has been marked as a duplicate of this bug. ***
Comment 9•22 years ago
|
||
Darin, is this still needed or did the fix for this bug land from bug 190256 as RKAa says?
Assignee | ||
Comment 10•22 years ago
|
||
asa: i'm not sure. it's possible that that was the real cause of this bug. mvl? what say you?
Comment 11•22 years ago
|
||
With 2003012308 it works for me (stringbundle fix is in there).
Comment 12•22 years ago
|
||
...and with 2003012208 it does not work, so it seems quite like 190256 really did fix this.
Comment 13•22 years ago
|
||
It works for me too now, so closing this bug.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Comment 14•22 years ago
|
||
Still here on linux/2003012408...
Updated•22 years ago
|
Attachment #112339 -
Flags: approval1.3b?
You need to log in
before you can comment on or make changes to this bug.
Description
•