Delete code in PointIsClippedByAncestorClipNode which checked render surface clip.

Delete code in PointIsClippedByAncestorClipNode which checked render surface clip.

Since all actual semantic clips are represented by the clip tree, this code
was only necessary to check for cases when a surface exceeded the maximum
texture size. Since the rendering would already be broken in such cases,
it seems ok to not count that as a clip, which is correct w.r.t. the semantics
of the web page, even if it disagrees with the apparent rendering.

Note: the relevant code was added in https://codereview.chromium.org/1551333003.

BUG=718149
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2858053003
Cr-Commit-Position: refs/heads/master@{#469183}
Committed: https://chromium.googlesource.com/chromium/src/+/51b96b368bc1ebb4b952465a4220f093f6850528

[chrishtr, 2017-05-03 21:31:10]

Description was changed from

==========
none

BUG=
==========

to

==========
none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[chrishtr, 2017-05-03 21:36:37]

Description was changed from

==========
none

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Delete code in PointIsClippedByAncestorClipNode which checked render surface
clip.

Since all actual semantic clips are represented by the clip tree, this code
was only necessary to check for cases when a surface exceeded the maximum
texture size. Since the rendering would already be broken in such cases,
it seems ok to not count that as a clip, which is correct w.r.t. the semantics
of the web page, even if it disagrees with the apparent rendering.

BUG=718149
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[chrishtr, 2017-05-03 21:37:04]

Description was changed from

==========
Delete code in PointIsClippedByAncestorClipNode which checked render surface
clip.

Since all actual semantic clips are represented by the clip tree, this code
was only necessary to check for cases when a surface exceeded the maximum
texture size. Since the rendering would already be broken in such cases,
it seems ok to not count that as a clip, which is correct w.r.t. the semantics
of the web page, even if it disagrees with the apparent rendering.

BUG=718149
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Delete code in PointIsClippedByAncestorClipNode which checked render surface
clip.

Since all actual semantic clips are represented by the clip tree, this code
was only necessary to check for cases when a surface exceeded the maximum
texture size. Since the rendering would already be broken in such cases,
it seems ok to not count that as a clip, which is correct w.r.t. the semantics
of the web page, even if it disagrees with the apparent rendering.

Note: the relevant code was added in https://codereview.chromium.org/1551333003.

BUG=718149
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[chrishtr, 2017-05-03 21:37:15]

chrishtr@chromium.org changed reviewers:
+ ajuma@chromium.org, jaydasika@chromium.org

[chrishtr, 2017-05-03 21:37:20]

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

[commit-bot: I haz the power, 2017-05-03 21:38:16]

Dry run: CQ is trying da patch.

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

[jaydasika, 2017-05-03 21:42:59]

lgtm % ajuma

[ajuma, 2017-05-03 21:48:49]

lgtm too

[chrishtr, 2017-05-03 21:49:20]

The CQ bit was unchecked by chrishtr@chromium.org

[chrishtr, 2017-05-03 21:49:22]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2017-05-03 21:49:48]

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

[commit-bot: I haz the power, 2017-05-03 21:49:50]

This CL has an open dependency (Issue 2861593004 Patch 1). Please resolve the
dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[chrishtr, 2017-05-03 22:59:50]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2017-05-03 23:00: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-05-03 23:06:12]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493852390884220, "parent_rev":
"92d03c12fddf6214bcb74ec328a284f39eb2f452", "commit_rev":
"51b96b368bc1ebb4b952465a4220f093f6850528"}

[commit-bot: I haz the power, 2017-05-03 23:06:23]

Description was changed from

==========
Delete code in PointIsClippedByAncestorClipNode which checked render surface
clip.

Since all actual semantic clips are represented by the clip tree, this code
was only necessary to check for cases when a surface exceeded the maximum
texture size. Since the rendering would already be broken in such cases,
it seems ok to not count that as a clip, which is correct w.r.t. the semantics
of the web page, even if it disagrees with the apparent rendering.

Note: the relevant code was added in https://codereview.chromium.org/1551333003.

BUG=718149
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Delete code in PointIsClippedByAncestorClipNode which checked render surface
clip.

Since all actual semantic clips are represented by the clip tree, this code
was only necessary to check for cases when a surface exceeded the maximum
texture size. Since the rendering would already be broken in such cases,
it seems ok to not count that as a clip, which is correct w.r.t. the semantics
of the web page, even if it disagrees with the apparent rendering.

Note: the relevant code was added in https://codereview.chromium.org/1551333003.

BUG=718149
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2858053003
Cr-Commit-Position: refs/heads/master@{#469183}
Committed:
https://chromium.googlesource.com/chromium/src/+/51b96b368bc1ebb4b952465a4220...
==========

[commit-bot: I haz the power, 2017-05-03 23:06:24]

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