Third patch in getting rid of caching MessageLoop pointers and always using C...

chrome/browser/bookmarks/bookmark_html_writer.cc

 // Copyright (c) 2006-2008 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 "app/l10n_util.h"
 #include "base/file_path.h"
 #include "base/message_loop.h"
 #include "base/platform_file.h"
 #include "base/scoped_ptr.h"
 #include "base/string_util.h"
 #include "base/time.h"
 #include "base/values.h"
 #include "chrome/browser/bookmarks/bookmark_codec.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
+#include "chrome/browser/chrome_thread.h"
 #include "chrome/browser/history/history_types.h"
 #include "grit/generated_resources.h"
 #include "net/base/escape.h"
 #include "net/base/file_stream.h"
 #include "net/base/net_errors.h"
 
 namespace bookmark_html_writer {
 
 namespace {
 
@@ skipping 282 matching lines @@
   // File we're writing to.
   net::FileStream file_stream_;
 
   // How much we indent when writing a bookmark/folder. This is modified
   // via IncrementIndent and DecrementIndent.
   std::string indent_;
 };
 
 }  // namespace
 
-void WriteBookmarks(MessageLoop* thread,
-                    BookmarkModel* model,
-                    const FilePath& path) {
+void WriteBookmarks(BookmarkModel* model, const FilePath& path) {
   // 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.
   BookmarkCodec codec;
-  scoped_ptr<Writer> writer(new Writer(codec.Encode(model), path));
-  if (thread)
-    thread->PostTask(FROM_HERE, writer.release());
-  else
-    writer->Run();
+  ChromeThread::PostTask(
+      ChromeThread::FILE, FROM_HERE, new Writer(codec.Encode(model), path));
 }
 
 }  // namespace bookmark_html_writer