Make consistent with Accelerated 2D Canvas flag on both implementation and chrome://gpu.

Make consistent with Accelerated 2D Canvas flag on both implementation and chrome://gpu.

Currently, we don't enable accelerated 2d canvas in software rendering mode, but
chrome://gpu does not concern software rendering mode.

This CL adds GPUInfo::SupportsAccelerated2dCanvas() so that any clients query if
accelerated 2d canvas is supported.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=238347

[piman, 2013-12-03 01:09:48]

lgtm

[Zhenyao Mo, 2013-12-03 02:57:14]

https://codereview.chromium.org/95013004/diff/20001/gpu/config/gpu_info.h
File gpu/config/gpu_info.h (right):

https://codereview.chromium.org/95013004/diff/20001/gpu/config/gpu_info.h#new...
gpu/config/gpu_info.h:47: bool SupportsAccelerated2dCanvas() const {
Does it violate coding style to define such a function in a struct?  if not,
LGTM

[piman, 2013-12-03 03:07:18]

On Mon, Dec 2, 2013 at 6:57 PM, <zmo@chromium.org> wrote:

>
> https://codereview.chromium.org/95013004/diff/20001/gpu/config/gpu_info.h
> File gpu/config/gpu_info.h (right):
>
> https://codereview.chromium.org/95013004/diff/20001/gpu/
> config/gpu_info.h#newcode47
> gpu/config/gpu_info.h:47: bool SupportsAccelerated2dCanvas() const {
> Does it violate coding style to define such a function in a struct?  if
> not, LGTM
>

It's ok, as long as it stays simple.


> https://codereview.chromium.org/95013004/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-12-03 05:52:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dongseong.hwang@intel.com/95013004/20001

[dshwang, 2013-12-03 05:57:36]

Thank you for review!

[commit-bot: I haz the power, 2013-12-03 11:20:32]

Change committed as 238347