Closed Bug 662414 Opened 9 years ago Closed 9 years ago

Provide error page/msg for new necko error NS_ERROR_CORRUPTED_CONTENT

Categories

(Core :: Networking, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla7

People

(Reporter: jduell.mcbugs, Assigned: mcmanus)

References

Details

Attachments

(1 file)

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)
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+
Keywords: checkin-needed
Assignee: nobody → mcmanus
http://hg.mozilla.org/mozilla-central/rev/571b534acde9
Status: NEW → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
Blocks: 663608
Depends on: 682511
You need to log in before you can comment on or make changes to this bug.