Bug 783863 - ArchiveInputStream::Tell incorrectly uses mStatus as an nsresult; r=baku
authorAryeh Gregor <ayg@aryeh.name>
Tue, 21 Aug 2012 14:56:39 +0300
changeset 102932 26da3bdbf618f18917bcbfe7c8c9b92b0377e8bb
parent 102931 788a128c8ea5db343c3e6f477d57a9291f24bc60
child 102933 a1d9dc5637315f791e02e470b847cee46e6f9ee4
push id23317
push userryanvm@gmail.com
push dateWed, 22 Aug 2012 02:05:02 +0000
treeherdermozilla-central@abc17059522b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbaku
bugs783863
milestone17.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 783863 - ArchiveInputStream::Tell incorrectly uses mStatus as an nsresult; r=baku
dom/file/ArchiveZipFile.cpp
--- a/dom/file/ArchiveZipFile.cpp
+++ b/dom/file/ArchiveZipFile.cpp
@@ -322,19 +322,16 @@ ArchiveInputStream::Seek(PRInt32 aWhence
   }
 
   return NS_OK;
 }
 
 NS_IMETHODIMP
 ArchiveInputStream::Tell(PRInt64 *aResult)
 {
-  if (NS_FAILED(mStatus))
-    return mStatus;
-
   LL_UI2L(*aResult, mData.cursor);
   return NS_OK;
 }
 
 NS_IMETHODIMP
 ArchiveInputStream::SetEOF()
 {
   return NS_ERROR_NOT_IMPLEMENTED;