Fix some div by 0s in safe_math_impl.h

Fix some div by 0s in safe_math_impl.h

The majority of these are already upstream in base/, the
remainder will need upstreaming. Also pull some upstream
changes to reduce diffing.

Upstream CL is https://codereview.chromium.org/2440143003/

BUG=657436
Committed: https://pdfium.googlesource.com/pdfium/+/a804fd48b71679ece1f1c2284c3c1835c9e20441

[Tom Sepez, 2016-10-21 17:50:47]

Description was changed from

==========
Fix some div by 0s in safe_math_impl.h

The majority of these are already upstream in base/, the
remainder will need upstreaming.

BUG=657436
==========

to

==========
Fix some div by 0s in safe_math_impl.h

The majority of these are already upstream in base/, the
remainder will need upstreaming. Also pull some upstream
changes to reduce diffing.

BUG=657436
==========

[Tom Sepez, 2016-10-21 17:52:09]

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

[commit-bot: I haz the power, 2016-10-21 17:52:10]

Dry run: 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, 2016-10-21 18:09:04]

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

[commit-bot: I haz the power, 2016-10-21 18:09:04]

Dry run: This issue passed the CQ dry run.

[Tom Sepez, 2016-10-21 18:34:01]

Description was changed from

==========
Fix some div by 0s in safe_math_impl.h

The majority of these are already upstream in base/, the
remainder will need upstreaming. Also pull some upstream
changes to reduce diffing.

BUG=657436
==========

to

==========
Fix some div by 0s in safe_math_impl.h

The majority of these are already upstream in base/, the
remainder will need upstreaming. Also pull some upstream
changes to reduce diffing.

Upstream CL is https://codereview.chromium.org/2440143003/

BUG=657436
==========

[Tom Sepez, 2016-10-21 18:34:01]

tsepez@chromium.org changed reviewers:
+ thestig@chromium.org

[Tom Sepez, 2016-10-21 18:49:14]

Regardless of what they do upstream, I think we want the div by 0 protection for
ourselves.

[Lei Zhang, 2016-10-21 18:52:34]

lgtm

[Tom Sepez, 2016-10-21 19:06:21]

The CQ bit was checked by tsepez@chromium.org

[commit-bot: I haz the power, 2016-10-21 19:06:30]

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, 2016-10-21 19:06:47]

Description was changed from

==========
Fix some div by 0s in safe_math_impl.h

The majority of these are already upstream in base/, the
remainder will need upstreaming. Also pull some upstream
changes to reduce diffing.

Upstream CL is https://codereview.chromium.org/2440143003/

BUG=657436
==========

to

==========
Fix some div by 0s in safe_math_impl.h

The majority of these are already upstream in base/, the
remainder will need upstreaming. Also pull some upstream
changes to reduce diffing.

Upstream CL is https://codereview.chromium.org/2440143003/

BUG=657436

Committed:
https://pdfium.googlesource.com/pdfium/+/a804fd48b71679ece1f1c2284c3c1835c9e2...
==========

[commit-bot: I haz the power, 2016-10-21 19:06:47]

Committed patchset #2 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/a804fd48b71679ece1f1c2284c3c1835c9e2...