pdf: Move the string/image resources for the pdf plugin.

pdf: Move the string/image resources for the pdf plugin.

Move the resources for the pdf plugin out of //content into //components. Also,
reuse the same code for getting the resource from PepperPDFHost and
PPB_PDF_Impl.

BUG=401242
R=blundell@chromium.org, jam@chromium.org, raymes@chromium.org

Committed: https://chromium.googlesource.com/chromium/src/+/8a0003bc269cfaabb44ce2db04046eb7a26bd78e

[sadrul, 2014-08-26 21:48:35]

sadrul@chromium.org changed reviewers:
+ jam@chromium.org, raymes@chromium.org, thestig@chromium.org

[sadrul, 2014-08-26 21:48:35]

This goes on top of https://codereview.chromium.org/505243003/ and
https://codereview.chromium.org/477263003/

[Lei Zhang, 2014-08-26 22:05:53]

This generally looks good.

https://codereview.chromium.org/506473003/diff/1/components/pdf/renderer/pdf_...
File components/pdf/renderer/pdf_resource_util.cc (right):

https://codereview.chromium.org/506473003/diff/1/components/pdf/renderer/pdf_...
components/pdf/renderer/pdf_resource_util.cc:20: static const ResourceImageInfo
kResourceImageMap[] = {
s/static //

https://codereview.chromium.org/506473003/diff/1/components/pdf/renderer/pdf_...
components/pdf/renderer/pdf_resource_util.cc:88: return resource_id ?
ResourceBundle::GetSharedInstance().GetImageSkiaNamed(
nit: formatting here is a wee bit ugly IMO.

[raymes, 2014-08-27 03:15:42]

lgtm

https://codereview.chromium.org/506473003/diff/1/components/pdf/renderer/pdf_...
File components/pdf/renderer/pdf_resource_util.cc (right):

https://codereview.chromium.org/506473003/diff/1/components/pdf/renderer/pdf_...
components/pdf/renderer/pdf_resource_util.cc:110: return
base::UTF16ToUTF8(l10n_util::GetStringUTF16(resource_id));
What happens in the case where resource_id == 0?

[jam, 2014-08-27 03:35:04]

rubberstamp lgtm

[sadrul, 2014-08-28 19:51:43]

sadrul@chromium.org changed reviewers:
+ blundell@chromium.org

[sadrul, 2014-08-28 19:51:43]

blundell@chromium.org for changes in the grdp/grd files, and
component/resources/ owners.

[blundell, 2014-09-01 11:37:32]

LGTM

I'm going to send you an email with the steps you need to take to update the
translation processfor pdf_strings.grdp.

[sadrul, 2014-09-02 04:18:21]

Committed patchset #3 (id:40001) manually as 8a0003b (presubmit successful).

[commit-bot: I haz the power, 2014-09-10 03:17:40]

Patchset 3 (id:??) landed as
https://crrev.com/8a0003bc269cfaabb44ce2db04046eb7a26bd78e
Cr-Commit-Position: refs/heads/master@{#292897}