[ON HOLD] Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

[ON HOLD]Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 
9KB read buffer. The DnsResponse is never used again, but SocketHandler 
can be kept around in MDnsConnection::|socket_handlers_| for a while.

This CL additionally renames DoLoop() to DoRead().

BUG=690095

[xunjieli, 2017-02-08 19:56:14]

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

[commit-bot: I haz the power, 2017-02-08 19:57:14]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[xunjieli, 2017-02-08 20:02:31]

Description was changed from

==========
Deallocate read buffers in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the read
buffer. The DnsResponse is never used again, but SocketHandler can be kept
around in MDnsConnection::|socket_handlers_| for a while.

BUG=690095
==========

to

==========
Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 9KB read
buffer. The DnsResponse is never used again, but SocketHandler can be kept
around in MDnsConnection::|socket_handlers_| for a while.

BUG=690095
==========

[xunjieli, 2017-02-08 20:02:40]

Description was changed from

==========
Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 9KB read
buffer. The DnsResponse is never used again, but SocketHandler can be kept
around in MDnsConnection::|socket_handlers_| for a while.

BUG=690095
==========

to

==========
Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 
9KB read buffer. The DnsResponse is never used again, but SocketHandler can be
kept around in MDnsConnection::|socket_handlers_| for a while.

BUG=690095
==========

[xunjieli, 2017-02-08 20:02:47]

Description was changed from

==========
Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 
9KB read buffer. The DnsResponse is never used again, but SocketHandler can be
kept around in MDnsConnection::|socket_handlers_| for a while.

BUG=690095
==========

to

==========
Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 
9KB read buffer. The DnsResponse is never used again, but SocketHandler 
can be kept around in MDnsConnection::|socket_handlers_| for a while.

BUG=690095
==========

[xunjieli, 2017-02-08 20:14:00]

Description was changed from

==========
Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 
9KB read buffer. The DnsResponse is never used again, but SocketHandler 
can be kept around in MDnsConnection::|socket_handlers_| for a while.

BUG=690095
==========

to

==========
Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 
9KB read buffer. The DnsResponse is never used again, but SocketHandler 
can be kept around in MDnsConnection::|socket_handlers_| for a while.

This CL additionally renames DoLoop() to DoRead().

BUG=690095
==========

[xunjieli, 2017-02-08 20:24:34]

xunjieli@chromium.org changed reviewers:
+ juliatuttle@chromium.org

[xunjieli, 2017-02-08 20:24:35]

Julia, PTAL. Thanks!

[commit-bot: I haz the power, 2017-02-08 21:13:38]

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

[commit-bot: I haz the power, 2017-02-08 21:13:39]

Dry run: This issue passed the CQ dry run.

[xunjieli, 2017-02-08 21:27:00]

On 2017/02/08 20:24:35, xunjieli wrote:
> Julia, PTAL. Thanks!

Hmm.. Sorry, this actually doesn't address the issue. Looks like the the socket
handlers are waiting for read result. They haven't done reading. I will think
more about it.

[xunjieli, 2017-02-08 21:31:22]

Description was changed from

==========
Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 
9KB read buffer. The DnsResponse is never used again, but SocketHandler 
can be kept around in MDnsConnection::|socket_handlers_| for a while.

This CL additionally renames DoLoop() to DoRead().

BUG=690095
==========

to

==========
[ON HOLD]Deallocate read buffer in MDnsClientImpl::SocketHandler if done reading

When SocketHandler is done reading, free its DnsResponse which owns the 
9KB read buffer. The DnsResponse is never used again, but SocketHandler 
can be kept around in MDnsConnection::|socket_handlers_| for a while.

This CL additionally renames DoLoop() to DoRead().

BUG=690095
==========