toBlob asserts with tall WebGL canvas

NEW
Assigned to

Status

()

Core
Canvas: WebGL
--
critical
4 years ago
4 years ago

People

(Reporter: Jesse Ruderman, Assigned: jgilbert)

Tracking

(Blocks: 2 bugs, {assertion, testcase})

Trunk
x86_64
Mac OS X
assertion, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Reporter)

Description

4 years ago
Created attachment 8339652 [details]
testcase

Assertion failure: elementSize.height == mCurrentContext->GetHeight() || (elementSize.height == 0 && mCurrentContext->GetHeight() == 1), at content/html/content/src/HTMLCanvasElement.cpp:525
(Reporter)

Comment 1

4 years ago
Created attachment 8339655 [details]
stack (lldb)
(Assignee)

Comment 2

4 years ago
Alright, so we're resizing the canvas height to <big>, which naturally fails to resize the GL context, which is fine.

It looks like this assert checks that the canvas and renderingContext's width and height match, which isn't guaranteed. Specifically, the sizes *will* differ if you request a too-large size.
Assignee: nobody → jgilbert
(Reporter)

Comment 3

4 years ago
Created attachment 8380154 [details]
width testcase
(Reporter)

Comment 4

4 years ago
Assertion failure: elementSize.width == mCurrentContext->GetWidth() || (elementSize.width == 0 && mCurrentContext->GetWidth() == 1), at content/html/content/src/HTMLCanvasElement.cpp:519
You need to log in before you can comment on or make changes to this bug.