Remove setMayNeedPaintInvalidation call from setNeedsOverflowRecalcAfterStyleChange()

Remove setMayNeedPaintInvalidation call from setNeedsOverflowRecalcAfterStyleChange()

This is a partial revert of https://codereview.chromium.org/2751523011
which caused extra painting cost in some cases. See the bug for data
and analysis.

Remove the call to fix the regression. Will continue to investigate the
case to see if the extra painting cost is legitimate and, if yes,
any optimizations to avoid the cost.

BUG=704182
Review-Url: https://codereview.chromium.org/2776583003
Cr-Commit-Position: refs/heads/master@{#459556}
Committed: https://chromium.googlesource.com/chromium/src/+/d3fc0f37f5b506fbc57edfdeebe02cbeac5b3002

[Xianzhu, 2017-03-24 19:57:52]

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

[Xianzhu, 2017-03-24 19:58:03]

wangxianzhu@chromium.org changed reviewers:
+ pdr@chromium.org

[commit-bot: I haz the power, 2017-03-24 19:58:12]

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

[pdr., 2017-03-24 20:00:23]

On 2017/03/24 at 19:58:03, wangxianzhu wrote:
> 

LGTM

[Xianzhu, 2017-03-24 20:22:24]

The CQ bit was unchecked by wangxianzhu@chromium.org

[Xianzhu, 2017-03-24 20:22:27]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2017-03-24 20:22:38]

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, 2017-03-24 21:21:50]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490386947707390, "parent_rev":
"9b5e6ff303ed188622c81186bb86fe37e60ebb67", "commit_rev":
"d3fc0f37f5b506fbc57edfdeebe02cbeac5b3002"}

[commit-bot: I haz the power, 2017-03-24 21:22:30]

Description was changed from

==========
Remove setMayNeedPaintInvalidation call from
setNeedsOverflowRecalcAfterStyleChange()

This is a partial revert of https://codereview.chromium.org/2751523011
which caused extra painting cost in some cases. See the bug for data
and analysis.

Remove the call to fix the regression. Will continue to investigate the
case to see if the extra painting cost is legitimate and, if yes,
any optimizations to avoid the cost.

BUG=704182
==========

to

==========
Remove setMayNeedPaintInvalidation call from
setNeedsOverflowRecalcAfterStyleChange()

This is a partial revert of https://codereview.chromium.org/2751523011
which caused extra painting cost in some cases. See the bug for data
and analysis.

Remove the call to fix the regression. Will continue to investigate the
case to see if the extra painting cost is legitimate and, if yes,
any optimizations to avoid the cost.

BUG=704182

Review-Url: https://codereview.chromium.org/2776583003
Cr-Commit-Position: refs/heads/master@{#459556}
Committed:
https://chromium.googlesource.com/chromium/src/+/d3fc0f37f5b506fbc57edfdeebe0...
==========

[commit-bot: I haz the power, 2017-03-24 21:22:31]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/d3fc0f37f5b506fbc57edfdeebe0...