Fix xpcshell-test bustage from 8f2cd683cba7. rs=bustage-fix for CLOSED TREE
authorMark Banner <standard8@mozilla.com>
Sat, 09 Aug 2014 09:50:22 +0100
changeset 16631 3de19029ba4d64c15336d78a2d441667448cd7d2
parent 16630 4e22d0e24362704410b139fe5de561983ced5c0b
child 16632 871de9d0d930f712c99bf3cd2db369e6afc4e442
push id10351
push usermbanner@mozilla.com
push dateSat, 09 Aug 2014 08:50:34 +0000
treeherdercomm-central@3de19029ba4d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbustage-fix
Fix xpcshell-test bustage from 8f2cd683cba7. rs=bustage-fix for CLOSED TREE
mailnews/local/src/nsLocalMailFolder.cpp
--- a/mailnews/local/src/nsLocalMailFolder.cpp
+++ b/mailnews/local/src/nsLocalMailFolder.cpp
@@ -2020,17 +2020,17 @@ NS_IMETHODIMP nsMsgLocalMailFolder::Begi
     rv = InitCopyMsgHdrAndFileStream();
     NS_ENSURE_SUCCESS(rv, rv);
   }
   nsCOMPtr <nsISeekableStream> seekableStream = do_QueryInterface(mCopyState->m_fileStream, &rv);
   NS_ENSURE_SUCCESS(rv, rv);
   seekableStream->Seek(nsISeekableStream::NS_SEEK_END, 0);
 
   int32_t messageIndex = (mCopyState->m_copyingMultipleMessages) ? mCopyState->m_curCopyIndex - 1 : mCopyState->m_curCopyIndex;
-  NS_ASSERTION(!mCopyState->m_copyingMultipleMessages || messageIndex, "messageIndex invalid");
+  NS_ASSERTION(!mCopyState->m_copyingMultipleMessages || messageIndex >= 0, "messageIndex invalid");
   // by the time we get here, m_curCopyIndex is 1 relative because WriteStartOfNewMessage increments it
   mCopyState->m_messages->QueryElementAt(messageIndex, NS_GET_IID(nsIMsgDBHdr),
                                   (void **)getter_AddRefs(mCopyState->m_message));
   // The flags of the source message can get changed when it is deleted, so
   // save them here.
   if (mCopyState->m_message)
     mCopyState->m_message->GetFlags(&(mCopyState->m_flags));
   DisplayMoveCopyStatusMsg();