Closed Bug 422445 Opened 13 years ago Closed 13 years ago

WebBrowserPersist treats StartUpload failure as a readError

Categories

(Core Graveyard :: Embedding: APIs, defect)

Product:
Core Graveyard
Component:
Embedding: APIs
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: timeless, Assigned: timeless)

References

(
URL
)

Details

Attachments

(1 file)

don't treat upload (write) failure as a readError
935 bytes, patch
Biesinger
: review+
damons
: approval1.9+
Details | Diff | Splinter Review 
No description provided.
Attachment #308886 - Flags: review?(cbiesinger)
Comment on attachment 308886 [details] [diff] [review]
don't treat upload (write) failure as a readError

looks good (sigh, http://lxr.mozilla.org/seamonkey/source/embedding/components/webbrowserpersist/src/nsWebBrowserPersist.cpp#531 turns all errors into NS_ERROR_FAILURE :( )
Attachment #308886 - Flags: review?(cbiesinger) → review+
Attachment #308886 - Flags: approval1.9?
Comment on attachment 308886 [details] [diff] [review]
don't treat upload (write) failure as a readError

https://bugzilla.mozilla.org/show_bug.cgi?id=399498
Attachment #308886 - Flags: approval1.9? → approval1.9+

(From update of attachment 308886 [details] [diff] [review])
mozilla/embedding/components/webbrowserpersist/src/nsWebBrowserPersist.cpp 	1.137
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.