RefCounted types should not have public destructors, chrome/browser/bookmarks/ edition

chrome/browser/bookmarks/bookmark_html_writer.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.
 
 #include "chrome/browser/bookmarks/bookmark_html_writer.h"
 
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/callback.h"
@@ skipping 133 matching lines @@
 
     Write(kFolderChildrenEnd);
     Write(kNewline);
     // File stream close is forced so that unit test could read it.
     file_stream_.CloseSync();
 
     NotifyOnFinish();
   }
 
  private:
+  friend class base::RefCountedThreadSafe<Writer>;
+
   // Types of text being written out. The type dictates how the text is
   // escaped.
   enum TextType {
     // The text is the value of an html attribute, eg foo in
     // <a href="foo">.
     ATTRIBUTE_VALUE,
 
     // Actual content, eg foo in <h1>foo</h2>.
     CONTENT
   };
 
+  ~Writer() {}
+
   // Opens the file, returning true on success.
   bool OpenFile() {
     int flags = base::PLATFORM_FILE_CREATE_ALWAYS | base::PLATFORM_FILE_WRITE;
     return (file_stream_.OpenSync(path_, flags) == net::OK);
   }
 
   // Increments the indent.
   void IncrementIndent() {
     indent_.resize(indent_.size() + kIndentSize, ' ');
   }
@@ skipping 314 matching lines @@
   // BookmarkModel isn't thread safe (nor would we want to lock it down
   // for the duration of the write), as such we make a copy of the
   // BookmarkModel using BookmarkCodec then write from that.
   if (fetcher == NULL) {
     fetcher = new BookmarkFaviconFetcher(profile, path, observer);
     fetcher->ExportBookmarks();
   }
 }
 
 }  // namespace bookmark_html_writer