Hide SkCanvas::{set,get}DrawFilter

Hide SkCanvas::{set,get}DrawFilter

Except for Android, which still makes use of it.

BUG=skia:3587
R=reed@google.com,djsollen@google.com
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1618843002

Committed: https://skia.googlesource.com/skia/+/7765000709dc64eb23be7df47d1f995d1f787115

[f(malita), 2016-01-21 15:43:20]

Description was changed from

==========
Hide SkCanvas::{set,get}DrawFilter

Except for Android, which still makes use of it.

BUG=skia:3587
R=reed@google.com,djsollen@google.com
==========

to

==========
Hide SkCanvas::{set,get}DrawFilter

Except for Android, which still makes use of it.

BUG=skia:3587
R=reed@google.com,djsollen@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[f(malita), 2016-01-21 15:48:58]

The CQ bit was checked by fmalita@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-01-21 15:49:03]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1618843002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1618843002/1

[commit-bot: I haz the power, 2016-01-21 15:56:56]

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

[commit-bot: I haz the power, 2016-01-21 15:56:56]

Dry run: This issue passed the CQ dry run.

[djsollen, 2016-01-21 16:34:28]

https://codereview.chromium.org/1618843002/diff/1/include/core/SkCanvas.h
File include/core/SkCanvas.h (right):

https://codereview.chromium.org/1618843002/diff/1/include/core/SkCanvas.h#new...
include/core/SkCanvas.h:1152: #ifdef SK_INTERNAL
why do we need SK_INTERNAL?

[f(malita), 2016-01-21 17:54:16]

https://codereview.chromium.org/1618843002/diff/1/include/core/SkCanvas.h
File include/core/SkCanvas.h (right):

https://codereview.chromium.org/1618843002/diff/1/include/core/SkCanvas.h#new...
include/core/SkCanvas.h:1152: #ifdef SK_INTERNAL
On 2016/01/21 16:34:27, djsollen wrote:
> why do we need SK_INTERNAL?

The intent is to only hide this for external users.  I figured it'd be nice to
still build & run the related tests/GMs internally, while the feature is still
alive.

[reed1, 2016-01-21 18:39:30]

lgtm

[f(malita), 2016-01-22 02:46:30]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2016-01-22 02:46:36]

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

[commit-bot: I haz the power, 2016-01-22 02:47:14]

Description was changed from

==========
Hide SkCanvas::{set,get}DrawFilter

Except for Android, which still makes use of it.

BUG=skia:3587
R=reed@google.com,djsollen@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Hide SkCanvas::{set,get}DrawFilter

Except for Android, which still makes use of it.

BUG=skia:3587
R=reed@google.com,djsollen@google.com
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/7765000709dc64eb23be7df47d1f995d1f787115
==========

[commit-bot: I haz the power, 2016-01-22 02:47:15]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/7765000709dc64eb23be7df47d1f995d1f787115