Bug 1079621 - Change non-fatal errors to warnings. r=rillian

This resolves intermittent failures in continuous integration.

NS_ERROR() triggers an assertion in debug builds, which tinderbox
reports as a failure. NS_WARNING calls NS_DebugBreak, but with the
lower severity level it skips invoking the debugger.

dom/media/fmp4/apple/AppleVDADecoder.cpp

@@ -351,7 +351,7 @@ AppleVDADecoder::SubmitFrame(mp4_demuxer::MP4Sample* aSample)
   MOZ_ASSERT(CFGetRetainCount(frameInfo) >= 2, "Bad retain count");
 
   if (rv != noErr) {
-    NS_ERROR("AppleVDADecoder: Couldn't pass frame to decoder");
+    NS_WARNING("AppleVDADecoder: Couldn't pass frame to decoder");
     return NS_ERROR_FAILURE;
   }
 

dom/media/fmp4/apple/AppleVTDecoder.cpp

@@ -240,7 +240,7 @@ AppleVTDecoder::SubmitFrame(mp4_demuxer::MP4Sample* aSample)
                                          CreateAppleFrameRef(aSample),
                                          &flags);
   if (rv != noErr) {
-    NS_ERROR("Couldn't pass frame to decoder");
+    NS_WARNING("Couldn't pass frame to decoder");
     return NS_ERROR_FAILURE;
   }