Remove debug code from Chromoting's IQ pretty printer

Remove debug code from Chromoting's IQ pretty printer

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114155

[Jamie, 2011-12-05 20:51:49]

I thought the purpose of this message was to draw our attention to the fact that
the pretty-printer needs updating. Without this message the only symptom will be
an un-prettified stanza.

[garykac, 2011-12-05 21:07:24]

On 2011/12/05 20:51:49, Jamie wrote:
> I thought the purpose of this message was to draw our attention to the fact
that
> the pretty-printer needs updating. Without this message the only symptom will
be
> an un-prettified stanza.

The un-pretty stanza *is* the signal that the pretty-printer needs updating.

There are many reasons why the stanza won't be pretty-printed - they all fall
back to showing the un-pretty version so that we don't accidentally hide debug
info.

[garykac, 2011-12-05 21:09:19]

On 2011/12/05 21:07:24, garykac wrote:
> On 2011/12/05 20:51:49, Jamie wrote:
> > I thought the purpose of this message was to draw our attention to the fact
> that
> > the pretty-printer needs updating. Without this message the only symptom
will
> be
> > an un-prettified stanza.
> 
> The un-pretty stanza *is* the signal that the pretty-printer needs updating.
> 
> There are many reasons why the stanza won't be pretty-printed - they all fall
> back to showing the un-pretty version so that we don't accidentally hide debug
> info.

In any case, if we wanted an extra signal when we show un-pretty stanzas (and I
don't think that we do), the proper place would be in the logIq() routine, right
before we show the unpretty version.

[garykac, 2011-12-13 00:11:12]

ping!

On Mon, Dec 5, 2011 at 1:09 PM, <garykac@chromium.org> wrote:

> On 2011/12/05 21:07:24, garykac wrote:
>
>> On 2011/12/05 20:51:49, Jamie wrote:
>> > I thought the purpose of this message was to draw our attention to the
>> fact
>> that
>> > the pretty-printer needs updating. Without this message the only symptom
>>
> will
>
>> be
>> > an un-prettified stanza.
>>
>
>  The un-pretty stanza *is* the signal that the pretty-printer needs
>> updating.
>>
>
>  There are many reasons why the stanza won't be pretty-printed - they all
>> fall
>> back to showing the un-pretty version so that we don't accidentally hide
>> debug
>> info.
>>
>
> In any case, if we wanted an extra signal when we show un-pretty stanzas
> (and I
> don't think that we do), the proper place would be in the logIq() routine,
> right
> before we show the unpretty version.
>
>
http://codereview.chromium.**org/8802020/<http://codereview.chromium.org/8802...
>

[Jamie, 2011-12-13 00:15:50]

lgtm

[commit-bot: I haz the power, 2011-12-13 01:09:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/garykac@chromium.org/8802020/1

[commit-bot: I haz the power, 2011-12-13 02:31:12]

Change committed as 114155