Simplify threading in browser thread by making only ChromeThread deal with di...

chrome/browser/spellchecker.cc

 // Copyright (c) 2006-2009 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/spellchecker.h"
 
 #include "app/l10n_util.h"
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/file_util.h"
@@ skipping 182 matching lines @@
     // Try saving it to |fallback_file_name_|, which almost surely has
     // write permission. If even this fails, there is nothing to be done.
     FilePath fallback_dir = fallback_file_name_.DirName();
     // Create the directory if it does not exist.
     if (!file_util::PathExists(fallback_dir))
       file_util::CreateDirectory(fallback_dir);
     SaveBufferToFile(data_, fallback_file_name_);
   }  // Unsuccessful save is taken care of in SpellChecker::Initialize().
 
   // Set Flag that dictionary is not downloading anymore.
-  MessageLoop* ui_loop = ChromeThread::GetMessageLoop(ChromeThread::UI);
-  ui_loop->PostTask(FROM_HERE,
+  ChromeThread::PostTask(
+      ChromeThread::UI, FROM_HERE,
       new UIProxyForIOTask(on_dictionary_save_complete_callback_task_, NULL));
 }
 
 // Design: this task tries to read the dictionary from disk and load it into
 // memory. It is executed on the file thread, and posts the results back to
 // the IO thread (via the UI thread---see UIProxyForIOTask).
 // The task first checks for the existence of the dictionary in one of the two
 // given locations. If it does not exist, the task informs the SpellChecker,
 // which will try to download the directory and run a new ReadDictionaryTask.
 class ReadDictionaryTask : public Task {
@@ skipping 46 matching lines @@
     }
 
     Finish(true);
   }
 
  private:
   void Finish(bool file_existed) {
     Task* task = NewRunnableMethod(spellchecker_, &SpellChecker::HunspellInited,
                                    hunspell_, bdict_file_, file_existed);
     if (spellchecker_->file_loop_) {
-      MessageLoop* ui_loop = ChromeThread::GetMessageLoop(ChromeThread::UI);
       // We were called on the file loop. Post back to the IO loop.
       // If this never gets posted to the IO loop, then we will leak |hunspell_|
       // and |bdict_file_|. But that can only happen during shutdown, so it's
       // not worth caring about.
-      ui_loop->PostTask(FROM_HERE, new UIProxyForIOTask(task, spellchecker_));
+      ChromeThread::PostTask(
+          ChromeThread::UI, FROM_HERE,
+          new UIProxyForIOTask(task, spellchecker_));
     } else {
       // We were called directly (e.g., during testing). Run the task directly.
       task->Run();
       delete task;
     }
   }
 
   // The SpellChecker we are working for. We are guaranteed to be outlived
   // by this object because it AddRefs() itself before calling us.
   // Accessing it is not necessarily thread safe, but are careful to only access
@@ skipping 608 matching lines @@
 
   // Populate the vector of WideStrings.
   for (int i = 0; i < number_of_suggestions; i++) {
     if (i < kMaxSuggestions)
       optional_suggestions->push_back(UTF8ToUTF16(suggestions[i]));
     free(suggestions[i]);
   }
   if (suggestions != NULL)
     free(suggestions);
 }