User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5a) Gecko/20030718 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5a) Gecko/20030718 File, Save page as, to write protect media doesn't give an error message. Mozilla should report that the media is write protected Reproducible: Always Steps to Reproduce: 1. Start Mozilla Build ID 2003071814 2. Menu: File, Save page as 3. Select a write protected media as destination for the file Actual Results: The media did not contain the file. Unexpected, because I did not knew the media was write protected. Expected Results: Report that the media is write protected The media is a TS128MJFLASHA USB Device
Shouldn't the OS file saving dialog give out a warning here (also Mozilla says Download completed if you ignore that warning)?
-> File Handling
Assignee: darin → law
Component: Networking: File → File Handling
QA Contact: benc → petersen
biesi: you have time to look into this bug?
Assignee: law → cbiesinger
15 years ago
found the bug. need to pass OFN_PATHMUSTEXIST as well, otherwise OFN_NOREADONLYRETURN is ignored. (in windows/nsFilePicker.cpp). note that linux does not have this bug.
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P1
Target Milestone: --- → mozilla1.5beta
15 years ago
Attachment #130009 - Flags: superreview?(darin)
Comment on attachment 130009 [details] [diff] [review] patch sr=darin
Attachment #130009 - Flags: superreview?(darin) → superreview+
Comment on attachment 130009 [details] [diff] [review] patch should be a low-risk change
Attachment #130009 - Flags: approval1.5b?
Comment on attachment 130009 [details] [diff] [review] patch a=mkaply
Attachment #130009 - Flags: approval1.5b? → approval1.5b+
Checking in src/windows/nsFilePicker.cpp; /cvsroot/mozilla/widget/src/windows/nsFilePicker.cpp,v <-- nsFilePicker.cpp new revision: 1.69; previous revision: 1.68 done
Status: ASSIGNED → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.