Simplify how we get the plugin list in the webkit platform support by removing the intermediate met…

Simplify how we get the plugin list in the webkit platform support by removing the intermediate method. This removes knowledge of WebPluginInfo from webkit/glue, in prepration of moving that struct to content/public/common.

BUG=237249
R=jamesr@chromium.org

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

[jamesr, 2013-07-20 00:26:01]

lgtm

https://codereview.chromium.org/19871003/diff/1/content/renderer/renderer_web...
File content/renderer/renderer_webkitplatformsupport_impl.cc (right):

https://codereview.chromium.org/19871003/diff/1/content/renderer/renderer_web...
content/renderer/renderer_webkitplatformsupport_impl.cc:931:
UTF8ToUTF16(mime_type.file_extensions[k]));
WebString::fromUTF8() here is better since it won't generate a 16 bit string
unless there are non-latin1 characters in the string.  UTF8ToUTF16 will inflate
to a wide string no matter what.  In this instance the size/perf benefits are
minimal I'm sure, but people may copy-paste the bad pattern into other code.

[jam, 2013-07-20 00:30:39]

https://codereview.chromium.org/19871003/diff/1/content/renderer/renderer_web...
File content/renderer/renderer_webkitplatformsupport_impl.cc (right):

https://codereview.chromium.org/19871003/diff/1/content/renderer/renderer_web...
content/renderer/renderer_webkitplatformsupport_impl.cc:931:
UTF8ToUTF16(mime_type.file_extensions[k]));
On 2013/07/20 00:26:01, jamesr wrote:
> WebString::fromUTF8() here is better since it won't generate a 16 bit string
> unless there are non-latin1 characters in the string.  UTF8ToUTF16 will
inflate
> to a wide string no matter what.  In this instance the size/perf benefits are
> minimal I'm sure, but people may copy-paste the bad pattern into other code.

Done.