corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1302632: P3. Provide failures details to error attribute. r=jwwang

Browse Source 
MozReview-Commit-ID: 3eIXOF96UR4
This commit is contained in:
Jean-Yves Avenard
2016-09-15 16:17:10 +10:00
parent b27a2970db
commit 1933de1197
2 changed files with 8 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
dom/html/HTMLMediaElement.cpp
View File

@@ -1040,9 +1040,9 @@ void HTMLMediaElement::AbortExistingLoads()
mPendingEvents.Clear();
}
void HTMLMediaElement::NoSupportedMediaSourceError()
void HTMLMediaElement::NoSupportedMediaSourceError(const nsACString& aErrorDetails)
{
mError = new MediaError(this, MEDIA_ERR_SRC_NOT_SUPPORTED);
mError = new MediaError(this, MEDIA_ERR_SRC_NOT_SUPPORTED, aErrorDetails);
ChangeNetworkState(nsIDOMHTMLMediaElement::NETWORK_NO_SOURCE);
DispatchAsyncEvent(NS_LITERAL_STRING("error"));
ChangeDelayLoadStatus(false);
@@ -4444,9 +4444,9 @@ void HTMLMediaElement::DecodeError(const MediaResult& aError)
NS_WARNING("Should know the source we were loading from!");
}
} else if (mReadyState == nsIDOMHTMLMediaElement::HAVE_NOTHING) {
NoSupportedMediaSourceError();
NoSupportedMediaSourceError(aError.Description());
} else {
Error(MEDIA_ERR_DECODE, aError);
Error(MEDIA_ERR_DECODE, aError.Description());
}
}
@@ -4461,7 +4461,7 @@ void HTMLMediaElement::LoadAborted()
}
void HTMLMediaElement::Error(uint16_t aErrorCode,
const MediaResult& aErrorDetails)
const nsACString& aErrorDetails)
{
NS_ASSERTION(aErrorCode == MEDIA_ERR_DECODE ||
aErrorCode == MEDIA_ERR_NETWORK ||
@@ -4474,11 +4474,7 @@ void HTMLMediaElement::Error(uint16_t aErrorCode,
if (mError) {
return;
}
nsCString message;
if (NS_FAILED(aErrorDetails)) {
message = aErrorDetails.Description();
}
mError = new MediaError(this, aErrorCode, message);
mError = new MediaError(this, aErrorCode, aErrorDetails);
DispatchAsyncEvent(NS_LITERAL_STRING("error"));
if (mReadyState == nsIDOMHTMLMediaElement::HAVE_NOTHING) {