base: Convert scoped_arrays to the new scoped_ptr style.

base: Convert scoped_arrays to the new scoped_ptr style.

BUG=109874
R=darin@chromium.org,ajwong@chromium.org


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

[awong, 2013-01-16 18:56:06]

most mechanical changes look okay with the exception of callback_internal.h

https://codereview.chromium.org/11961021/diff/1/base/callback_internal.h
File base/callback_internal.h (right):

https://codereview.chromium.org/11961021/diff/1/base/callback_internal.h#newc...
base/callback_internal.h:140: struct CallbackParamTraits<scoped_ptr<T[]> > {
Does this compile?

If we make this change, we remove support for for scoped_array<> from Bind(). 
Are there no users of scoped_array & Bind() in chromium?

[tfarina, 2013-01-16 19:00:28]

https://codereview.chromium.org/11961021/diff/1/base/callback_internal.h
File base/callback_internal.h (right):

https://codereview.chromium.org/11961021/diff/1/base/callback_internal.h#newc...
base/callback_internal.h:140: struct CallbackParamTraits<scoped_ptr<T[]> > {
On 2013/01/16 18:56:06, awong wrote:
> Does this compile?
> 
I compiled only base_unittests.

> If we make this change, we remove support for for scoped_array<> from Bind(). 
> Are there no users of scoped_array & Bind() in chromium?

I'll revert those changes for now.

[awong, 2013-01-16 19:55:31]

LGTM

[darin (slow to review), 2013-01-16 22:33:36]

LGTM

[commit-bot: I haz the power, 2013-01-18 12:15:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tfarina@chromium.org/11961021/4002

[commit-bot: I haz the power, 2013-01-18 14:22:00]

Change committed as 177670

[dcheng, 2013-01-20 01:47:21]

https://codereview.chromium.org/11961021/diff/4002/base/nix/mime_util_xdg.cc
File base/nix/mime_util_xdg.cc (right):

https://codereview.chromium.org/11961021/diff/4002/base/nix/mime_util_xdg.cc#...
base/nix/mime_util_xdg.cc:163: scoped_ptr<SubDirInfo[]> info_array_;  // List of
sub-directories.
FYI, there are some issues we still need to resolve with scoped_ptr<T[]>. To
reduce the amount of work to resolve these issues, please hold off on making
more patches removing scoped_array for now (in this case, crbug.com/171118 is
relevant)