Changing PluginPrefs to use PluginFinder's async interface.

chrome/browser/plugin_prefs_unittest.cc

Index: chrome/browser/plugin_prefs_unittest.cc
diff --git a/chrome/browser/plugin_prefs_unittest.cc b/chrome/browser/plugin_prefs_unittest.cc
index f91daf3184f4c616ee472691ad2dba2edb974ce1..1f324cc10e242d991def3eda77c4870ba709d7a3 100644
--- a/chrome/browser/plugin_prefs_unittest.cc
+++ b/chrome/browser/plugin_prefs_unittest.cc
@@ -21,6 +21,14 @@
 using content::BrowserThread;
 using content::PluginService;
 
+namespace {
+
+void CanEnablePluginCallback(bool dummy) {
+  MessageLoop::current()->QuitWhenIdle();

[ibraaaa, 2012/08/29 15:35:50]

This is to maintain the exact same behavior, however, I would be happier if I
knew how calling MessageLoop::QuitClosure at this point is useful to the test?
:)

[Bernhard Bauer, 2012/08/29 16:53:21]

Yeah, something like base::IgnoreResult for parameters would be nice. However,
in this case you might want to check the result anyway?

[ibraaaa, 2012/08/29 17:37:53]

Sorry, I don't get what you mean?

On 2012/08/29 16:53:21, Bernhard Bauer wrote:

[Bernhard Bauer, 2012/08/29 18:10:17]

You ignore the result at the moment, but maybe you should ASSERT that it
succeeded (which would obviate the need for a callback that ignores its value).

[ibraaaa, 2012/08/29 18:28:19]

ok, I thought you were answering my first question "I would be happier if I
knew how calling MessageLoop::QuitClosure at this point is useful to the test?",
I mean originally the MessageLoop::QuitClosure was passed as a callback to
EnablePlugin, I can't see the point in this.

On 2012/08/29 18:10:17, Bernhard Bauer wrote:

[Bernhard Bauer, 2012/08/31 12:54:11]

We are running a message loop in line 179, so we need to quit it when
PluginPrefs is done doing its thing. If we don't do anything else (e.g. checking
the success value like we do now), then passing in a callback that just calls
MessageLoop::QuitWhenIdle instead of directly passing a QuitClosure would be
unnecessary.

+}
+
+}  // namespace
+
 class PluginPrefsTest : public ::testing::Test {
  public:
   virtual void SetUp() OVERRIDE {
@@ -164,8 +172,9 @@ TEST_F(PluginPrefsTest, DisableGlobally) {
                                ASCIIToUTF16("1.0.0"),
                                ASCIIToUTF16("Foo plug-in"));
   plugin_list.AddPluginToLoad(plugin);
-  PluginPrefs::EnablePluginGlobally(false, plugin.path,
-                                    MessageLoop::QuitClosure());
+  PluginPrefs::EnablePluginGlobally(
+      false, plugin.path,
+      base::Bind(&CanEnablePluginCallback));
 
   message_loop.Run();