Expand GPU rasterization trigger for Finch experiment

Expand GPU rasterization trigger for field trial

This allows expanding the GPU rasterization trigger for
a field trial. The expanded trigger is
<meta name="viewport" content="width=device-width, minimum-scale=1.0">

BUG=380454
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175699

[ajuma, 2014-06-04 01:05:54]

The chromium side of this is https://codereview.chromium.org/314753002/

[jamesr, 2014-06-04 01:15:45]

Somebody working on the feature should review this

[aelias_OOO_until_Jul13, 2014-06-04 01:27:14]

lgtm for Source/web.  Still needs OWNERS for Source/public.

[ajuma, 2014-06-04 01:41:20]

On 2014/06/04 01:27:14, aelias wrote:
> lgtm for Source/web.  Still needs OWNERS for Source/public.

jamesr, can you have a look for public/ OWNERS?

[blink-reviews, 2014-06-04 01:57:03]

As I said someone involved in this should do the review. I'm not, so no.
On Jun 3, 2014 6:41 PM, <ajuma@chromium.org> wrote:

> On 2014/06/04 01:27:14, aelias wrote:
>
>> lgtm for Source/web.  Still needs OWNERS for Source/public.
>>
>
> jamesr, can you have a look for public/ OWNERS?
>
> https://codereview.chromium.org/309593007/
>

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

[ajuma, 2014-06-04 02:14:22]

+jochen, abarth for public/ OWNERS

[abarth-chromium, 2014-06-04 02:23:43]

https://codereview.chromium.org/309593007/diff/1/public/web/WebViewClient.h
File public/web/WebViewClient.h (right):

https://codereview.chromium.org/309593007/diff/1/public/web/WebViewClient.h#n...
public/web/WebViewClient.h:134: virtual bool
useExpandedHeuristicsForGpuRasterization() { return false; }
This should be a WebSetting.

https://codereview.chromium.org/309593007/diff/1/public/web/WebViewClient.h#n...
public/web/WebViewClient.h:135: 
Two spaces between sections.

[ajuma, 2014-06-04 14:48:00]

PTAL.

https://codereview.chromium.org/309593007/diff/1/public/web/WebViewClient.h
File public/web/WebViewClient.h (right):

https://codereview.chromium.org/309593007/diff/1/public/web/WebViewClient.h#n...
public/web/WebViewClient.h:134: virtual bool
useExpandedHeuristicsForGpuRasterization() { return false; }
On 2014/06/04 02:23:43, abarth wrote:
> This should be a WebSetting.

Done.

[abarth-chromium, 2014-06-06 17:46:40]

lgtm

[abarth-chromium, 2014-06-06 17:46:43]

The CQ bit was checked by abarth@chromium.org

[commit-bot: I haz the power, 2014-06-06 17:47:03]

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

[commit-bot: I haz the power, 2014-06-06 20:07:38]

Change committed as 175699