Fix a bug that would cause the plugin_loader to stall.

content/browser/plugin_loader_posix_unittest.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 "content/browser/plugin_loader_posix.h"
 
 #include "base/at_exit.h"
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/memory/ref_counted.h"
@@ skipping 29 matching lines @@
   }
 
   std::vector<WebPluginInfo>* internal_plugins() {
     return &internal_plugins_;
   }
 
   void RealLoadPluginsInternal() {
     PluginLoaderPosix::LoadPluginsInternal();
   }
 
+  bool LaunchUtilityProcess() override {
+    // This method always does nothing and returns false. The actual
+    // implementation of this method launches another process, which is not
+    // very unit_test friendly.
+    return false;
+  }
+
   void TestOnPluginLoaded(uint32 index, const WebPluginInfo& plugin) {
     OnPluginLoaded(index, plugin);
   }
 
   void TestOnPluginLoadFailed(uint32 index, const base::FilePath& path) {
     OnPluginLoadFailed(index, path);
   }
 
  protected:
   virtual ~MockPluginLoaderPosix() {}
@@ skipping 327 matching lines @@
   plugin_loader()->OnProcessCrashed(42);
   plugin_loader()->OnProcessCrashed(42);
   plugin_loader()->OnProcessCrashed(42);
 
   message_loop()->RunUntilIdle();
   EXPECT_EQ(1, did_callback);
 
   EXPECT_EQ(0u, plugin_loader()->loaded_plugins().size());
 }
 
+TEST_F(PluginLoaderPosixTest, PluginLaunchFailed) {
+  int did_callback = 0;
+  PluginService::GetPluginsCallback callback =
+      base::Bind(&VerifyCallback, base::Unretained(&did_callback));
+
+  EXPECT_CALL(*plugin_loader(), LoadPluginsInternal())
+      .WillOnce(testing::Invoke(
+          plugin_loader(), &MockPluginLoaderPosix::RealLoadPluginsInternal));
+
+  plugin_loader()->GetPlugins(callback);
+  message_loop()->RunUntilIdle();
+  EXPECT_EQ(1, did_callback);
+  EXPECT_EQ(0u, plugin_loader()->loaded_plugins().size());
+
+  // TODO(erikchen): This is a genuine leak that should be fixed.
+  // https://code.google.com/p/chromium/issues/detail?id=431906
+  testing::Mock::AllowLeak(plugin_loader());
+}
+
 }  // namespace content