Add support for CheckedNumeric bitshift operators

Add support for CheckedNumeric bitshift operators

BUG=665584
Committed: https://crrev.com/3ab540427fc419718e96e8310e817a79a837efbb
Cr-Commit-Position: refs/heads/master@{#432790}

[jschuh, 2016-11-17 00:21:00]

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

[jschuh, 2016-11-17 00:21:01]

ptal

[jschuh, 2016-11-17 00:21:33]

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

[commit-bot: I haz the power, 2016-11-17 00:23:05]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Tom Sepez, 2016-11-17 00:29:46]

lgtm

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

https://codereview.chromium.org/2510973002/diff/20001/base/numerics/safe_math...
base/numerics/safe_math.h:120: template <typename Src>
nit: format as above?

[jschuh, 2016-11-17 00:47:51]

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

https://codereview.chromium.org/2510973002/diff/20001/base/numerics/safe_math...
base/numerics/safe_math.h:120: template <typename Src>
On 2016/11/17 00:29:45, Tom Sepez wrote:
> nit: format as above?

Done. Stupid git cl format.

[jschuh, 2016-11-17 02:47:42]

The CQ bit was checked by jschuh@chromium.org

[jschuh, 2016-11-17 02:47:43]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org
Link to the patchset: https://codereview.chromium.org/2510973002/#ps40001
(title: "format")

[commit-bot: I haz the power, 2016-11-17 02:48:42]

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-17 05:24:00]

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

[commit-bot: I haz the power, 2016-11-17 05:24:01]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jschuh, 2016-11-17 05:26:26]

The CQ bit was checked by jschuh@chromium.org

[commit-bot: I haz the power, 2016-11-17 05:27:14]

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-17 06:59:13]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-17 07:02:29]

Description was changed from

==========
Add support for CheckedNumeric bitshift operators

BUG=665584
==========

to

==========
Add support for CheckedNumeric bitshift operators

BUG=665584

Committed: https://crrev.com/3ab540427fc419718e96e8310e817a79a837efbb
Cr-Commit-Position: refs/heads/master@{#432790}
==========

[commit-bot: I haz the power, 2016-11-17 07:02:30]

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