Add bench and gm for shapes

Add bench and gm for shapes

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1677253002

patch from issue 1686673002 at patchset 20001 (http://crrev.com/1686673002#ps20001)

Committed: https://skia.googlesource.com/skia/+/4a6e40d9d2e0eb81e00d8b2bd20532aa28f61afb

[Chris Dalton, 2016-02-08 20:44:01]

Description was changed from

==========
Add bench and gm for shapes

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

to

==========
Add bench and gm for shapes

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[Chris Dalton, 2016-02-08 20:49:18]

cdalton@nvidia.com changed reviewers:
+ brianosman@google.com, bsalomon@google.com

[Chris Dalton, 2016-02-08 20:49:18]

https://codereview.chromium.org/1677253002/diff/1/bench/ShapesBench.cpp
File bench/ShapesBench.cpp (right):

https://codereview.chromium.org/1677253002/diff/1/bench/ShapesBench.cpp#newco...
bench/ShapesBench.cpp:25: DEFINE_bool(shapesPersp, false, "Use slight
perspective tilt in ShapesBench?");
This is one extreme where everything comes from command line flags and we only
run a CPU bound bench by default.

There are other interesting benchmarks as well:
 - GPU bound
 - Different types of shapes
 - Inner shapes?
 - Perspective?

I'd like to decide what we want to have as its own bench and what can still be a
command line flag.

[joshualitt, 2016-02-10 19:55:24]

On 2016/02/08 20:49:18, Chris Dalton wrote:
> https://codereview.chromium.org/1677253002/diff/1/bench/ShapesBench.cpp
> File bench/ShapesBench.cpp (right):
> 
>
https://codereview.chromium.org/1677253002/diff/1/bench/ShapesBench.cpp#newco...
> bench/ShapesBench.cpp:25: DEFINE_bool(shapesPersp, false, "Use slight
> perspective tilt in ShapesBench?");
> This is one extreme where everything comes from command line flags and we only
> run a CPU bound bench by default.
> 
> There are other interesting benchmarks as well:
>  - GPU bound
>  - Different types of shapes
>  - Inner shapes?
>  - Perspective?
> 
> I'd like to decide what we want to have as its own bench and what can still be
a
> command line flag.

The code itself is pretty clear, but could you document the purpose /
methodology of both the GM and bench?

[Chris Dalton, 2016-02-10 20:35:30]

Description was changed from

==========
Add bench and gm for shapes

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Add bench and gm for shapes

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

patch from issue 1686673002 at patchset 20001
(http://crrev.com/1686673002#ps20001)
==========

[Chris Dalton, 2016-02-10 21:01:38]

cdalton@nvidia.com changed reviewers:
+ joshualitt@google.com

[Chris Dalton, 2016-02-10 21:01:38]

Comments added. Is that about what you were looking for?

[joshualitt, 2016-02-10 21:02:42]

On 2016/02/10 21:01:38, Chris Dalton wrote:
> Comments added. Is that about what you were looking for?

yea, that is good.  Thanks!

[Brian Osman, 2016-02-10 21:12:43]

lgtm

[Chris Dalton, 2016-02-10 21:25:33]

The CQ bit was checked by cdalton@nvidia.com

[commit-bot: I haz the power, 2016-02-10 21:25:45]

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

[commit-bot: I haz the power, 2016-02-10 21:25:46]

This CL has an open dependency (Issue 1673093002 Patch 60001). Please resolve
the dependency and try again.

[Chris Dalton, 2016-02-10 21:33:31]

The CQ bit was checked by cdalton@nvidia.com

[Chris Dalton, 2016-02-10 21:33:32]

The patchset sent to the CQ was uploaded after l-g-t-m from
brianosman@google.com
Link to the patchset: https://codereview.chromium.org/1677253002/#ps80001
(title: "Remove unneeded dependency")

[commit-bot: I haz the power, 2016-02-10 21:33:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1677253002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1677253002/80001

[commit-bot: I haz the power, 2016-02-10 21:36:25]

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

[commit-bot: I haz the power, 2016-02-10 21:36:27]

Try jobs failed on following builders:
  Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[Chris Dalton, 2016-02-10 22:25:22]

The CQ bit was checked by cdalton@nvidia.com

[Chris Dalton, 2016-02-10 22:25:22]

The patchset sent to the CQ was uploaded after l-g-t-m from
brianosman@google.com
Link to the patchset: https://codereview.chromium.org/1677253002/#ps100001
(title: "MSVC warnings")

[commit-bot: I haz the power, 2016-02-10 22:25:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1677253002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1677253002/100001

[commit-bot: I haz the power, 2016-02-10 22:54:24]

Description was changed from

==========
Add bench and gm for shapes

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

patch from issue 1686673002 at patchset 20001
(http://crrev.com/1686673002#ps20001)
==========

to

==========
Add bench and gm for shapes

BUG=skia:
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

patch from issue 1686673002 at patchset 20001
(http://crrev.com/1686673002#ps20001)

Committed:
https://skia.googlesource.com/skia/+/4a6e40d9d2e0eb81e00d8b2bd20532aa28f61afb
==========

[commit-bot: I haz the power, 2016-02-10 22:54:25]

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