scoped_ptr and new[] don't mix.

chrome_frame/test/urlmon_moniker_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <atlbase.h>
 #include <atlcom.h>
 
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/win/scoped_comptr.h"
@@ skipping 155 matching lines @@
   ASSERT_TRUE(ReadFileAsString(kSmallHtmlMetaTag, &small_html_meta_tag));
   ASSERT_TRUE(StringToStream(small_html_meta_tag, stream_with_meta.Receive()));
   EXPECT_HRESULT_SUCCEEDED(sniffer.ReadIntoCache(stream_with_meta, true));
 
   CComObjectStackEx<MockBindStatusCallbackImpl> mock;
   const CLIPFORMAT cf = 0xd0d0;
   const DWORD flags = BSCF_FIRSTDATANOTIFICATION | BSCF_LASTDATANOTIFICATION;
   const DWORD data_size = small_html_meta_tag.size();
 
   DWORD read_size1 = data_size * 2;
-  scoped_ptr<char> read_buffer1(new char[read_size1]);
+  scoped_array<char> read_buffer1(new char[read_size1]);
   ZeroMemory(read_buffer1.get(), read_size1);
 
   char read_buffer2[10] = {0};
   DWORD read_size2 = sizeof(read_buffer2);
   HRESULT ret1 = E_FAIL, ret2 = E_FAIL;
 
   EXPECT_CALL(mock, OnDataAvailable(flags, data_size,
                     testing::Field(&FORMATETC::cfFormat, cf),
                     testing::Field(&STGMEDIUM::tymed, TYMED_ISTREAM)))
       .WillOnce(testing::DoAll(
@@ skipping 23 matching lines @@
   ASSERT_TRUE(StringToStream(small_html_meta_tag, stream_with_meta.Receive()));
   EXPECT_HRESULT_SUCCEEDED(sniffer.ReadIntoCache(stream_with_meta, true));
 
   CComObjectStackEx<MockBindStatusCallbackImpl> mock;
   const CLIPFORMAT cf = 0xd0d0;
   const DWORD flags = BSCF_FIRSTDATANOTIFICATION | BSCF_LASTDATANOTIFICATION;
   const DWORD data_size = small_html_meta_tag.size();
 
   DWORD read_size1 = data_size / 2; // First read is half the data.
   DWORD read_size2 = data_size; // Second read, try to read past data.
-  scoped_ptr<char> read_buffer1(new char[read_size1]);
-  scoped_ptr<char> read_buffer2(new char[read_size2]);
+  scoped_array<char> read_buffer1(new char[read_size1]);
+  scoped_array<char> read_buffer2(new char[read_size2]);
   ZeroMemory(read_buffer1.get(), read_size1);
   ZeroMemory(read_buffer2.get(), read_size2);
 
   char read_buffer3[10] = {0};
   DWORD read_size3 = sizeof(read_buffer3);
   HRESULT ret1 = E_FAIL, ret2 = E_FAIL, ret3 = E_FAIL;
 
   EXPECT_CALL(mock, OnDataAvailable(flags, data_size,
                     testing::Field(&FORMATETC::cfFormat, cf),
                     testing::Field(&STGMEDIUM::tymed, TYMED_ISTREAM)))
@@ skipping 12 matching lines @@
 
   std::string data_read;
   data_read.append(read_buffer1.get(), read_size1);
   data_read.append(read_buffer2.get(), read_size2);
   EXPECT_EQ(small_html_meta_tag, data_read);
 
   EXPECT_EQ(S_FALSE, ret3);
   EXPECT_STREQ("", read_buffer3);
   EXPECT_EQ(sizeof(read_buffer3), read_size3);
 }