Changing PluginPrefs to use PluginFinder's async interface.

chrome/browser/chrome_plugin_browsertest.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 "base/command_line.h"
 #include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/process_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/plugin_prefs.h"
@@ skipping 66 matching lines @@
     return plugins;
   }
 
   static void EnableFlash(bool enable, Profile* profile) {
     FilePath flash_path = GetFlashPath();
     ASSERT_FALSE(flash_path.empty());
 
     PluginPrefs* plugin_prefs = PluginPrefs::GetForProfile(profile);
     scoped_refptr<content::MessageLoopRunner> runner =
         new content::MessageLoopRunner;
-    plugin_prefs->EnablePlugin(enable, flash_path, runner->QuitClosure());
+    plugin_prefs->EnablePlugin(enable, flash_path,
+                               base::Bind(&AssertPluginEnabled, runner));
     runner->Run();
   }
 
   static void EnsureFlashProcessCount(int expected) {
     int actual = 0;
     scoped_refptr<content::MessageLoopRunner> runner =
         new content::MessageLoopRunner;
     BrowserThread::PostTask(
         BrowserThread::IO,
         FROM_HERE,
@@ skipping 27 matching lines @@
 
   static void CountPluginProcesses(int* count, const base::Closure& quit_task) {
     for (content::BrowserChildProcessHostIterator iter; !iter.Done(); ++iter) {
       if (iter.GetData().type == content::PROCESS_TYPE_PLUGIN ||
           iter.GetData().type == content::PROCESS_TYPE_PPAPI_PLUGIN) {
         (*count)++;
       }
     }
     BrowserThread::PostTask(BrowserThread::UI, FROM_HERE, quit_task);
   }
+
+  static void AssertPluginEnabled(
+      scoped_refptr<content::MessageLoopRunner> runner,
+      bool did_enable) {
+    ASSERT_TRUE(did_enable);
+    runner->Quit();
+  }
 };
 
 // Tests a bunch of basic scenarios with Flash.
 IN_PROC_BROWSER_TEST_F(ChromePluginTest, Flash) {
   // Official builds always have bundled Flash.
 #if !defined(OFFICIAL_BUILD)
   if (GetFlashPath().empty()) {
     LOG(INFO) << "Test not running because couldn't find Flash.";
     return;
   }
@@ skipping 52 matching lines @@
   std::vector<webkit::WebPluginInfo> plugins = GetPlugins();
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(expected); ++i) {
     size_t j = 0;
     for (; j < plugins.size(); ++j) {
       if (plugins[j].name == ASCIIToUTF16(expected[i]))
         break;
     }
     ASSERT_TRUE(j != plugins.size()) << "Didn't find " << expected[i];
   }
 }