Bug 1300549 - Bail out if we have already generated exceptions. r=jgilbert a=jcristau a=gchang
authorChih-Yi Leu <cleu@mozilla.com>
Thu, 24 Nov 2016 01:55:00 +0100
changeset 359094 fc2ea16ec63b39c78facad87271fb5cde0cf939c
parent 359093 4333d321b0dec733f17af7bb0c6514cbaeeb9222
child 359095 3b95f041cd04424e7a90dab3bfa0a2d6836da536
push id1324
push usermtabara@mozilla.com
push dateMon, 16 Jan 2017 13:07:44 +0000
treeherdermozilla-release@a01c49833940 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjgilbert, jcristau, gchang
bugs1300549
milestone51.0
Bug 1300549 - Bail out if we have already generated exceptions. r=jgilbert a=jcristau a=gchang
dom/canvas/TexUnpackBlob.cpp
--- a/dom/canvas/TexUnpackBlob.cpp
+++ b/dom/canvas/TexUnpackBlob.cpp
@@ -645,17 +645,17 @@ TexUnpackImage::TexOrSubImage(bool isSub
     if (surf) {
         // WARNING: OSX can lose our MakeCurrent here.
         dataSurf = surf->GetDataSurface();
     }
     if (!dataSurf) {
         webgl->ErrorOutOfMemory("%s: GetAsSourceSurface or GetDataSurface failed after"
                                 " blit failed for TexUnpackImage.",
                                 funcName);
-        return true;
+        return false;
     }
 
     const TexUnpackSurface surfBlob(webgl, target, mWidth, mHeight, mDepth, dataSurf,
                                     mIsSrcPremult);
 
     return surfBlob.TexOrSubImage(isSubImage, needsRespec, funcName, tex, target, level,
                                   dui, xOffset, yOffset, zOffset, out_error);
 }