Closed
Bug 662414
Opened 13 years ago
Closed 13 years ago
Provide error page/msg for new necko error NS_ERROR_CORRUPTED_CONTENT
Categories
(Core :: Networking, defect)
Core
Networking
Tracking
()
RESOLVED
FIXED
mozilla7
People
(Reporter: jduell.mcbugs, Assigned: mcmanus)
References
Details
Attachments
(1 file)
12.97 KB,
patch
|
jduell.mcbugs
:
review+
bzbarsky
:
feedback+
|
Details | Diff | Splinter Review |
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•13 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 2•13 years ago
|
||
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•13 years ago
|
Keywords: checkin-needed
Updated•13 years ago
|
Assignee: nobody → mcmanus
Comment 3•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/571b534acde9
Status: NEW → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
You need to log in
before you can comment on or make changes to this bug.
Description
•