Created attachment 813453 [details] [diff] [review] Patch v1 While writing tests, I noticed that the prompt service isn't doing what I'd expect when the user hits "back" while its open. Namely, if the prompt has a checkbox on it, we act as if the checkbox was unchecked, even if it defaulted to checked. We should ignore changes to the checkbox if the dialog is closed without pressing one of its buttons. I also noticed that in these cases, we're not actually even returning the checkbox value (so we always set the checkbox return value to false). Along the way, I also split complex functions into smaller ones, and removed some unnecessary functions.
Attachment #813453 - Flags: review?(mark.finkle)
Comment on attachment 813453 [details] [diff] [review] Patch v1 Looks like some nice changes and good cleanup
Assignee: nobody → wjohnston
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
You need to log in before you can comment on or make changes to this bug.