Enable distance field path rendering in Chrome.

Enable distance field path rendering in Chrome.

- Add stat tracking for df-renderable paths
- Modify GPU rasterization veto to account for df-renderable paths
- Remove #define hiding df paths from Chromium and Android

BUG=skia:2935
Committed: https://skia.googlesource.com/skia/+/f7007b023d03de0693a39cba45ecdcc63eb9950d

[jvanverth1, 2014-10-29 20:31:02]

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

[jvanverth1, 2014-10-29 20:33:32]

Still need to add volatile flag to Canvas2D paths before this can go in, but
just setting up for quick landing.

[bsalomon, 2014-10-29 20:53:45]

https://codereview.chromium.org/685113003/diff/1/src/core/SkPicture.cpp
File src/core/SkPicture.cpp (right):

https://codereview.chromium.org/685113003/diff/1/src/core/SkPicture.cpp#newco...
src/core/SkPicture.cpp:182: pathBounds.height() < 64.f) {
&& !op.path.isVolatile()
?

https://codereview.chromium.org/685113003/diff/1/src/core/SkPictureContentInf...
File src/core/SkPictureContentInfo.cpp (right):

https://codereview.chromium.org/685113003/diff/1/src/core/SkPictureContentInf...
src/core/SkPictureContentInfo.cpp:67: pathBounds.height() < 64.f) {
ditto

[bsalomon, 2014-10-29 21:02:09]

On 2014/10/29 20:53:45, bsalomon wrote:
> https://codereview.chromium.org/685113003/diff/1/src/core/SkPicture.cpp
> File src/core/SkPicture.cpp (right):
> 
>
https://codereview.chromium.org/685113003/diff/1/src/core/SkPicture.cpp#newco...
> src/core/SkPicture.cpp:182: pathBounds.height() < 64.f) {
> && !op.path.isVolatile()
> ?
> 
>
https://codereview.chromium.org/685113003/diff/1/src/core/SkPictureContentInf...
> File src/core/SkPictureContentInfo.cpp (right):
> 
>
https://codereview.chromium.org/685113003/diff/1/src/core/SkPictureContentInf...
> src/core/SkPictureContentInfo.cpp:67: pathBounds.height() < 64.f) {
> ditto

lgtm, otherwise

[jvanverth1, 2014-10-30 18:55:47]

https://codereview.chromium.org/685113003/diff/1/src/core/SkPicture.cpp
File src/core/SkPicture.cpp (right):

https://codereview.chromium.org/685113003/diff/1/src/core/SkPicture.cpp#newco...
src/core/SkPicture.cpp:182: pathBounds.height() < 64.f) {
On 2014/10/29 20:53:45, bsalomon wrote:
> && !op.path.isVolatile()
> ?

Done.

https://codereview.chromium.org/685113003/diff/1/src/core/SkPictureContentInf...
File src/core/SkPictureContentInfo.cpp (right):

https://codereview.chromium.org/685113003/diff/1/src/core/SkPictureContentInf...
src/core/SkPictureContentInfo.cpp:67: pathBounds.height() < 64.f) {
On 2014/10/29 20:53:45, bsalomon wrote:
> ditto

Done.

[jvanverth1, 2014-11-04 15:38:59]

The CQ bit was checked by jvanverth@google.com

[commit-bot: I haz the power, 2014-11-04 15:40:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/685113003/20001

[commit-bot: I haz the power, 2014-11-04 15:40:16]

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

[commit-bot: I haz the power, 2014-11-04 15:40:17]

Failed to apply patch for gyp/skia_for_chromium_defines.gypi:
While running git apply --index -3 -p1;
  error: patch failed: gyp/skia_for_chromium_defines.gypi:14
  Falling back to three-way merge...
  Applied patch to 'gyp/skia_for_chromium_defines.gypi' with conflicts.
  U gyp/skia_for_chromium_defines.gypi

Patch:       gyp/skia_for_chromium_defines.gypi
Index: gyp/skia_for_chromium_defines.gypi
diff --git a/gyp/skia_for_chromium_defines.gypi
b/gyp/skia_for_chromium_defines.gypi
index
fee2b82fffbd1dc1fbe1f0de005dbb864dca0bb9..e6cb65e448a088aed20240dac2fa1109b207ba06
100644
--- a/gyp/skia_for_chromium_defines.gypi
+++ b/gyp/skia_for_chromium_defines.gypi
@@ -14,7 +14,6 @@
     #
     'skia_for_chromium_defines': [
       'SK_SUPPORT_LEGACY_TEXTRENDERMODE',
-      'SK_LEGACY_NO_DISTANCE_FIELD_PATHS'
     ],
   },
 }

[jvanverth1, 2014-11-04 15:48:28]

The CQ bit was checked by jvanverth@google.com

[commit-bot: I haz the power, 2014-11-04 15:49:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/685113003/40001

[commit-bot: I haz the power, 2014-11-04 15:59:06]

Committed patchset #3 (id:40001) as f7007b023d03de0693a39cba45ecdcc63eb9950d