Blipshot icon indicating copy to clipboard operation
Blipshot copied to clipboard

Published 20 hours ago • verified publisher icon folletto

Bug: screenshot maximum height is 32767px (Chrome/Skia bug)

Open povbet opened this issue 7 years ago • 9 comments

Hello, Blipshot worked correctly with Chrome 56, but hangs with Chrome 57 (Slimjet 14), on the make icon, for any page lengths Sincerely

povbet avatar May 28 '17 08:05 povbet

Heya, I'm not able to test on Chrome 57 as mine has already updated, but I tried on Chrome 58 and it works. Can you try updating and see what happens? Thanks!

screen shot 2017-05-28 at 13 41 40

folletto avatar May 28 '17 12:05 folletto

I'm on Version 59.0.3071.115 (Official Build) (64-bit) and nothing is happening after the grab.

a4jp-com avatar Jul 11 '17 14:07 a4jp-com

Can you please refer to which site are you trying on, and if you have errors in the console?

folletto avatar Jul 11 '17 14:07 folletto

http://www.009.cd2.com/members/how_to/nouaillier_a.htm

Error in event handler for runtime.onMessage: TypeError: Cannot read property '3' of null at dataToBlobURL (chrome-extension://mdaboflcmhejfihjcbmdiebgfchigjcf/screenshotter.DOM.js:187:28) at screenshotReturn (chrome-extension://mdaboflcmhejfihjcbmdiebgfchigjcf/screenshotter.DOM.js:80:19) at chrome-extension://mdaboflcmhejfihjcbmdiebgfchigjcf/screenshotter.DOM.js:121:36

a4jp-com avatar Jul 11 '17 19:07 a4jp-com

Given the page length my first impression is that it might be a limit of size: Data URL which is used internally has a limit (2Mb apparently).

I'll look into that, thanks!

folletto avatar Jul 12 '17 10:07 folletto

Great. Thanks for that.

a4jp-com avatar Jul 12 '17 22:07 a4jp-com

I found the issue: apparently Chrome (Chromium) uses a library named Skia that has a limit on the maximum height and width of images. So the maximum image Chrome's canvas can generate is 32767px.

I introduced a better handling in Blipshot 1.2.1 so at the very least it generates the image up to the limit. It's cut, but at least it's something.

Here the bug on Chromium ⌗339725 (closed as wontfix)

Here the bug on Skia ⌗2122 (since 2014)

I'll keep the issue open to keep an eye on it, even if it's outside what I can do.

folletto avatar Aug 06 '17 21:08 folletto

Anything to either fix the bug or make everything work is cool. Thanks for checking up on everything.

a4jp-com avatar Aug 07 '17 00:08 a4jp-com

I sample checked also other full-page screenshot extensions and they all have the same issue: they break at 32k pixels.

If you happen to find one that doesn't, let me know. :)

Until then, we can just hope that someone in Google fixes it. Even if seems unlikely given it's open since 2014 and buried deep in the Skia library, while closed in Chromium. :(

folletto avatar Aug 07 '17 11:08 folletto