Reland 138502 - Move version metadata from PluginGroup into PluginInstaller.

Reland 138502 - Move version metadata from PluginGroup into PluginInstaller.

This also moves the plug-in information JSON files into their own subdirectory, so they can be changed without requiring OWNERS reviews for chrome/browser/resources (which is mostly WebUI otherwise).

BUG=124396
TEST=PluginInstallerTest.SecurityStatus:PluginFinderTest.JsonSyntax
TBR=jam@chromium.org,jhawkins@chromium.org
Original review URL: https://chromiumcodereview.appspot.com/10263022


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

[Bernhard Bauer, 2012-05-24 09:16:36]

Please review.

The old version is in patch set 1, for comparison.

win_aura compile failure looks unrelated :-/

[jochen (gone - plz use gerrit), 2012-05-24 09:19:49]

lgtm

On Thu, May 24, 2012 at 11:16 AM, <bauerb@chromium.org> wrote:

> Reviewers: jochen,
>
> Message:
> Please review.
>
> The old version is in patch set 1, for comparison.
>
> win_aura compile failure looks unrelated :-/
>
> Description:
> Reland 138502 - Move version metadata from PluginGroup into
> PluginInstaller.
>
> This also moves the plug-in information JSON files into their own
> subdirectory,
> so they can be changed without requiring OWNERS reviews for
> chrome/browser/resources (which is mostly WebUI otherwise).
>
> BUG=124396
> TEST=PluginInstallerTest.**SecurityStatus:**PluginFinderTest.JsonSyntax
> TBR=jam@chromium.org,jhawkins@**chromium.org <jhawkins@chromium.org>
> Original review URL:
https://chromiumcodereview.**appspot.com/10263022<https://chromiumcodereview....
>
>
> Please review this at
http://codereview.chromium.**org/10411098/<http://codereview.chromium.org/104...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>  M chrome/browser/browser_**resources.grd
>  M chrome/browser/extensions/api/**content_settings/content_**
> settings_apitest.cc
>  M chrome/browser/plugin_finder.**cc
>  M chrome/browser/plugin_finder_**unittest.cc
>  M chrome/browser/plugin_infobar_**delegates.cc
>  M chrome/browser/plugin_**installer.h
>  M chrome/browser/plugin_**installer.cc
>  A chrome/browser/plugin_**installer_unittest.cc
>  M chrome/browser/plugin_**observer.cc
>  M chrome/browser/renderer_host/**plugin_info_message_filter.cc
>  A chrome/browser/resources/**plugin_metadata/OWNERS
>  A chrome/browser/resources/**plugin_metadata/plugins_linux.**json
>  A + chrome/browser/resources/**plugin_metadata/plugins_mac.**json
>  A + chrome/browser/resources/**plugin_metadata/plugins_win.**json
>  D chrome/browser/resources/**plugins_linux.json
>  D chrome/browser/resources/**plugins_mac.json
>  D chrome/browser/resources/**plugins_win.json
>  M chrome/browser/ui/pdf/pdf_**unsupported_feature.cc
>  M chrome/browser/ui/webui/**plugins_ui.cc
>  M chrome/chrome_tests.gypi
>  M webkit/plugins/npapi/plugin_**group.h
>  M webkit/plugins/npapi/plugin_**group.cc
>  M webkit/plugins/npapi/plugin_**group_unittest.cc
>  M webkit/plugins/npapi/plugin_**list.cc
>  M webkit/plugins/npapi/plugin_**list_unittest.cc
>
>
>

[commit-bot: I haz the power, 2012-05-24 09:22:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bauerb@chromium.org/10411098/4002

[commit-bot: I haz the power, 2012-05-24 11:19:43]

Change committed as 138779