add note about security and new subclasses

add note about security and new subclasses

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

Committed: https://skia.googlesource.com/skia/+/50b98b424ccf780f76a1a6fb6e24940ffe681d53

[reed1, 2016-02-03 23:48:23]

Description was changed from

==========
add note about security and new subclasses

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

to

==========
add note about security and new subclasses

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

[reed1, 2016-02-03 23:49:23]

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

[reed1, 2016-02-03 23:49:31]

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

[commit-bot: I haz the power, 2016-02-03 23:49:42]

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

[Oliver Chang, 2016-02-03 23:53:58]

ochang@chromium.org changed reviewers:
+ ochang@chromium.org

[Oliver Chang, 2016-02-03 23:53:58]

thanks for adding this!

https://codereview.chromium.org/1667663004/diff/1/src/ports/SkGlobalInitializ...
File src/ports/SkGlobalInitialization_default.cpp (right):

https://codereview.chromium.org/1667663004/diff/1/src/ports/SkGlobalInitializ...
src/ports/SkGlobalInitialization_default.cpp:51: // As new subclasses are added
here, they should be reviewed by chrome security before they
could you please also add a note to extend the filter fuzzer? and specify
chrome-security@google.com?

[reed1, 2016-02-04 00:03:17]

https://codereview.chromium.org/1667663004/diff/1/src/ports/SkGlobalInitializ...
File src/ports/SkGlobalInitialization_default.cpp (right):

https://codereview.chromium.org/1667663004/diff/1/src/ports/SkGlobalInitializ...
src/ports/SkGlobalInitialization_default.cpp:51: // As new subclasses are added
here, they should be reviewed by chrome security before they
On 2016/02/03 23:53:58, Oliver Chang wrote:
> could you please also add a note to extend the filter fuzzer? and specify
> mailto:chrome-security@google.com?

Done.

[reed1, 2016-02-04 00:03:38]

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

[commit-bot: I haz the power, 2016-02-04 00:03:48]

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

[Oliver Chang, 2016-02-04 00:03:57]

lgtm, thanks again

[reed1, 2016-02-04 00:04:48]

The CQ bit was unchecked by reed@google.com

[reed1, 2016-02-04 00:05:07]

The CQ bit was checked by reed@google.com

[commit-bot: I haz the power, 2016-02-04 00:05:19]

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

[commit-bot: I haz the power, 2016-02-04 00:31:37]

Description was changed from

==========
add note about security and new subclasses

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

to

==========
add note about security and new subclasses

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

Committed:
https://skia.googlesource.com/skia/+/50b98b424ccf780f76a1a6fb6e24940ffe681d53
==========

[commit-bot: I haz the power, 2016-02-04 00:31:38]

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