Remove the unused parts of PPB_PDF

Remove the unused parts of PPB_PDF

These parts were only used by the in-process viewer.

BUG=303491
TBR=jam@chromium.org for components/resources/

Committed: https://crrev.com/ec40c91c3f90649a4e7a29a459839d1fb5a46e1a
Cr-Commit-Position: refs/heads/master@{#328695}

[raymes, 2015-05-06 06:21:26]

raymes@chromium.org changed reviewers:
+ sammc@chromium.org

[raymes, 2015-05-06 06:21:28]

Take your time reviewing this. 

There are a ton of files but most of them are just images.

[Sam McNally, 2015-05-06 07:16:45]

LGTM

https://codereview.chromium.org/1125113002/diff/20001/components/pdf/browser/...
File components/pdf/browser/pdf_web_contents_helper_client.h (left):

https://codereview.chromium.org/1125113002/diff/20001/components/pdf/browser/...
components/pdf/browser/pdf_web_contents_helper_client.h:20:
PasswordDialogClosedCallback;
Remove.

https://codereview.chromium.org/1125113002/diff/20001/components/pdf/renderer...
File components/pdf/renderer/pdf_resource_util.cc (left):

https://codereview.chromium.org/1125113002/diff/20001/components/pdf/renderer...
components/pdf/renderer/pdf_resource_util.cc:110: return
base::UTF16ToUTF8(l10n_util::GetStringUTF16(resource_id));
Can you change this to l10n_util::GetStringUTF8 while you're here?

[raymes, 2015-05-07 00:49:39]

raymes@chromium.org changed reviewers:
+ thestig@chromium.org

[raymes, 2015-05-07 00:49:40]

+thestig for chrome/ OWNERS

https://codereview.chromium.org/1125113002/diff/20001/components/pdf/browser/...
File components/pdf/browser/pdf_web_contents_helper_client.h (left):

https://codereview.chromium.org/1125113002/diff/20001/components/pdf/browser/...
components/pdf/browser/pdf_web_contents_helper_client.h:20:
PasswordDialogClosedCallback;
On 2015/05/06 07:16:45, Sam McNally wrote:
> Remove.

Done.

https://codereview.chromium.org/1125113002/diff/20001/components/pdf/renderer...
File components/pdf/renderer/pdf_resource_util.cc (left):

https://codereview.chromium.org/1125113002/diff/20001/components/pdf/renderer...
components/pdf/renderer/pdf_resource_util.cc:110: return
base::UTF16ToUTF8(l10n_util::GetStringUTF16(resource_id));
On 2015/05/06 07:16:45, Sam McNally wrote:
> Can you change this to l10n_util::GetStringUTF8 while you're here?

Done.

[Lei Zhang, 2015-05-07 01:06:40]

chrome/ lgtm

[raymes, 2015-05-07 01:40:17]

raymes@chromium.org changed reviewers:
+ jam@chromium.org

[raymes, 2015-05-07 01:40:18]

TBR=jam@chromium.org for components/resources/

[raymes, 2015-05-07 01:40:29]

The CQ bit was checked by raymes@chromium.org

[raymes, 2015-05-07 01:40:30]

The patchset sent to the CQ was uploaded after l-g-t-m from sammc@chromium.org
Link to the patchset: https://codereview.chromium.org/1125113002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2015-05-07 01:41:05]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1125113002/40001

[commit-bot: I haz the power, 2015-05-07 01:59:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-05-07 01:59:16]

Try jobs failed on following builders:
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[raymes, 2015-05-07 02:02:45]

The CQ bit was checked by raymes@chromium.org

[raymes, 2015-05-07 02:02:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
thestig@chromium.org, sammc@chromium.org
Link to the patchset: https://codereview.chromium.org/1125113002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2015-05-07 02:05:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1125113002/60001

[commit-bot: I haz the power, 2015-05-07 03:44:47]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-05-07 03:46:26]

Patchset 4 (id:??) landed as
https://crrev.com/ec40c91c3f90649a4e7a29a459839d1fb5a46e1a
Cr-Commit-Position: refs/heads/master@{#328695}