Whitelist Qualcomm GPUs for GPU rasterization on Android 4.4

Whitelist Qualcomm GPUs for GPU rasterization on Android 4.4

This expands the GPU rasterization device whitelist to include all
Android 4.4 devices with Qualcomm GPUs.

BUG=None

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

[vangelis, 2014-08-12 15:01:05]

On 2014/08/12 14:27:19, ajuma wrote:

LGTM

[ajuma, 2014-08-12 18:28:29]

The gl_vendor check seems to be causing a bunch of GPU test failures across
platforms.

zmo@, any idea what might be happening? I can reproduce the GPU failures locally
on Linux (and confirmed that removing the "gl_vendor" part of the entry fixes
the failures), but haven't had any luck debugging so far.

[Zhenyao Mo, 2014-08-12 21:31:49]

LGTM

I don't know what's going on with your first CL.  In theory the first CL should
do exactly the same thing as the second CL.

Can you file a bug about that and assign to me?

https://codereview.chromium.org/463043004/diff/20001/gpu/config/software_rend...
File gpu/config/software_rendering_list_json.cc (right):

https://codereview.chromium.org/463043004/diff/20001/gpu/config/software_rend...
gpu/config/software_rendering_list_json.cc:1216: }
nit: need an extra space before }

[Zhenyao Mo, 2014-08-12 21:42:47]

By the way, https://codereview.chromium.org/452293002/ just landed, so you need
to modify the entries accordingly.

[ajuma, 2014-08-13 14:58:57]

Filed http://crbug.com/403057 for the difference in behavior between the first
and second patch sets.

On 2014/08/12 21:42:47, Zhenyao Mo wrote:
> By the way, https://codereview.chromium.org/452293002/ just landed, so you
need
> to modify the entries accordingly.

That got reverted, so I've un-done the modifications here.

https://codereview.chromium.org/463043004/diff/20001/gpu/config/software_rend...
File gpu/config/software_rendering_list_json.cc (right):

https://codereview.chromium.org/463043004/diff/20001/gpu/config/software_rend...
gpu/config/software_rendering_list_json.cc:1216: }
On 2014/08/12 21:31:49, Zhenyao Mo wrote:
> nit: need an extra space before }

Fixed.

[ajuma, 2014-08-13 14:59:22]

The CQ bit was checked by ajuma@chromium.org

[commit-bot: I haz the power, 2014-08-13 14:59:42]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/ajuma@chromium.org/463043004/100001

[commit-bot: I haz the power, 2014-08-13 15:52:17]

Committed patchset #5 (100001) as 289301