New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 704571 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 702492



Sign in to add a comment

EnableAccelerationToAvoidReadbacks for SW-HW canvas switches

Project Member Reported by fs...@chromium.org, Mar 23 2017

Issue description

This has been removed by:
https://codereview.chromium.org/2767753002
due to:  crbug.com/702492  

The underlying issue still remains: the switch from software to accelerated leaves the invalidation in a weird state. I'm attaching a follow up test I'm using.


 
simple_refresh.html
1.3 KB View Download

Comment 1 by fs...@chromium.org, Mar 31 2017

Cc: fs...@chromium.org toyoshim@chromium.org
 Issue 705375  has been merged into this issue.
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 7 2017

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

commit 2f0fa56eb8fc2eee27d437e7b6c8038ba5a52d87
Author: fserb <fserb@chromium.org>
Date: Fri Apr 07 00:25:33 2017

Do compositing update when switching from software to accelerated canvas

Also, re-enables switching from sw to accelerated canvas.

BUG= 704571 

Review-Url: https://codereview.chromium.org/2799033002
Cr-Commit-Position: refs/heads/master@{#462706}

[modify] https://crrev.com/2f0fa56eb8fc2eee27d437e7b6c8038ba5a52d87/third_party/WebKit/Source/core/html/HTMLCanvasElement.cpp
[modify] https://crrev.com/2f0fa56eb8fc2eee27d437e7b6c8038ba5a52d87/third_party/WebKit/Source/platform/graphics/ExpensiveCanvasHeuristicParameters.h

Project Member

Comment 3 by bugdroid1@chromium.org, Apr 7 2017

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

commit 9c1cc6a2801bf174202550d981303837277f058d
Author: kbr <kbr@chromium.org>
Date: Fri Apr 07 05:54:58 2017

Revert of Do compositing update when switching from software to accelerated canvas (patchset #1 id:1 of https://codereview.chromium.org/2799033002/ )

Reason for revert:
Suspect this caused linux_optional_gpu_tests_rel tryserver to break in  http://crbug.com/709342  .

Original issue's description:
> Do compositing update when switching from software to accelerated canvas
>
> Also, re-enables switching from sw to accelerated canvas.
>
> BUG= 704571 
>
> Review-Url: https://codereview.chromium.org/2799033002
> Cr-Commit-Position: refs/heads/master@{#462706}
> Committed: https://chromium.googlesource.com/chromium/src/+/2f0fa56eb8fc2eee27d437e7b6c8038ba5a52d87

TBR=junov@chromium.org,fserb@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 704571 

Review-Url: https://codereview.chromium.org/2799133004
Cr-Commit-Position: refs/heads/master@{#462783}

[modify] https://crrev.com/9c1cc6a2801bf174202550d981303837277f058d/third_party/WebKit/Source/core/html/HTMLCanvasElement.cpp
[modify] https://crrev.com/9c1cc6a2801bf174202550d981303837277f058d/third_party/WebKit/Source/platform/graphics/ExpensiveCanvasHeuristicParameters.h

Project Member

Comment 4 by bugdroid1@chromium.org, Apr 7 2017

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

commit 1dbe1136b6651ba54f52c2569ef95a62d31aac8f
Author: fserb <fserb@chromium.org>
Date: Fri Apr 07 17:07:47 2017

Do compositing update when switching from software to accelerated canvas

Also, re-enables switching from sw to accelerated canvas.

BUG= 704571 

Review-Url: https://codereview.chromium.org/2799033002
Cr-Original-Commit-Position: refs/heads/master@{#462706}
Committed: https://chromium.googlesource.com/chromium/src/+/2f0fa56eb8fc2eee27d437e7b6c8038ba5a52d87
Review-Url: https://codereview.chromium.org/2799033002
Cr-Commit-Position: refs/heads/master@{#462900}

[modify] https://crrev.com/1dbe1136b6651ba54f52c2569ef95a62d31aac8f/third_party/WebKit/Source/core/html/HTMLCanvasElement.cpp
[modify] https://crrev.com/1dbe1136b6651ba54f52c2569ef95a62d31aac8f/third_party/WebKit/Source/platform/graphics/ExpensiveCanvasHeuristicParameters.h

Comment 5 by fs...@chromium.org, Apr 7 2017

Cc: verwa...@chromium.org mythria@chromium.org cbruni@chromium.org simonhatch@chromium.org
 Issue 704861  has been merged into this issue.

Comment 6 by fs...@chromium.org, Apr 7 2017

Status: Fixed (was: Assigned)
After that unjust revert, we are back! :)

Sign in to add a comment