Mojo: Specify/check alignment of pointers more carefully.

Mojo: Specify/check alignment of pointers more carefully.

Improve the verification routines especially in the case that no count
is necessary.

Also add a test.

R=darin@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=273936

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=274461

[darin (slow to review), 2014-05-30 22:38:46]

LGTM

https://codereview.chromium.org/304303006/diff/1/mojo/system/memory_unittest.cc
File mojo/system/memory_unittest.cc (right):

https://codereview.chromium.org/304303006/diff/1/mojo/system/memory_unittest....
mojo/system/memory_unittest.cc:132: 
nit: extraneous newline

[viettrungluu, 2014-05-30 23:04:16]

Thanks.

https://codereview.chromium.org/304303006/diff/1/mojo/system/memory_unittest.cc
File mojo/system/memory_unittest.cc (right):

https://codereview.chromium.org/304303006/diff/1/mojo/system/memory_unittest....
mojo/system/memory_unittest.cc:132: 
On 2014/05/30 22:38:46, darin wrote:
> nit: extraneous newline

Oops, already fixed. I forgot to save the file before committing/uploading.

[viettrungluu, 2014-05-30 23:37:25]

Committed patchset #2 manually as r273936 (presubmit successful).

[DaleCurtis, 2014-05-31 22:06:48]

A revert of this CL has been created in
https://codereview.chromium.org/312463002/ by dalecurtis@chromium.org.

The reason for reverting is: Broke the windows bots:
http://build.chromium.org/p/chromium.win/builders/Vista%20Tests%20%283%29/bui...
http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%28....

[viettrungluu, 2014-06-02 23:18:43]

The CQ bit was checked by viettrungluu@chromium.org

[commit-bot: I haz the power, 2014-06-02 23:20:10]

CQ is trying da patch. Follow status at

https://chromium-status.appspot.com/cq/viettrungluu@chromium.org/304303006/40001

[commit-bot: I haz the power, 2014-06-03 03:44:52]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_x64_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/win_chromium_x64_rel/...)

[commit-bot: I haz the power, 2014-06-03 10:37:18]

Change committed as 274461