Revert of Make 2D canvas disable gpu acceleration when getImageData is called

Revert of Make 2D canvas disable gpu acceleration when getImageData is called (patchset #6 id:100001 of https://codereview.chromium.org/2063473002/ )

Reason for revert:
Causing crash in mailbox release callback - bug 623101

Original issue's description:
> Make 2D canvas disable gpu acceleration when getImageData is called
>
> In order to optimize performance, GPU-accelerated canvases should
> permanently switch out of accalerated mode as soon as
> getImageData is called.  The cost of getImageData on the GPU is
> so prohibitively high that it almost always outways the benefit
> of acceleration.
>
> This is the first time we implement a SW/GPU switch that can happen
> after the first frame was presented, which may result in a small one
> time glitch due to rendering engine discrepancies. Let's see if we get
> any complaints about this.  If there are complaints, we may inhibit
> the switch based on what kind of content was rendered to the canvas.
>
> BUG=606688
>
> Committed: https://crrev.com/33732aec33c88cb18c212544da7d0a4ba53cd9ce
> Cr-Commit-Position: refs/heads/master@{#401751}

TBR=xlai@chromium.org,xidachen@chromium.org,junov@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606688, 623101
Committed: https://crrev.com/27543ee6015a5d358e3abdb6ffb6121f6c847e31
Cr-Commit-Position: refs/heads/master@{#402391}

[sunnyps, 2016-06-28 00:02:18]

The CQ bit was checked by sunnyps@chromium.org

[sunnyps, 2016-06-28 00:02:18]

Created Revert of Make 2D canvas disable gpu acceleration when getImageData is
called

[sunnyps, 2016-06-28 00:02:58]

Description was changed from

==========
Revert of Make 2D canvas disable gpu acceleration when getImageData is called
(patchset #6 id:100001 of https://codereview.chromium.org/2063473002/ )

Reason for revert:
Causing crash in mailbox release callback - bug 623101

Original issue's description:
> Make 2D canvas disable gpu acceleration when getImageData is called
> 
> In order to optimize performance, GPU-accelerated canvases should
> permanently switch out of accalerated mode as soon as
> getImageData is called.  The cost of getImageData on the GPU is
> so prohibitively high that it almost always outways the benefit
> of acceleration.
> 
> This is the first time we implement a SW/GPU switch that can happen
> after the first frame was presented, which may result in a small one
> time glitch due to rendering engine discrepancies. Let's see if we get
> any complaints about this.  If there are complaints, we may inhibit
> the switch based on what kind of content was rendered to the canvas.
> 
> BUG=606688
> 
> Committed: https://crrev.com/33732aec33c88cb18c212544da7d0a4ba53cd9ce
> Cr-Commit-Position: refs/heads/master@{#401751}

TBR=xlai@chromium.org,xidachen@chromium.org,junov@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606688
==========

to

==========
Revert of Make 2D canvas disable gpu acceleration when getImageData is called
(patchset #6 id:100001 of https://codereview.chromium.org/2063473002/ )

Reason for revert:
Causing crash in mailbox release callback - bug 623101

Original issue's description:
> Make 2D canvas disable gpu acceleration when getImageData is called
> 
> In order to optimize performance, GPU-accelerated canvases should
> permanently switch out of accalerated mode as soon as
> getImageData is called.  The cost of getImageData on the GPU is
> so prohibitively high that it almost always outways the benefit
> of acceleration.
> 
> This is the first time we implement a SW/GPU switch that can happen
> after the first frame was presented, which may result in a small one
> time glitch due to rendering engine discrepancies. Let's see if we get
> any complaints about this.  If there are complaints, we may inhibit
> the switch based on what kind of content was rendered to the canvas.
> 
> BUG=606688
> 
> Committed: https://crrev.com/33732aec33c88cb18c212544da7d0a4ba53cd9ce
> Cr-Commit-Position: refs/heads/master@{#401751}

BUG=623101
TBR=xlai@chromium.org,xidachen@chromium.org,junov@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606688
==========

[sunnyps, 2016-06-28 00:02:58]

sunnyps@chromium.org changed reviewers:
+ piman@chromium.org

[commit-bot: I haz the power, 2016-06-28 00:03:23]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sunnyps, 2016-06-28 00:03:55]

Description was changed from

==========
Revert of Make 2D canvas disable gpu acceleration when getImageData is called
(patchset #6 id:100001 of https://codereview.chromium.org/2063473002/ )

Reason for revert:
Causing crash in mailbox release callback - bug 623101

Original issue's description:
> Make 2D canvas disable gpu acceleration when getImageData is called
> 
> In order to optimize performance, GPU-accelerated canvases should
> permanently switch out of accalerated mode as soon as
> getImageData is called.  The cost of getImageData on the GPU is
> so prohibitively high that it almost always outways the benefit
> of acceleration.
> 
> This is the first time we implement a SW/GPU switch that can happen
> after the first frame was presented, which may result in a small one
> time glitch due to rendering engine discrepancies. Let's see if we get
> any complaints about this.  If there are complaints, we may inhibit
> the switch based on what kind of content was rendered to the canvas.
> 
> BUG=606688
> 
> Committed: https://crrev.com/33732aec33c88cb18c212544da7d0a4ba53cd9ce
> Cr-Commit-Position: refs/heads/master@{#401751}

BUG=623101
TBR=xlai@chromium.org,xidachen@chromium.org,junov@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606688
==========

to

==========
Revert of Make 2D canvas disable gpu acceleration when getImageData is called
(patchset #6 id:100001 of https://codereview.chromium.org/2063473002/ )

Reason for revert:
Causing crash in mailbox release callback - bug 623101

Original issue's description:
> Make 2D canvas disable gpu acceleration when getImageData is called
> 
> In order to optimize performance, GPU-accelerated canvases should
> permanently switch out of accalerated mode as soon as
> getImageData is called.  The cost of getImageData on the GPU is
> so prohibitively high that it almost always outways the benefit
> of acceleration.
> 
> This is the first time we implement a SW/GPU switch that can happen
> after the first frame was presented, which may result in a small one
> time glitch due to rendering engine discrepancies. Let's see if we get
> any complaints about this.  If there are complaints, we may inhibit
> the switch based on what kind of content was rendered to the canvas.
> 
> BUG=606688
> 
> Committed: https://crrev.com/33732aec33c88cb18c212544da7d0a4ba53cd9ce
> Cr-Commit-Position: refs/heads/master@{#401751}

TBR=xlai@chromium.org,xidachen@chromium.org,junov@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606688, 623101
==========

[commit-bot: I haz the power, 2016-06-28 03:41:06]

Description was changed from

==========
Revert of Make 2D canvas disable gpu acceleration when getImageData is called
(patchset #6 id:100001 of https://codereview.chromium.org/2063473002/ )

Reason for revert:
Causing crash in mailbox release callback - bug 623101

Original issue's description:
> Make 2D canvas disable gpu acceleration when getImageData is called
> 
> In order to optimize performance, GPU-accelerated canvases should
> permanently switch out of accalerated mode as soon as
> getImageData is called.  The cost of getImageData on the GPU is
> so prohibitively high that it almost always outways the benefit
> of acceleration.
> 
> This is the first time we implement a SW/GPU switch that can happen
> after the first frame was presented, which may result in a small one
> time glitch due to rendering engine discrepancies. Let's see if we get
> any complaints about this.  If there are complaints, we may inhibit
> the switch based on what kind of content was rendered to the canvas.
> 
> BUG=606688
> 
> Committed: https://crrev.com/33732aec33c88cb18c212544da7d0a4ba53cd9ce
> Cr-Commit-Position: refs/heads/master@{#401751}

TBR=xlai@chromium.org,xidachen@chromium.org,junov@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606688, 623101
==========

to

==========
Revert of Make 2D canvas disable gpu acceleration when getImageData is called
(patchset #6 id:100001 of https://codereview.chromium.org/2063473002/ )

Reason for revert:
Causing crash in mailbox release callback - bug 623101

Original issue's description:
> Make 2D canvas disable gpu acceleration when getImageData is called
> 
> In order to optimize performance, GPU-accelerated canvases should
> permanently switch out of accalerated mode as soon as
> getImageData is called.  The cost of getImageData on the GPU is
> so prohibitively high that it almost always outways the benefit
> of acceleration.
> 
> This is the first time we implement a SW/GPU switch that can happen
> after the first frame was presented, which may result in a small one
> time glitch due to rendering engine discrepancies. Let's see if we get
> any complaints about this.  If there are complaints, we may inhibit
> the switch based on what kind of content was rendered to the canvas.
> 
> BUG=606688
> 
> Committed: https://crrev.com/33732aec33c88cb18c212544da7d0a4ba53cd9ce
> Cr-Commit-Position: refs/heads/master@{#401751}

TBR=xlai@chromium.org,xidachen@chromium.org,junov@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606688, 623101
==========

[commit-bot: I haz the power, 2016-06-28 03:41:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-28 03:43:23]

Description was changed from

==========
Revert of Make 2D canvas disable gpu acceleration when getImageData is called
(patchset #6 id:100001 of https://codereview.chromium.org/2063473002/ )

Reason for revert:
Causing crash in mailbox release callback - bug 623101

Original issue's description:
> Make 2D canvas disable gpu acceleration when getImageData is called
> 
> In order to optimize performance, GPU-accelerated canvases should
> permanently switch out of accalerated mode as soon as
> getImageData is called.  The cost of getImageData on the GPU is
> so prohibitively high that it almost always outways the benefit
> of acceleration.
> 
> This is the first time we implement a SW/GPU switch that can happen
> after the first frame was presented, which may result in a small one
> time glitch due to rendering engine discrepancies. Let's see if we get
> any complaints about this.  If there are complaints, we may inhibit
> the switch based on what kind of content was rendered to the canvas.
> 
> BUG=606688
> 
> Committed: https://crrev.com/33732aec33c88cb18c212544da7d0a4ba53cd9ce
> Cr-Commit-Position: refs/heads/master@{#401751}

TBR=xlai@chromium.org,xidachen@chromium.org,junov@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606688, 623101
==========

to

==========
Revert of Make 2D canvas disable gpu acceleration when getImageData is called
(patchset #6 id:100001 of https://codereview.chromium.org/2063473002/ )

Reason for revert:
Causing crash in mailbox release callback - bug 623101

Original issue's description:
> Make 2D canvas disable gpu acceleration when getImageData is called
>
> In order to optimize performance, GPU-accelerated canvases should
> permanently switch out of accalerated mode as soon as
> getImageData is called.  The cost of getImageData on the GPU is
> so prohibitively high that it almost always outways the benefit
> of acceleration.
>
> This is the first time we implement a SW/GPU switch that can happen
> after the first frame was presented, which may result in a small one
> time glitch due to rendering engine discrepancies. Let's see if we get
> any complaints about this.  If there are complaints, we may inhibit
> the switch based on what kind of content was rendered to the canvas.
>
> BUG=606688
>
> Committed: https://crrev.com/33732aec33c88cb18c212544da7d0a4ba53cd9ce
> Cr-Commit-Position: refs/heads/master@{#401751}

TBR=xlai@chromium.org,xidachen@chromium.org,junov@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=606688, 623101

Committed: https://crrev.com/27543ee6015a5d358e3abdb6ffb6121f6c847e31
Cr-Commit-Position: refs/heads/master@{#402391}
==========

[commit-bot: I haz the power, 2016-06-28 03:43:24]

Patchset 1 (id:??) landed as
https://crrev.com/27543ee6015a5d358e3abdb6ffb6121f6c847e31
Cr-Commit-Position: refs/heads/master@{#402391}