PluginList cleanup

PluginList cleanup to fix a race condition and decrease the API surface for future refactorings: 

* Remove "webkit/glue/plugins/plugin_list.h" includes in favor of "webkit/npapi/plugins/plugin_list.h"
* Remove |refresh| parameter from |GetPlugins| in favor of calling |RefreshPlugins| beforehand.
* Remove |GetPluginInfo| in favor of calling |GetPluginInfoArray| and looking for the first enabled plug-in.
* Remove |GetEnabledPlugins| in favor of calling |GetPlugins| and filtering out disabled plugins.
* Remove |stale| in favor of an outparameter to |GetPluginInfoArray|, to remove the race condition.

BUG=69516, 80794
TEST=none

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

[Bernhard Bauer, 2011-07-28 13:23:13]

Please review.

[jam, 2011-07-28 20:34:08]

lgtm with one caveat:

It's a bit awkward for a lot of code that just wants the first enabled plugin to
now get the array and look for it. Since you'll need a function in chrome that
does this later and uses profile, why not add it now?

[Bernhard Bauer, 2011-07-28 22:53:48]

On Thu, Jul 28, 2011 at 22:34,  <[email protected]> wrote:
> lgtm with one caveat:
>
> It's a bit awkward for a lot of code that just wants the first enabled
> plugin to
> now get the array and look for it. Since you'll need a function in chrome
> that
> does this later and uses profile, why not add it now?

To keep this CL manageable? :-) I already have two other CLs pipelined
which do that (http://codereview.chromium.org/7387010/ and
http://codereview.chromium.org/7477044/), and tried to focus on the
PluginList changes in this one.

> http://codereview.chromium.org/7497030/
>