corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 802647 - Send browser screenshots as blobs, not base-64 strings. r=khuey,daleharvey

Browse Source
This commit is contained in:
Justin Lebar
2012-10-31 19:28:11 -04:00
parent c39820e22a
commit 9f178d999c
6 changed files with 139 additions and 49 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
dom/browser-element/BrowserElementChild.js
View File

@@ -522,14 +522,16 @@ BrowserElementChild.prototype = {
ctx.drawWindow(content, 0, 0, content.innerWidth, content.innerHeight,
"rgb(255,255,255)");
sendAsyncMsg('got-screenshot', {
id: domRequestID,
// Use JPEG to hack around the fact that we can't specify opaque PNG.
// This requires us to unpremultiply the alpha channel, which is
// expensive on ARM processors because they lack a hardware integer
// division instruction.
successRv: canvas.toDataURL("image/jpeg")
});
// Take a JPEG screenshot to hack around the fact that we can't specify
// opaque PNG. This requires us to unpremultiply the alpha channel, which
// is expensive on ARM processors because they lack a hardware integer
// division instruction.
canvas.toBlob(function(blob) {
sendAsyncMsg('got-screenshot', {
id: domRequestID,
successRv: blob
});
}, 'image/jpeg');
},
_recvFireCtxCallback: function(data) {
@@ -539,7 +541,7 @@ BrowserElementChild.prototype = {
this._ctxHandlers[data.json.menuitem].click();
this._ctxHandlers = {};
} else {
debug("Ignored invalid contextmenu invokation");
debug("Ignored invalid contextmenu invocation");
}
},