New issue
Advanced search Search tips

Issue 838780 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

data_decoder: Use a single shared memory region for series of SkBitmaps

Project Member Reported by roc...@chromium.org, May 2 2018

Issue description

ImageDecoder.DecodeAnimation[1] replies with an array of elements, with each element containing a skia.mojom.InlineBitmap representing a single frame.

If this is changed to a skia.mojom.Bitmap, larger frames will be backed by shared memory handles, and longer animations may then exceed the maximum allowed handle count per message. This was covered by  issue 838706 .

We should consider introducing a separate mojom type for a sequence of Bitmap images, using a single mojo_base.mojom.BigBuffer to carry the pixel data for all bitmaps in the sequence.

[1] https://cs.chromium.org/chromium/src/services/data_decoder/public/mojom/image_decoder.mojom?rcl=f9c2cf42a20b6b7ebbd1b4d005ffff9cbb623e64&l=48
 

Sign in to add a comment