Closed Bug 1241436 Opened 4 years ago Closed 4 years ago

User shouldn't be able to enter email address if the "email me when more info is available" option is unchecked in the tab crash reporter

Categories

(Firefox :: General, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 51
Tracking Status
e10s + ---
firefox46 --- affected
firefox51 --- verified

People

(Reporter: pauly, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

46.0a1 (2016-01-20) Win 7
STR:
1. Install http://people.mozilla.org/~tmielczarek/crashme/
2. Go to Customize and move the "crash content process" button to the menu panel
3. Crash the current tab
4. Expand the crash reporter
5. Without checking the "email me when more info is available" option, try to write something in the 'email address' field

Actual results:
able to enter email address even if "email me when more info is available" option is unchecked

Expected results:
User shouldn't be able to enter email address if the "email me when more info is available" option is unchecked
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
Build ID: 20160205004003
 
This issue is also reproducible on the latest Aurora 46.0a2 when e10s is enabled.
Priority: -- → P1
I honestly think that the behavior should be: if the user starts typing an email address there, we should automatically check the checkbox (1).

When you think about it, the checkbox is really pointless: either the user leaves a blank e-mail field and doesn't want to be contacted, or enters an e-mail and do. (2)

What about changing the patch to do (1), and we file a new bug to discuss more about removing the checkbox and changing the placeholder text to make (2) clear?
Attachment #8774691 - Flags: review?(felipc)
Comment on attachment 8774691 [details]
Bug 1241436 - Update the "email me" checkbox in accordance with the email input field in the tab crash reporter.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/67138/diff/1-2/
Attachment #8774691 - Flags: review?(felipc)
Attachment #8774691 - Flags: review?(felipc) → review+
Comment on attachment 8774691 [details]
Bug 1241436 - Update the "email me" checkbox in accordance with the email input field in the tab crash reporter.

https://reviewboard.mozilla.org/r/67138/#review64806
Comment on attachment 8774691 [details]
Bug 1241436 - Update the "email me" checkbox in accordance with the email input field in the tab crash reporter.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/67138/diff/2-3/
Attachment #8774691 - Attachment description: Bug 1241436 - Disable the email text input field if the "email me" checkbox is unchecked in the tab crash reporter UI. → Bug 1241436 - Update the "email me" checkbox in accordance with the email input field in the tab crash reporter.
Pushed by cyu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2ce7bd81fd06
Update the "email me" checkbox in accordance with the email input field in the tab crash reporter. r=Felipe
https://hg.mozilla.org/mozilla-central/rev/2ce7bd81fd06
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51
I have reproduced this with Nightly 46.0a1 (2016-01-21) on Elementary OS 64bit. 

This fix is now verified on latest Nightly 51.0a1

Build ID 	20160806030806
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:51.0) Gecko/20100101 Firefox/51.0

[bugday-20160803]
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.