Don't assert rect mapping result if impossible, and enable assert in debug build

Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,mac_blink_dbg,win_blink_dbg

Committed: https://crrev.com/c46c639d88881940f8ce4cb2fe6b50c41739d9de
Cr-Commit-Position: refs/heads/master@{#390814}

[Xianzhu, 2016-04-05 01:27:41]

Description was changed from

==========
Don't check fast-path/slow-path rect mapping if the rect may have been saturated
==========

to

==========
Don't assert rect mapping result if the rect may have been saturated, and enable
assert
==========

[Xianzhu, 2016-04-06 00:25:39]

Description was changed from

==========
Don't assert rect mapping result if the rect may have been saturated, and enable
assert
==========

to

==========
Don't assert rect mapping result if impossible, and enable assert

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.
==========

[Xianzhu, 2016-04-06 00:25:51]

Description was changed from

==========
Don't assert rect mapping result if impossible, and enable assert

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.
==========

to

==========
Don't assert rect mapping result if impossible, and enable assert

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
==========

[Xianzhu, 2016-04-06 23:22:14]

Description was changed from

==========
Don't assert rect mapping result if impossible, and enable assert

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
==========

to

==========
Don't assert rect mapping result if impossible, and enable assert

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=linux_blink_dbg,mac_blink_dbg,win_blink_dbg
==========

[Xianzhu, 2016-04-29 19:42:51]

Description was changed from

==========
Don't assert rect mapping result if impossible, and enable assert

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=linux_blink_dbg,mac_blink_dbg,win_blink_dbg
==========

to

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=linux_blink_dbg,mac_blink_dbg,win_blink_dbg
==========

[Xianzhu, 2016-04-29 19:43:25]

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

[chrishtr, 2016-04-29 21:22:08]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2016-04-29 21:22:09]

lgtm

[commit-bot: I haz the power, 2016-04-29 21:22:28]

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

[commit-bot: I haz the power, 2016-04-29 21:22:29]

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

[commit-bot: I haz the power, 2016-04-29 21:22:30]

Transient error: Invalid delimiter in
"linux_blink_dbg,mac_blink_dbg,win_blink_dbg":
Correct syntax is "tryserver:bot1,bot2;tryserver2:bot3,bot4;".

[Xianzhu, 2016-04-29 21:28:10]

Description was changed from

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=linux_blink_dbg,mac_blink_dbg,win_blink_dbg
==========

to

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,tryserver.blink:mac_blink_dbg,tryserver.blink:win_blink_dbg
==========

[Xianzhu, 2016-04-29 21:28:16]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2016-04-29 21:28:38]

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

[commit-bot: I haz the power, 2016-04-29 21:28:39]

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

[commit-bot: I haz the power, 2016-04-29 21:28:40]

Transient error: Too many
"tryserver.blink:linux_blink_dbg,tryserver.blink:mac_blink_dbg,tryserver.blink:win_blink_dbg"
delimiters in ":".
Correct syntax is "tryserver:bot1,bot2;tryserver2:bot3,bot4;".

[Xianzhu, 2016-04-29 21:54:27]

Description was changed from

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,tryserver.blink:mac_blink_dbg,tryserver.blink:win_blink_dbg
==========

to

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,mac_blink_dbg,win_blink_dbg
==========

[Xianzhu, 2016-04-29 21:54:34]

The CQ bit was checked by wangxianzhu@chromium.org

[commit-bot: I haz the power, 2016-04-29 21:54:48]

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

[commit-bot: I haz the power, 2016-04-30 00:09:53]

Description was changed from

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,mac_blink_dbg,win_blink_dbg
==========

to

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,mac_blink_dbg,win_blink_dbg
==========

[commit-bot: I haz the power, 2016-04-30 00:09:54]

Committed patchset #15 (id:280001)

[Dirk Pranke, 2016-04-30 01:47:54]

A revert of this CL (patchset #15 id:280001) has been created in
https://codereview.chromium.org/1934863002/ by dpranke@chromium.org.

The reason for reverting is: I think this probably broke one of the telemetry
benchmarks (which looks like it's hitting a crash inside blink now):

https://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%2...
https://build.chromium.org/p/chromium.mac/builders/Mac10.9%20Tests%20%28dbg%2....

[commit-bot: I haz the power, 2016-04-30 17:30:24]

Patchset 15 (id:??) landed as
https://crrev.com/c46c639d88881940f8ce4cb2fe6b50c41739d9de
Cr-Commit-Position: refs/heads/master@{#390814}

[commit-bot: I haz the power, 2016-04-30 18:03:39]

Description was changed from

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,mac_blink_dbg,win_blink_dbg
==========

to

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,mac_blink_dbg,win_blink_dbg

Committed: https://crrev.com/c46c639d88881940f8ce4cb2fe6b50c41739d9de
Cr-Commit-Position: refs/heads/master@{#390814}
==========

[Xianzhu, 2016-05-02 21:39:45]

Description was changed from

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,mac_blink_dbg,win_blink_dbg

Committed: https://crrev.com/c46c639d88881940f8ce4cb2fe6b50c41739d9de
Cr-Commit-Position: refs/heads/master@{#390814}
==========

to

==========
Don't assert rect mapping result if impossible, and enable assert in debug build

Avoid rect mapping result checking in the following cases:
- Any rect may contain saturated values;
- If a fixed-position ancestor disabled clipping.

BUG=591199
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_dbg,mac_blink_dbg,win_blink_dbg

Committed: https://crrev.com/c46c639d88881940f8ce4cb2fe6b50c41739d9de
Cr-Commit-Position: refs/heads/master@{#390814}
==========