Update audio code to use size_t more correctly, webrtc/ portion.

Update audio code to use size_t more correctly, webrtc/ portion.

This is a piece of https://codereview.webrtc.org/1230503003 , split out to a
separate change to make reviewing easier.

BUG=chromium:81439
TEST=none

[Peter Kasting, 2015-07-09 22:41:39]

pkasting@chromium.org changed reviewers:
+ niklas.enbom@webrtc.org

[Peter Kasting, 2015-07-11 08:31:16]

pkasting@chromium.org changed reviewers:
+ henrika@webrtc.org
- niklas.enbom@webrtc.org

[Peter Kasting, 2015-07-11 08:31:16]

-> henrika

[henrika_webrtc, 2015-07-13 08:01:47]

henrika@webrtc.org changed reviewers:
+ pbos@webrtc.org

[pbos-webrtc, 2015-07-13 08:13:30]

https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/interf...
File webrtc/system_wrappers/interface/aligned_array.h (right):

https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/interf...
webrtc/system_wrappers/interface/aligned_array.h:23: AlignedArray(int rows,
size_t cols, int alignment)
size_t for row and alignment as well?

https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/source...
File webrtc/system_wrappers/source/aligned_array_unittest.cc (right):

https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/source...
webrtc/system_wrappers/source/aligned_array_unittest.cc:57:
ASSERT_EQ(arr.cols(), 7U);
7u imo, not going to be annoying about it

[Peter Kasting, 2015-07-16 23:59:43]

https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/interf...
File webrtc/system_wrappers/interface/aligned_array.h (right):

https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/interf...
webrtc/system_wrappers/interface/aligned_array.h:23: AlignedArray(int rows,
size_t cols, int alignment)
On 2015/07/13 at 08:13:30, pbos-webrtc wrote:
> size_t for row and alignment as well?

Yes, but doing this properly involves converting channel counts to size_t, which
is another big change: see https://codereview.webrtc.org/1238083005 for that. 
So I'll leave this as-is in this change (both in this small piece and in the
larger merged version) and then make that other change after all this stuff
lands.

https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/source...
File webrtc/system_wrappers/source/aligned_array_unittest.cc (right):

https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/source...
webrtc/system_wrappers/source/aligned_array_unittest.cc:57:
ASSERT_EQ(arr.cols(), 7U);
On 2015/07/13 at 08:13:30, pbos-webrtc wrote:
> 7u imo, not going to be annoying about it

Done.

[pbos-webrtc, 2015-07-17 04:02:27]

On 2015/07/16 23:59:43, Peter Kasting wrote:
>
https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/interf...
> File webrtc/system_wrappers/interface/aligned_array.h (right):
> 
>
https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/interf...
> webrtc/system_wrappers/interface/aligned_array.h:23: AlignedArray(int rows,
> size_t cols, int alignment)
> On 2015/07/13 at 08:13:30, pbos-webrtc wrote:
> > size_t for row and alignment as well?
> 
> Yes, but doing this properly involves converting channel counts to size_t,
which
> is another big change: see https://codereview.webrtc.org/1238083005 for that. 
> So I'll leave this as-is in this change (both in this small piece and in the
> larger merged version) and then make that other change after all this stuff
> lands.
> 
>
https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/source...
> File webrtc/system_wrappers/source/aligned_array_unittest.cc (right):

That's fine by me, thanks for fixing the follow-up as well. lgtm. :)

>
https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/source...
> webrtc/system_wrappers/source/aligned_array_unittest.cc:57:
> ASSERT_EQ(arr.cols(), 7U);
> On 2015/07/13 at 08:13:30, pbos-webrtc wrote:
> > 7u imo, not going to be annoying about it
> 
> Done.

[pbos-webrtc, 2015-07-17 04:02:30]

On 2015/07/16 23:59:43, Peter Kasting wrote:
>
https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/interf...
> File webrtc/system_wrappers/interface/aligned_array.h (right):
> 
>
https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/interf...
> webrtc/system_wrappers/interface/aligned_array.h:23: AlignedArray(int rows,
> size_t cols, int alignment)
> On 2015/07/13 at 08:13:30, pbos-webrtc wrote:
> > size_t for row and alignment as well?
> 
> Yes, but doing this properly involves converting channel counts to size_t,
which
> is another big change: see https://codereview.webrtc.org/1238083005 for that. 
> So I'll leave this as-is in this change (both in this small piece and in the
> larger merged version) and then make that other change after all this stuff
> lands.
> 
>
https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/source...
> File webrtc/system_wrappers/source/aligned_array_unittest.cc (right):

That's fine by me, thanks for fixing the follow-up as well. lgtm. :)

>
https://codereview.webrtc.org/1227893002/diff/1/webrtc/system_wrappers/source...
> webrtc/system_wrappers/source/aligned_array_unittest.cc:57:
> ASSERT_EQ(arr.cols(), 7U);
> On 2015/07/13 at 08:13:30, pbos-webrtc wrote:
> > 7u imo, not going to be annoying about it
> 
> Done.