Allow the same version plugin to be loaded multiple times on Windows.

Allow the same version plugin to be loaded multiple times on Windows.

Resolves the issue of using a plugin installed through an extension in
more than one profile.

BUG=132894
TEST=Install an extension with plugin in two profiles and observe it works (e.g. Legacy Browser Support)

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207869

[pastarmovj, 2013-06-21 09:28:38]

Please review this CL. I verified that it indeed fixes the issue for my
extension. 

Thanks!

[Bernhard Bauer, 2013-06-21 09:34:12]

LGTM with a nit:

https://codereview.chromium.org/17543004/diff/1/webkit/plugins/npapi/plugin_l...
File webkit/plugins/npapi/plugin_list_win.cc (right):

https://codereview.chromium.org/17543004/diff/1/webkit/plugins/npapi/plugin_l...
webkit/plugins/npapi/plugin_list_win.cc:394: return false;  // We have already
loaded a plugin with newer version.
Tried to get this into 80 columns, didn'tcha? It's "with a newer version" though
:)

[pastarmovj, 2013-06-21 11:20:20]

https://codereview.chromium.org/17543004/diff/1/webkit/plugins/npapi/plugin_l...
File webkit/plugins/npapi/plugin_list_win.cc (right):

https://codereview.chromium.org/17543004/diff/1/webkit/plugins/npapi/plugin_l...
webkit/plugins/npapi/plugin_list_win.cc:394: return false;  // We have already
loaded a plugin with newer version.
On 2013/06/21 09:34:12, Bernhard Bauer wrote:
> Tried to get this into 80 columns, didn'tcha? It's "with a newer version"
though
> :)

Darn! Reverted that change :)

[commit-bot: I haz the power, 2013-06-21 11:25:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pastarmovj@chromium.org/17543004/6001

[commit-bot: I haz the power, 2013-06-21 17:04:13]

Change committed as 207869