-
Notifications
You must be signed in to change notification settings - Fork 9.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Removing set-presence-in-ImageData check from canvas.js #4420
Removing set-presence-in-ImageData check from canvas.js #4420
Conversation
if (window.CanvasPixelArray) { | ||
if (typeof window.CanvasPixelArray.prototype.set !== 'function') { | ||
window.CanvasPixelArray.prototype.set = function(arr) { | ||
for (var i = 0, ii = this.length; i < ii; i++) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@timvandermeij Have a look
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's better, yes.
Could you remind me what browsers don't support set()? Also, is there any documentation that only browsers with CanvasPixelArray need the emulated set function? |
That's mostly to remove ie9 logic from the main code |
@@ -507,20 +507,15 @@ var CanvasGraphics = (function CanvasGraphicsClosure() { | |||
|
|||
} else if (imgData.kind === ImageKind.RGBA_32BPP) { | |||
// RGBA, 32-bits per pixel. | |||
var haveSetAndSubarray = 'set' in dest && 'subarray' in src; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@brendandahl Removed!
@yurydelendik Is everything okay with the PR now? |
Looks good, thank you |
…sence Removing set-presence-in-ImageData check from canvas.js
oops, forgot /botio-linux test |
From: Bot.io (Linux)ReceivedCommand cmd_test from @yurydelendik received. Current queue size: 0 Live output at: http://107.21.233.14:8877/912ec6a20012099/output.txt |
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/912ec6a20012099/output.txt Total script time: 26.47 mins
|
Trying to Fix #4270