多个图像的context.drawImage最终导致FireFox崩溃

时间:2015-10-10 11:25:39

标签: javascript jquery html5 firefox canvas

我正在创建一个Image Uploader,它应该能够接受至少1000张图像。一旦用户选择/拖放图像,就应显示图像以用于预览目的。然后,用户将单击“上载”按钮以启动上载。

我已经实现了整个功能,它可以像Chrome一样在Chrome上运行。但这在Firefox中失败了。

我已经创建了一种排队机制,在某个时间点,一旦用户选择图像,只会在浏览器中加载n个图像。

对于超过20张图像也会发生崩溃(即使使用排队机制创建一批1张图像)。我已经使用Firefox中的Tab Memory Usage Plugin(http://mybrowseraddon.com/tab-memory.html)检查了内存使用情况。它甚至崩溃了40 MB的内存利用率。所以我想,这不是因为任何内存泄漏。

function readAndDisplay(file, index){
    id = getFileId(file);
    reader = new FileReader();

    reader.onload = function (e) {
        addImageCard(file, index);
        resizeImage(e, index);
        delete this;
        //$('#image' + id).attr('src', e.target.result);

    }

    reader.readAsDataURL(file);
}

function resizeImage(fileReaderEvent, index){

    var img = new Image();
    img.onload=function(){
        var MAX_WIDTH = 800;
        var MAX_HEIGHT = 600;
        var width = img.width;
        var height = img.height;

        if (width > height) {
          if (width > MAX_WIDTH) {
            height *= MAX_WIDTH / width;
            width = MAX_WIDTH;
          }
        } else {
          if (height > MAX_HEIGHT) {
            width *= MAX_HEIGHT / height;
            height = MAX_HEIGHT;
          }
        }
        var canvas = document.getElementById("image" + index);
        var ctx = canvas.getContext("2d");
        canvas.width = width;
        canvas.height = height;
        ctx.drawImage(img, 0, 0, width, height);

        img.src = "";
        delete canvas;
        delete ctx;
        img = null;
        delete fileReaderEvent;

        uploadQueue.push(index);
        if(loadImageIterator < updatedFiles.length){
            loadImage(updatedFiles[loadImageIterator++]);
        }
        else{
            updatedFiles = [];
        }

    }
    img.src = fileReaderEvent.target.result;
    var md5 = CryptoJS.MD5(fileReaderEvent.target.result).toString();
    console.log(md5);
}

function addImageCard(file, index){

    imageHolder = $("#imageholder");

    imageCard = $("<div>", {
        class: "col m1 s4",
        id: "card" + index
    });

    innerDiv = $("<div>", {
        class: "card"
    });

    cardImage = $("<div>", {
        class: "card-image",
    }).append($("<canvas>", {
        id: "image" + index,
        css: {
            width: "100%"
        }
        //src: "images/sample-1.jpg"
    })).append($("<span>", {
        class: "card-title",
        text: ""
    }));

    cardContent = $("<div>",{
        class: "card-content",
        html: "<p>" + file.name + "</p>"
    });

    cardAction = $("<div>", {
        class: "card-action"
    });

    removeButton = $("<a>", {
        title: "Remove",
        css: {
            "font-size": "20px"
        },
        click: function(){
            remove(index)
        }

    }).append($("<span>", {
        class: "glyphicon glyphicon-remove-circle"
    }));

    uploadButton = $("<a>", {
        title: "Upload Now",
        css: {
            "font-size": "20px",
            float: "right",
            "margin-right": "4px"
        }

    }).append($("<span>", {
        class: "glyphicon glyphicon-cloud-upload"
    }));

    cardAction.append(removeButton);
    cardAction.append(uploadButton);

    innerDiv.append(cardImage);
    innerDiv.append(cardContent);
    innerDiv.append(cardAction);
    imageCard.append(innerDiv);
    imageHolder.append(imageCard);

}

当我评论以下代码时,Firefox不会崩溃     ctx.drawImage(img,0,0,width,height)

所以到目前为止我已经想到了问题是这个drawImage。也许在Chrome和Firefox中drawImage的实现是不同的,我使用它的方式使它在Chrome中运行,但是使用Firefox,它会引发错误。

有没有人对这个问题发生的原因有任何具体的想法?

JSFiddle- http://jsfiddle.net/sanchit235/7xh1gbj6/

Firefox崩溃报告 - https://crash-stats.mozilla.com/report/index/6107c530-12a1-4aea-96c9-0366c2151010

1 个答案:

答案 0 :(得分:0)

您的代码中存在一些问题:

  • 第一个是您在loadImage()(onchange)的for循环中同步调用FileSelectHandler(),如果循环完成绘图,它本身会异步调用loadImage()没完成(在我的comp上,20张图片发生了15次) 只在onchange上调用一次并让异步回调就足够了。

  • 第二个是检查回调:
    if(loadImageIterator < updatedFiles.length){ loadImage(updatedFiles[loadImageIterator++]); }
    应该用替换 if(++loadImageIterator < updatedFiles.length){ loadImage(updatedFiles[loadImageIterator]); }
    或者第一个电话会通过updatedFiles[0]

  • 第三个是你的变量声明。您忘记了var之前的readernewFile以及其他一些,制作了这些全局变量。

以下是corrected fiddle,它不会在Chrome或FF中引发任何错误,也不会自行删除任何内容。