Replace ASSERTs with exit condition in StaticBitmapImage::draw

Replace ASSERTs with exit condition in StaticBitmapImage::draw

In degenerate cases (like the layout test added by this change),
the layout box may end up with a negative dimension due to an overflow
in the layout arithmetic.  Is such cases, we should just fail
gracefully by rejecting the draw operation.

BUG=544329
Committed: https://crrev.com/35f1d95ad849975161a609311a265974ac93c71b
Cr-Commit-Position: refs/heads/master@{#386704}

[Justin Novosad, 2016-04-11 18:42:54]

junov@chromium.org changed reviewers:
+ senorblanco@chromium.org

[Stephen White, 2016-04-11 18:54:13]

LGTM

[Justin Novosad, 2016-04-11 18:58:02]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2016-04-11 18:58:33]

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

[commit-bot: I haz the power, 2016-04-11 22:19:31]

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

[commit-bot: I haz the power, 2016-04-11 22:19:32]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Justin Novosad, 2016-04-12 15:32:51]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2016-04-12 15:33:22]

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

[commit-bot: I haz the power, 2016-04-12 16:13:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-12 16:14:55]

Description was changed from

==========
Replace ASSERTs with exit condition in StaticBitmapImage::draw

In degenerate cases (like the layout test added by this change),
the layout box may end up with a negative dimension due to an overflow
in the layout arithmetic.  Is such cases, we should just fail
gracefully by rejecting the draw operation.

BUG=544329
==========

to

==========
Replace ASSERTs with exit condition in StaticBitmapImage::draw

In degenerate cases (like the layout test added by this change),
the layout box may end up with a negative dimension due to an overflow
in the layout arithmetic.  Is such cases, we should just fail
gracefully by rejecting the draw operation.

BUG=544329

Committed: https://crrev.com/35f1d95ad849975161a609311a265974ac93c71b
Cr-Commit-Position: refs/heads/master@{#386704}
==========

[commit-bot: I haz the power, 2016-04-12 16:14:56]

Patchset 1 (id:??) landed as
https://crrev.com/35f1d95ad849975161a609311a265974ac93c71b
Cr-Commit-Position: refs/heads/master@{#386704}