Don't include descendants into LocalVisualRect if a box has both mask and overflow clip

Don't include descendants into LocalVisualRect if a box has both mask and overflow clip

This avoids unnecessary paint invalidation when the box's descedants
change visual overflow which doesn't affect the box's local visual rect
because of the overflow clip.

About the bug: the page contains the following HTML:
  <div style="overflow: hidden; -webkit-mask-image: url(#a)>
    <div class="animating rotation transform"></div>
  </div>
The animating child changes the container's layout overflow.
https://codereview.chromium.org/2791933003/ fixed an under-invalidation
issue when layout overflow changes, but unnecessarily invalidated the
container because previously LayoutBox::LocalVisualRect() also included
descendant visual overflow if there is a mask regardless of overflow
clip.

BUG=709054
Review-Url: https://codereview.chromium.org/2826273005
Cr-Commit-Position: refs/heads/master@{#466243}
TBR=wangxianzhu@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2833263002
Cr-Commit-Position: refs/branch-heads/3071@{#141}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
Committed: https://chromium.googlesource.com/chromium/src/+/198917a86accada7eb94e50b93ca0a7253fce823

[Xianzhu, 2017-04-22 03:41:17]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2017-04-22 03:41:34]

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-04-22 03:42:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492832477863090, "parent_rev":
"955a01b06b3cb7fb6ca51acd085313439c0ef5bb", "commit_rev":
"198917a86accada7eb94e50b93ca0a7253fce823"}

[commit-bot: I haz the power, 2017-04-22 03:42:51]

Description was changed from

==========
Don't include descendants into LocalVisualRect if a box has both mask and
overflow clip

This avoids unnecessary paint invalidation when the box's descedants
change visual overflow which doesn't affect the box's local visual rect
because of the overflow clip.

About the bug: the page contains the following HTML:
  <div style="overflow: hidden; -webkit-mask-image: url(#a)>
    <div class="animating rotation transform"></div>
  </div>
The animating child changes the container's layout overflow.
https://codereview.chromium.org/2791933003/ fixed an under-invalidation
issue when layout overflow changes, but unnecessarily invalidated the
container because previously LayoutBox::LocalVisualRect() also included
descendant visual overflow if there is a mask regardless of overflow
clip.

BUG=709054

Review-Url: https://codereview.chromium.org/2826273005
Cr-Commit-Position: refs/heads/master@{#466243}
TBR=wangxianzhu@chromium.org
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
Don't include descendants into LocalVisualRect if a box has both mask and
overflow clip

This avoids unnecessary paint invalidation when the box's descedants
change visual overflow which doesn't affect the box's local visual rect
because of the overflow clip.

About the bug: the page contains the following HTML:
  <div style="overflow: hidden; -webkit-mask-image: url(#a)>
    <div class="animating rotation transform"></div>
  </div>
The animating child changes the container's layout overflow.
https://codereview.chromium.org/2791933003/ fixed an under-invalidation
issue when layout overflow changes, but unnecessarily invalidated the
container because previously LayoutBox::LocalVisualRect() also included
descendant visual overflow if there is a mask regardless of overflow
clip.

BUG=709054

Review-Url: https://codereview.chromium.org/2826273005
Cr-Commit-Position: refs/heads/master@{#466243}
TBR=wangxianzhu@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2833263002
Cr-Commit-Position: refs/branch-heads/3071@{#141}
Cr-Branched-From:
a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}
Committed:
https://chromium.googlesource.com/chromium/src/+/198917a86accada7eb94e50b93ca...
==========

[commit-bot: I haz the power, 2017-04-22 03:42:51]

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