New issue
Advanced search Search tips

Issue 651456 link

Starred by 5 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Aug 23
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Optimization on OffscreenCanvas commit(): allocate canvas backings in shared memory

Project Member Reported by xlai@chromium.org, Sep 29 2016

Issue description

In the current implementation of OffscreenCanvas commit(), when the 
Skimage is not texture-backed, the pixels are copied to sharedbitmap.

A possible optimization is to allocate the canvas backings in shared memory right off the beginning, to avoid the copy later.
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 12 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/087971c69243909ff9d706cd07682b5e20d40040

commit 087971c69243909ff9d706cd07682b5e20d40040
Author: xlai <xlai@chromium.org>
Date: Sat May 12 03:21:53 2018

Correct comment in OffscreenCanvasResourceProvider

Over code changes, the comment has been shifted to wrong places.

Bug:  651456 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I5b9cbe92d55a68bdd9547a000398d53fc58c3958
Reviewed-on: https://chromium-review.googlesource.com/1055892
Reviewed-by: Justin Novosad <junov@chromium.org>
Commit-Queue: Olivia Lai <xlai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558103}
[modify] https://crrev.com/087971c69243909ff9d706cd07682b5e20d40040/third_party/blink/renderer/platform/graphics/offscreen_canvas_resource_provider.cc

Comment 2 by xlai@chromium.org, May 12 2018

Owner: junov@chromium.org
Owner: ----
Status: Available (was: Assigned)
Status: WontFix (was: Available)

Sign in to add a comment