RefCounted types should not have public destructors, webkit/

webkit/fileapi/file_writer_delegate_unittest.cc

 // Copyright (c) 2012 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.
 //
 // NOTE: These tests are run as part of "unit_tests" (in chrome/test/unit)
 // rather than as part of test_shell_tests because they rely on being able
 // to instantiate a MessageLoop of type TYPE_IO.  test_shell_tests uses
 // TYPE_UI, which URLRequest doesn't allow.
 //
 
@@ skipping 140 matching lines @@
         remaining_bytes_(content.length()),
         cursor_(0) {
   }
 
   virtual void Start() OVERRIDE {
     MessageLoop::current()->PostTask(
         FROM_HERE,
         base::Bind(&FileWriterDelegateTestJob::NotifyHeadersComplete, this));
   }
 
-  virtual bool ReadRawData(net::IOBuffer* buf, int buf_size, int *bytes_read)
-      OVERRIDE {
+  virtual bool ReadRawData(net::IOBuffer* buf,
+                           int buf_size,
+                           int *bytes_read) OVERRIDE {
     if (remaining_bytes_ < buf_size)
       buf_size = static_cast<int>(remaining_bytes_);
 
     for (int i = 0; i < buf_size; ++i)
       buf->data()[i] = content_[cursor_++];
     remaining_bytes_ -= buf_size;
 
     SetStatus(net::URLRequestStatus());
     *bytes_read = buf_size;
     return true;
   }
 
   virtual int GetResponseCode() const OVERRIDE {
     return 200;
   }
 
+ protected:
+  virtual ~FileWriterDelegateTestJob() {}
+
  private:
   std::string content_;
   int remaining_bytes_;
   int cursor_;
 };
 
 }  // namespace (anonymous)
 
 // static
 net::URLRequestJob* FileWriterDelegateTest::Factory(
@@ skipping 236 matching lines @@
   EXPECT_EQ(base::PLATFORM_FILE_ERROR_NO_SPACE, result_->status());
   EXPECT_TRUE(result_->complete());
 }
 
 class FileWriterDelegateUnlimitedTest : public FileWriterDelegateTest {
  protected:
   virtual void SetUpTestHelper(const FilePath& path) OVERRIDE;
 };
 
 }  // namespace fileapi