Add 10 Skia methods to CFI blacklist for perf reasons.

Add 10 Skia methods to CFI blacklist for perf reasons.

While they have not been observed to slow down real-world use cases,
some blink_layout microbenchmarks feel better with these methods
disabled. In order to be concervative at the launch time, lift
the CFI defense for these methods.

BUG=634139
Committed: https://crrev.com/4e341f5368a22ceeab4d5133f03ec6a71729737f
Cr-Commit-Position: refs/heads/master@{#412979}

[krasin1, 2016-08-18 22:44:39]

krasin@chromium.org changed reviewers:
+ kcc@chromium.org, krasin@chromium.org

[krasin1, 2016-08-18 22:44:40]

Hi Kostya,

please, review this tiny CL that eliminates CFI perf impact on skia.

[kcc2, 2016-08-18 22:45:53]

The CQ bit was checked by kcc@chromium.org

[kcc2, 2016-08-18 22:45:53]

lgtm

[commit-bot: I haz the power, 2016-08-18 22:46:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-18 22:46:36]

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

[commit-bot: I haz the power, 2016-08-18 22:46:37]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[krasin1, 2016-08-18 22:46:49]

(self LGTM)

[krasin1, 2016-08-18 22:46:54]

The CQ bit was checked by krasin@chromium.org

[commit-bot: I haz the power, 2016-08-18 22:47:49]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

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

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-19 00:11:06]

Description was changed from

==========
Add 10 Skia methods to CFI blacklist for perf reasons.

While they have not been observed to slow down real-world use cases,
some blink_layout microbenchmarks feel better with these methods
disabled. In order to be concervative at the launch time, lift
the CFI defense for these methods.

BUG=634139
==========

to

==========
Add 10 Skia methods to CFI blacklist for perf reasons.

While they have not been observed to slow down real-world use cases,
some blink_layout microbenchmarks feel better with these methods
disabled. In order to be concervative at the launch time, lift
the CFI defense for these methods.

BUG=634139

Committed: https://crrev.com/4e341f5368a22ceeab4d5133f03ec6a71729737f
Cr-Commit-Position: refs/heads/master@{#412979}
==========

[commit-bot: I haz the power, 2016-08-19 00:11:07]

Patchset 2 (id:??) landed as
https://crrev.com/4e341f5368a22ceeab4d5133f03ec6a71729737f
Cr-Commit-Position: refs/heads/master@{#412979}