SkAlphaThresholdFilter allow flattening

SkAlphaThresholdFilter.h allow flattening

Motivation:  allows this:

    #include "SkAlphaThresholdFilter.h"
    void init() {
      SkAlphaThresholdFilter::InitializeFlattenables();
    }

BUG=skia:4613
Committed: https://skia.googlesource.com/skia/+/0745653a677f405bb683b7f7a71b56a7e0dc7921

[hal.canary, 2015-12-07 16:09:19]

halcanary@google.com changed reviewers:
+ mtklein@google.com

[hal.canary, 2015-12-07 16:09:19]

PTAL.  broken off of https://crrev.com/1499443002

[mtklein, 2015-12-07 16:47:11]

https://codereview.chromium.org/1500373003/diff/1/src/effects/SkAlphaThreshol...
File src/effects/SkAlphaThresholdFilter.cpp (left):

https://codereview.chromium.org/1500373003/diff/1/src/effects/SkAlphaThreshol...
src/effects/SkAlphaThresholdFilter.cpp:18: class SK_API
SkAlphaThresholdFilterImpl : public SkImageFilter {
Why can't this stay here in the .cpp file?  Seems easier to read as it is.

[hal.canary, 2015-12-07 17:30:24]

On 2015/12/07 at 16:47:11, mtklein wrote:
>
https://codereview.chromium.org/1500373003/diff/1/src/effects/SkAlphaThreshol...
> File src/effects/SkAlphaThresholdFilter.cpp (left):
> 
>
https://codereview.chromium.org/1500373003/diff/1/src/effects/SkAlphaThreshol...
> src/effects/SkAlphaThresholdFilter.cpp:18: class SK_API
SkAlphaThresholdFilterImpl : public SkImageFilter {
> Why can't this stay here in the .cpp file?  Seems easier to read as it is.

Done.

[hal.canary, 2015-12-07 18:04:21]

Description was changed from

==========
SkAlphaThresholdFilterImpl.h: allow flattening

Motivation:  allows this:

    #include "SkAlphaThresholdFilterImpl.h"
    void init() {
      SK_DEFINE_FLATTENABLE_REGISTRAR_ENTRY(SkAlphaThresholdFilterImpl);
    }

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

to

==========
SkAlphaThresholdFilter.h allow flattening

Motivation:  allows this:

    #include "SkAlphaThresholdFilter.h"
    void init() {
      SkAlphaThresholdFilter::InitializeFlattenables();
    }

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

[hal.canary, 2015-12-07 18:04:35]

PTAL again.

[mtklein, 2015-12-07 18:05:52]

lgtm

[hal.canary, 2015-12-07 18:07:16]

halcanary@google.com changed reviewers:
+ reed@google.com

[hal.canary, 2015-12-07 18:07:16]

reed@: PTAL @ API

[reed1, 2015-12-07 18:09:17]

lgtm

[hal.canary, 2015-12-07 18:12:56]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2015-12-07 18:13:08]

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

[commit-bot: I haz the power, 2015-12-07 18:29:56]

Description was changed from

==========
SkAlphaThresholdFilter.h allow flattening

Motivation:  allows this:

    #include "SkAlphaThresholdFilter.h"
    void init() {
      SkAlphaThresholdFilter::InitializeFlattenables();
    }

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

to

==========
SkAlphaThresholdFilter.h allow flattening

Motivation:  allows this:

    #include "SkAlphaThresholdFilter.h"
    void init() {
      SkAlphaThresholdFilter::InitializeFlattenables();
    }

BUG=skia:4613

Committed:
https://skia.googlesource.com/skia/+/0745653a677f405bb683b7f7a71b56a7e0dc7921
==========

[commit-bot: I haz the power, 2015-12-07 18:29:57]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/0745653a677f405bb683b7f7a71b56a7e0dc7921