Add CheckedNumeric::AssignIfValid() method

Add CheckedNumeric::AssignIfValid() method

Helper for safely testing and extracting the underlying value.

BUG=668713
Committed: https://crrev.com/e17e0f13e7c600c813674aca0265021da82b7f02
Cr-Commit-Position: refs/heads/master@{#434564}

[scottmg, 2016-11-25 23:47:50]

scottmg@chromium.org changed reviewers:
+ scottmg@chromium.org

[scottmg, 2016-11-25 23:47:51]

lgtm

https://codereview.chromium.org/2535603002/diff/1/base/numerics/safe_math.h
File base/numerics/safe_math.h (right):

https://codereview.chromium.org/2535603002/diff/1/base/numerics/safe_math.h#n...
base/numerics/safe_math.h:46: //          pointer if the value is currently
valid and within the range
indent

[jschuh, 2016-11-25 23:49:52]

The CQ bit was checked by jschuh@chromium.org

[commit-bot: I haz the power, 2016-11-25 23:50:02]

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-11-26 03:00:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480117792438100, "parent_rev":
"d5cdee032d751316a789eb7c2c4c4f7ec036bca5", "commit_rev":
"43166e4ce0453f12181b877253e62977a329c219"}

[commit-bot: I haz the power, 2016-11-26 03:00:46]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-26 03:03:47]

Description was changed from

==========
Add CheckedNumeric::AssignIfValid() method

Helper for safely testing and extracting the underlying value.

BUG=668713
==========

to

==========
Add CheckedNumeric::AssignIfValid() method

Helper for safely testing and extracting the underlying value.

BUG=668713

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

[commit-bot: I haz the power, 2016-11-26 03:03:48]

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

[scottmg, 2016-11-26 05:43:01]

On 2016/11/26 03:03:48, commit-bot: I haz the power wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/e17e0f13e7c600c813674aca0265021da82b7f02
> Cr-Commit-Position: refs/heads/master@{#434564}

lgtm with a comment means you should address the comment. :p