NaCl cleanup: Remove not-very-useful debug logging in the plugin code

NaCl cleanup: Remove not-very-useful debug logging in the plugin code

This removes all the PLUGIN_PRINTF/MODULE_PRINTF calls from
components/nacl/renderer/plugin/.

There aren't very many of these logging calls left now.  They don't
look useful to have in release binaries, and it's against Chromium
style to keep this sort of logging in the code.  Anyone doing
debugging can manually add their own logging.

My main motivation here is to remove a non-standard logging mechanism
that depends on some native_client/ headers.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=2832
TEST=e.g. NaClBrowserTestPnacl.PPAPICore (tests PNaCl translation)

Committed: https://crrev.com/58c568b9327125473321742c277a26742d3eebd2
Cr-Commit-Position: refs/heads/master@{#371865}

[Mark Seaborn, 2016-01-26 20:43:56]

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

[commit-bot: I haz the power, 2016-01-26 20:44:03]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1631413002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1631413002/1

[commit-bot: I haz the power, 2016-01-26 21:23:30]

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

[commit-bot: I haz the power, 2016-01-26 21:23:31]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Mark Seaborn, 2016-01-27 18:25:38]

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

[commit-bot: I haz the power, 2016-01-27 18:27:25]

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

[commit-bot: I haz the power, 2016-01-27 19:11:49]

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

[commit-bot: I haz the power, 2016-01-27 19:11:50]

Dry run: This issue passed the CQ dry run.

[Mark Seaborn, 2016-01-27 20:13:32]

Description was changed from

==========
NaCl cleanup: Remove debug logging in components/nacl/renderer/plugin/
==========

to

==========
NaCl cleanup: Remove not-very-useful debug logging in the plugin code

This removes all the PLUGIN_PRINTF/MODULE_PRINTF calls from
components/nacl/renderer/plugin/.

There aren't very many of these logging calls left now.  They don't
look useful to have in release binaries, and it's against Chromium
style to keep this sort of logging in the code.  Anyone doing
debugging can manually add their own logging.

My main motivation here is to remove a non-standard logging mechanism
that depends on some native_client/ headers.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=2832
TEST=e.g. NaClBrowserTestPnacl.PPAPICore (tests PNaCl translation)
==========

[Mark Seaborn, 2016-01-27 20:13:32]

mseaborn@chromium.org changed reviewers:
+ bbudge@chromium.org

[bbudge, 2016-01-27 20:27:51]

lgtm

[Mark Seaborn, 2016-01-27 20:33:21]

The CQ bit was checked by mseaborn@chromium.org

[commit-bot: I haz the power, 2016-01-27 20:37:54]

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

[commit-bot: I haz the power, 2016-01-27 20:47:41]

Description was changed from

==========
NaCl cleanup: Remove not-very-useful debug logging in the plugin code

This removes all the PLUGIN_PRINTF/MODULE_PRINTF calls from
components/nacl/renderer/plugin/.

There aren't very many of these logging calls left now.  They don't
look useful to have in release binaries, and it's against Chromium
style to keep this sort of logging in the code.  Anyone doing
debugging can manually add their own logging.

My main motivation here is to remove a non-standard logging mechanism
that depends on some native_client/ headers.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=2832
TEST=e.g. NaClBrowserTestPnacl.PPAPICore (tests PNaCl translation)
==========

to

==========
NaCl cleanup: Remove not-very-useful debug logging in the plugin code

This removes all the PLUGIN_PRINTF/MODULE_PRINTF calls from
components/nacl/renderer/plugin/.

There aren't very many of these logging calls left now.  They don't
look useful to have in release binaries, and it's against Chromium
style to keep this sort of logging in the code.  Anyone doing
debugging can manually add their own logging.

My main motivation here is to remove a non-standard logging mechanism
that depends on some native_client/ headers.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=2832
TEST=e.g. NaClBrowserTestPnacl.PPAPICore (tests PNaCl translation)
==========

[commit-bot: I haz the power, 2016-01-27 20:47:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-27 20:49:45]

Description was changed from

==========
NaCl cleanup: Remove not-very-useful debug logging in the plugin code

This removes all the PLUGIN_PRINTF/MODULE_PRINTF calls from
components/nacl/renderer/plugin/.

There aren't very many of these logging calls left now.  They don't
look useful to have in release binaries, and it's against Chromium
style to keep this sort of logging in the code.  Anyone doing
debugging can manually add their own logging.

My main motivation here is to remove a non-standard logging mechanism
that depends on some native_client/ headers.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=2832
TEST=e.g. NaClBrowserTestPnacl.PPAPICore (tests PNaCl translation)
==========

to

==========
NaCl cleanup: Remove not-very-useful debug logging in the plugin code

This removes all the PLUGIN_PRINTF/MODULE_PRINTF calls from
components/nacl/renderer/plugin/.

There aren't very many of these logging calls left now.  They don't
look useful to have in release binaries, and it's against Chromium
style to keep this sort of logging in the code.  Anyone doing
debugging can manually add their own logging.

My main motivation here is to remove a non-standard logging mechanism
that depends on some native_client/ headers.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=2832
TEST=e.g. NaClBrowserTestPnacl.PPAPICore (tests PNaCl translation)

Committed: https://crrev.com/58c568b9327125473321742c277a26742d3eebd2
Cr-Commit-Position: refs/heads/master@{#371865}
==========

[commit-bot: I haz the power, 2016-01-27 20:49:46]

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