Reduce geometry size for circles to help fill rate.

Reduce geometry size for circles to help fill rate.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2391133004

Committed: https://skia.googlesource.com/skia/+/6ca48820407244bbdeb8f9e0ed7d76dd94270460

[jvanverth1, 2016-10-05 16:07:32]

Description was changed from

==========
Reduce geometry size for circles to help fill rate.

BUG=skia:
==========

to

==========
Reduce geometry size for circles to help fill rate.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2391133004
==========

[jvanverth1, 2016-10-06 19:29:39]

jvanverth@google.com changed reviewers:
+ robertphillips@google.com

[jvanverth1, 2016-10-06 19:29:40]

This does appear to run a little faster on Android, as far as one can tell from
nanobench.

[robertphillips, 2016-10-06 20:09:56]

lgtm

https://codereview.chromium.org/2391133004/diff/60001/src/gpu/GrOvalRenderer.cpp
File src/gpu/GrOvalRenderer.cpp (right):

https://codereview.chromium.org/2391133004/diff/60001/src/gpu/GrOvalRenderer....
src/gpu/GrOvalRenderer.cpp:968: if (geom.fStroked) {
Could this path share a "FillInOctVerts" subroutine with the above path ?

[robertphillips, 2016-10-06 20:11:07]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-10-06 20:11:10]

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

[commit-bot: I haz the power, 2016-10-06 20:28:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-06 20:28:21]

Dry run: This issue passed the CQ dry run.

[jvanverth1, 2016-10-06 21:46:19]

https://codereview.chromium.org/2391133004/diff/60001/src/gpu/GrOvalRenderer.cpp
File src/gpu/GrOvalRenderer.cpp (right):

https://codereview.chromium.org/2391133004/diff/60001/src/gpu/GrOvalRenderer....
src/gpu/GrOvalRenderer.cpp:968: if (geom.fStroked) {
On 2016/10/06 20:09:56, robertphillips wrote:
> Could this path share a "FillInOctVerts" subroutine with the above path ?

I tried that but got stymied -- I'll take another stab at it.

[robertphillips, 2016-10-06 21:51:22]

https://codereview.chromium.org/2391133004/diff/60001/src/gpu/GrOvalRenderer.cpp
File src/gpu/GrOvalRenderer.cpp (right):

https://codereview.chromium.org/2391133004/diff/60001/src/gpu/GrOvalRenderer....
src/gpu/GrOvalRenderer.cpp:968: if (geom.fStroked) {
On 2016/10/06 21:46:19, jvanverth1 wrote:
> On 2016/10/06 20:09:56, robertphillips wrote:
> > Could this path share a "FillInOctVerts" subroutine with the above path ?
> 
> I tried that but got stymied -- I'll take another stab at it.

It should probably be a follow up CL.

[jvanverth1, 2016-10-07 13:56:36]

The CQ bit was checked by jvanverth@google.com

[commit-bot: I haz the power, 2016-10-07 13:56:40]

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

[commit-bot: I haz the power, 2016-10-07 13:57:33]

Description was changed from

==========
Reduce geometry size for circles to help fill rate.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2391133004
==========

to

==========
Reduce geometry size for circles to help fill rate.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2391133004

Committed:
https://skia.googlesource.com/skia/+/6ca48820407244bbdeb8f9e0ed7d76dd94270460
==========

[commit-bot: I haz the power, 2016-10-07 13:57:34]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/6ca48820407244bbdeb8f9e0ed7d76dd94270460