Make ARRAYSIZE_UNSAFE() just use arraysize().

Make ARRAYSIZE_UNSAFE() just use arraysize().

For every two problems that C++11 causes, it fixes at least one.

R=thestig@chromium.org,tsepez@chromium.org
BUG=423134
Committed: https://crrev.com/eb552d78ea8a447dcecc5d828bc11eae7a2121d3
Cr-Commit-Position: refs/heads/master@{#299419}

[viettrungluu, 2014-10-13 22:37:42]

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

[viettrungluu, 2014-10-13 22:38:39]

tsepez -> chrome/utility/chrome_content_utility_ipc_whitelist.cc
thestig -> everything else

[Tom Sepez, 2014-10-13 22:58:14]

lgtm

[Tom Sepez, 2014-10-13 23:07:13]

https://codereview.chromium.org/647753003/diff/320001/chrome/utility/chrome_c...
File chrome/utility/chrome_content_utility_ipc_whitelist.cc (right):

https://codereview.chromium.org/647753003/diff/320001/chrome/utility/chrome_c...
chrome/utility/chrome_content_utility_ipc_whitelist.cc:24: const size_t
kMessageWhitelistSize = 0;
Alternatively, It looks like this is used only in one place in
chrome_content_utility_client, right after a ENABLE_EXTENSIONS ifdef, as it
turns out.  Maybe that usage gets guarded as well, and this just doesn't exist
unless ENABLE_EXTENSIONS.

[Lei Zhang, 2014-10-13 23:17:18]

lgtm

[viettrungluu, 2014-10-13 23:35:26]

Thanks, all.

On 2014/10/13 23:07:13, Tom Sepez wrote:
>
https://codereview.chromium.org/647753003/diff/320001/chrome/utility/chrome_c...
> File chrome/utility/chrome_content_utility_ipc_whitelist.cc (right):
> 
>
https://codereview.chromium.org/647753003/diff/320001/chrome/utility/chrome_c...
> chrome/utility/chrome_content_utility_ipc_whitelist.cc:24: const size_t
> kMessageWhitelistSize = 0;
> Alternatively, It looks like this is used only in one place in
> chrome_content_utility_client, right after a ENABLE_EXTENSIONS ifdef, as it
> turns out.  Maybe that usage gets guarded as well, and this just doesn't exist
> unless ENABLE_EXTENSIONS.

True, though that particular use doesn't seem intrinsically tied to
ENABLE_EXTENSIONS (i.e., it's conceivable that someone would want to use the
whitelist for something else), so I'll leave it the way it is for now and think
about it some more.

[viettrungluu, 2014-10-13 23:35:31]

The CQ bit was checked by viettrungluu@chromium.org

[commit-bot: I haz the power, 2014-10-13 23:37:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/647753003/320001

[commit-bot: I haz the power, 2014-10-14 01:40:09]

Committed patchset #4 (id:320001)

[commit-bot: I haz the power, 2014-10-14 01:40:54]

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