Provide error page/msg for new necko error NS_ERROR_CORRUPTED_CONTENT

RESOLVED FIXED in mozilla7

Status

()

Core
Networking
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jduell, Assigned: mcmanus)

Tracking

unspecified
mozilla7
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 537670 [details] [diff] [review]
error page for ns_error_corrupted content v1

We missed adding an error page and description message for NS_ERROR_CORRUPTED_CONTENT when we added it in bug 597706.  

Attachment written by Patrick McManus and split out from bug 232030.  I'm looking over it.
Attachment #537670 - Flags: review?(jduell.mcbugs)
(Reporter)

Comment 1

6 years ago
Comment on attachment 537670 [details] [diff] [review]
error page for ns_error_corrupted content v1

Wow--that's a lot of different files to tweak for adding a single error msg!  I've verified that the error page shows up fine.   But I'm not sure I'm the right person to say whether this has been tweaked in all the right places.  

Asking bz to have a look in case he knows (or knows who knows).
Attachment #537670 - Flags: review?(jduell.mcbugs)
Attachment #537670 - Flags: review+
Attachment #537670 - Flags: feedback?(bzbarsky)
Comment on attachment 537670 [details] [diff] [review]
error page for ns_error_corrupted content v1

Yeah, this looks right....
Attachment #537670 - Flags: feedback?(bzbarsky) → feedback+
(Reporter)

Updated

6 years ago
Keywords: checkin-needed

Updated

6 years ago
Assignee: nobody → mcmanus
http://hg.mozilla.org/mozilla-central/rev/571b534acde9
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla7

Updated

6 years ago
Blocks: 663608

Updated

6 years ago
Depends on: 682511
You need to log in before you can comment on or make changes to this bug.