Closed Bug 944204 Opened 11 years ago Closed 3 years ago

toBlob asserts with tall WebGL canvas

Categories

(Core :: Graphics: CanvasWebGL, defect, P2)

x86_64
macOS
defect

Tracking

()

RESOLVED INCOMPLETE

People

(Reporter: jruderman, Unassigned, NeedInfo)

References

Details

(Keywords: assertion, testcase)

Attachments

(3 files)

Attached file testcase
Assertion failure: elementSize.height == mCurrentContext->GetHeight() || (elementSize.height == 0 && mCurrentContext->GetHeight() == 1), at content/html/content/src/HTMLCanvasElement.cpp:525
Attached file stack (lldb)
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
Attached file width testcase
Assertion failure: elementSize.width == mCurrentContext->GetWidth() || (elementSize.width == 0 && mCurrentContext->GetWidth() == 1), at content/html/content/src/HTMLCanvasElement.cpp:519
Assignee: jgilbert → nobody
Priority: -- → P2

Hey Jesse,
Can you still reproduce this issue or should we close it?

Flags: needinfo?(jruderman)

Marking this as Resolved > Incomplete as per reporter's lack of response.
If anyone can still reproduce this issue re-open it or file a new bug.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: