Cleanup: Put more plugin code behind the ENABLE_PLUGINS ifdef.

Cleanup: Put more plugin code behind the ENABLE_PLUGINS ifdef.

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

[Lei Zhang, 2014-09-02 21:56:54]

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

[jam, 2014-09-03 17:37:06]

lgtm

https://codereview.chromium.org/525513003/diff/60001/content/renderer/render_...
File content/renderer/render_thread_impl.cc (right):

https://codereview.chromium.org/525513003/diff/60001/content/renderer/render_...
content/renderer/render_thread_impl.cc:1337: #if defined(ENABLE_PLUGINS)
nit: put the ifdefs at the end of this block, i.e. by the android/mac ones

[Lei Zhang, 2014-09-04 00:30:24]

https://codereview.chromium.org/525513003/diff/60001/content/renderer/render_...
File content/renderer/render_thread_impl.cc (right):

https://codereview.chromium.org/525513003/diff/60001/content/renderer/render_...
content/renderer/render_thread_impl.cc:1337: #if defined(ENABLE_PLUGINS)
On 2014/09/03 17:37:05, jam wrote:
> nit: put the ifdefs at the end of this block, i.e. by the android/mac ones

Done.

[Lei Zhang, 2014-09-04 00:33:10]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2014-09-04 00:35:42]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/thestig@chromium.org/525513003/100001

[commit-bot: I haz the power, 2014-09-04 01:35:00]

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

[commit-bot: I haz the power, 2014-09-04 01:35:01]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Lei Zhang, 2014-09-04 02:19:36]

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

[Lei Zhang, 2014-09-04 02:19:37]

+palmer for IPC

[palmer, 2014-09-04 18:44:03]

Rubberstamp LGTM

[Lei Zhang, 2014-09-04 19:51:53]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2014-09-04 19:57:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/thestig@chromium.org/525513003/100001

[commit-bot: I haz the power, 2014-09-04 21:19:21]

Committed patchset #6 (id:100001) as c5ee8caf7e3a78c5e6dc95b36d695eea65b7dc8e

[commit-bot: I haz the power, 2014-09-10 03:32:44]

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