[Linux] Load plugins out-of-process.

[Linux] Load plugins out-of-process.

This also fixes a bug in PluginLoaderPosix where if the last N plugins in the
canonical list crash, the callbacks are never run.

BUG=17863
TEST=On Linux, plugins work as before.

Originally Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=106962
Reverted: http://src.chromium.org/viewvc/chrome?view=rev&revision=106977

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=108286

[Bernhard Bauer, 2011-10-24 14:57:08]

http://codereview.chromium.org/8372015/diff/1/content/browser/plugin_loader_p...
File content/browser/plugin_loader_posix.cc (right):

http://codereview.chromium.org/8372015/diff/1/content/browser/plugin_loader_p...
content/browser/plugin_loader_posix.cc:50: LoadPluginsInternal();
You could have the check for an empty canonical list in LoadPluginsInternal and
short-circuit to RunPendingCallbacks there. This would also catch the
(admittedly very esoteric) case that the canonical list of plug-ins starts out
empty.

[Robert Sesek, 2011-10-24 16:44:40]

http://codereview.chromium.org/8372015/diff/1/content/browser/plugin_loader_p...
File content/browser/plugin_loader_posix.cc (right):

http://codereview.chromium.org/8372015/diff/1/content/browser/plugin_loader_p...
content/browser/plugin_loader_posix.cc:50: LoadPluginsInternal();
On 2011/10/24 14:57:08, Bernhard Bauer wrote:
> You could have the check for an empty canonical list in LoadPluginsInternal
and
> short-circuit to RunPendingCallbacks there. This would also catch the
> (admittedly very esoteric) case that the canonical list of plug-ins starts out
> empty.

Done.

[Bernhard Bauer, 2011-10-24 16:58:04]

LGTM.

[jam, 2011-10-24 17:47:13]

lgtm

[commit-bot: I haz the power, 2011-10-24 17:56:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rsesek@chromium.org/8372015/1005

[commit-bot: I haz the power, 2011-10-24 20:01:50]

Change committed as 106962