"Save Changes" in "Manage Data" dialog in preferences doesn't close the dialog
Categories
(Firefox :: Preferences, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox72 | --- | fixed |
People
(Reporter: red-fox0, Assigned: stensonj, Mentored)
Details
Attachments
(1 file)
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:70.0) Gecko/20100101 Firefox/70.0
Steps to reproduce:
- about:preferences#privacy
- click "Manage Data…" in section "Cookies and Site Data".
- do nothing and delete nothing (!)
- click "Save Change"
Actual results:
Widget did not close.
Expected results:
Closed widget
Comment 1•3 years ago
|
||
Bugbug thinks this bug should belong to this component, but please revert this change in case of error.
Not a regression.
Updated•3 years ago
|
Comment 4•3 years ago
|
||
Huh, good question, I would say it should close. Seems like a good mentored bug. Not aware of any dupes, but maybe I'm wrong.
Assignee | ||
Comment 5•3 years ago
|
||
Hi there, I would like to take on this bug. I am new to debugging mozilla code and also new to bugzilla so I may need some guidance completing it if that is okay. Could you give me an idea as to which file I would need to edit to fix this bug? Thanks
Comment 6•3 years ago
|
||
(In reply to Jacob Stenson from comment #5)
Hi there, I would like to take on this bug. I am new to debugging mozilla code and also new to bugzilla so I may need some guidance completing it if that is okay. Could you give me an idea as to which file I would need to edit to fix this bug? Thanks
Best needinfo the mentor rather than the reporter. It so happens Johann is out, so I'll step in for the moment.
In https://searchfox.org/mozilla-central/rev/e7c61f4a68b974d5fecd216dc7407b631a24eb8f/browser/components/preferences/siteDataSettings.js#265-291, allowed
starts out as false
and is never set to true
if there are no items to remove. Does that help?
if (removals.length) {
//…
} else {
//nothing
allowed = true;
}
Right?
Assignee | ||
Comment 8•3 years ago
|
||
Thank you for stepping in for Johann. Yes that appears to help. I will start work on fixing this and get back to you. Thanks a lot.
Assignee | ||
Comment 9•3 years ago
|
||
I believed I have solved the issue. Heading to about:preferences#privacy then going to "Manage Data…" in the "Cookies and Site Data" section. After changing no settings I click "Save Changes" and the widget closes.
What are the next steps for me to get this code commited, reviewed and completed? Sorry to ask for help on every step, I am very new to this. I appreciate the help.
Assignee | ||
Comment 10•3 years ago
|
||
Comment 11•3 years ago
|
||
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/2eaeb35d81d6 Save Changes in Manage Data dialog in preferences doesn't close the dialog, r=Gijs
Comment 12•3 years ago
|
||
Looks like you figured it out - thanks!
Assignee | ||
Comment 13•3 years ago
|
||
Thank you. Do I need to do anything else?
Assignee | ||
Updated•3 years ago
|
Comment 14•3 years ago
|
||
(In reply to Jacob Stenson from comment #13)
Thank you. Do I need to do anything else?
Nope. The bug will get closed automatically once the change is merged to mozilla-central, so leaving it open for now.
Assignee | ||
Comment 15•3 years ago
|
||
Ah okay, apologies for closing it prematurely.
Comment 16•3 years ago
|
||
bugherder |
Updated•3 years ago
|
Description
•