Remove the OOP PDF flag

Remove the OOP PDF flag

This removes the flag for enabling OOP PDF since it is enabled by default and
the in process viewer no longer works anyway. The old in-process browser tests
have been removed. They are mostly replaced by pdf_extension_test.cc and they
are disabled anyway.

BUG=483870, 436444
Committed: https://crrev.com/be67c3d1a41734b948b91cb0b7d57a0ecb6ef389
Cr-Commit-Position: refs/heads/master@{#328687}

[raymes, 2015-05-05 01:13:33]

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

[Sam McNally, 2015-05-05 01:22:31]

LGTM

https://codereview.chromium.org/1128453003/diff/1/chrome/browser/pdf/pdf_exte...
File chrome/browser/pdf/pdf_extension_test.cc (right):

https://codereview.chromium.org/1128453003/diff/1/chrome/browser/pdf/pdf_exte...
chrome/browser/pdf/pdf_extension_test.cc:28: void
SetUpCommandLine(base::CommandLine* command_line) override {
Remove.

[raymes, 2015-05-05 01:26:18]

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

[raymes, 2015-05-05 01:26:19]

+thestig@

https://codereview.chromium.org/1128453003/diff/1/chrome/browser/pdf/pdf_exte...
File chrome/browser/pdf/pdf_extension_test.cc (right):

https://codereview.chromium.org/1128453003/diff/1/chrome/browser/pdf/pdf_exte...
chrome/browser/pdf/pdf_extension_test.cc:28: void
SetUpCommandLine(base::CommandLine* command_line) override {
On 2015/05/05 01:22:31, Sam McNally wrote:
> Remove.

Done.

[Lei Zhang, 2015-05-05 01:36:24]

lgtm, remember to delete the pdf_private directory at some point.

[raymes, 2015-05-05 02:11:36]

That directory is now used by pdf_extension_test :)

[raymes, 2015-05-05 02:12:21]

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

[raymes, 2015-05-05 02:12:22]

+mnaganov for android_webview

[raymes, 2015-05-05 04:09:39]

The CQ bit was checked by raymes@chromium.org to run a CQ dry run

[raymes, 2015-05-05 04:09:40]

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/1128453003/#ps20001
(title: " ")

[commit-bot: I haz the power, 2015-05-05 04:10:03]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128453003/20001

[commit-bot: I haz the power, 2015-05-05 05:25:26]

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

[commit-bot: I haz the power, 2015-05-05 05:25:27]

Dry run: This issue passed the CQ dry run.

[raymes, 2015-05-05 23:41:29]

mnaganov: friendly ping in case you missed this :)

[mnaganov (inactive), 2015-05-06 15:55:07]

Apologies for the delay!

android_webview/ LGTM

[Lei Zhang, 2015-05-06 23:28:29]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2015-05-06 23:29:35]

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

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

Committed patchset #2 (id:20001)

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

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