Belated code review regarding pexe content handler.

Belated code review regarding pexe content handler.

Adds logging statement for translation process, and fixes
incorrect comment regarding parent class of NaCl content handlers.

See https://codereview.chromium.org/1382713002/ for original CL.

BUG=#396
R=jamesr@chromium.org

Committed: https://chromium.googlesource.com/external/mojo/+/5cdf1e75070e2bcb6f1d121cbc04628e658a7084

[Sean Klein, 2015-11-03 14:31:28]

smklein@chromium.org changed reviewers:
+ jamesr@chromium.org

[jamesr, 2015-11-03 18:34:21]

Please put something more descriptive in the patch description (i.e. say what
this patch does).  The spammy logs should be temporary, add a FIXME(smklein) or
TODO(smklein) to remind yourself to remove them when needed.

[Sean Klein, 2015-11-03 18:41:37]

Description was changed from

==========
Belated code review regarding pexe content handler.

See https://codereview.chromium.org/1382713002/ for original CL.

BUG=https://github.com/domokit/mojo/issues/396
==========

to

==========
Belated code review regarding pexe content handler.

Adds logging statement for translation process, and fixes incorrect comment
regarding parent class of NaCl content handlers.

See https://codereview.chromium.org/1382713002/ for original CL.

BUG=https://github.com/domokit/mojo/issues/396
==========

[Sean Klein, 2015-11-03 18:42:03]

Description was changed from

==========
Belated code review regarding pexe content handler.

Adds logging statement for translation process, and fixes incorrect comment
regarding parent class of NaCl content handlers.

See https://codereview.chromium.org/1382713002/ for original CL.

BUG=https://github.com/domokit/mojo/issues/396
==========

to

==========
Belated code review regarding pexe content handler.

Adds logging statement for translation process, and fixes
incorrect comment regarding parent class of NaCl content handlers.

See https://codereview.chromium.org/1382713002/ for original CL.

BUG=https://github.com/domokit/mojo/issues/396
==========

[Sean Klein, 2015-11-03 18:42:09]

On 2015/11/03 18:34:21, jamesr wrote:
> Please put something more descriptive in the patch description (i.e. say what
> this patch does).  The spammy logs should be temporary, add a FIXME(smklein)
or
> TODO(smklein) to remind yourself to remove them when needed.

Done.

[jamesr, 2015-11-03 18:46:22]

lgtm

Instead of BUG=https://../396 say just #396

[Sean Klein, 2015-11-03 18:46:49]

Description was changed from

==========
Belated code review regarding pexe content handler.

Adds logging statement for translation process, and fixes
incorrect comment regarding parent class of NaCl content handlers.

See https://codereview.chromium.org/1382713002/ for original CL.

BUG=https://github.com/domokit/mojo/issues/396
==========

to

==========
Belated code review regarding pexe content handler.

Adds logging statement for translation process, and fixes
incorrect comment regarding parent class of NaCl content handlers.

See https://codereview.chromium.org/1382713002/ for original CL.

BUG=#396
==========

[Sean Klein, 2015-11-03 18:48:27]

Committed patchset #2 (id:20001) manually as
5cdf1e75070e2bcb6f1d121cbc04628e658a7084.