Remove unnecessary loop and quit in CronetChunkedOutputStream

Remove unnecessary loop and quit in CronetChunkedOutputStream

The MessageLoop.loop() in CronetChunkedOutputStream.close()
is potentially problematic. If the user closes the
OutputStream unintentionally when MessageLoop is not going
to be quit, it might wait forever. This removes the
corresponding unnecessary quit().

BUG=626653
TBR=pauljensen@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/1657103002
Cr-Commit-Position: refs/heads/master@{#404391}
(cherry picked from commit 578950706701e926c468cc1ab8128e94bd032e08)

[xunjieli, 2016-07-11 17:07:46]

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

[commit-bot: I haz the power, 2016-07-11 17:07:59]

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

[commit-bot: I haz the power, 2016-07-11 17:08:00]

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

[commit-bot: I haz the power, 2016-07-11 17:08:01]

Dry run: CLs for remote refs other than refs/pending/heads/master must contain
NOTRY=true and NOPRESUBMIT=true in order for the CQ to process them

[xunjieli, 2016-07-11 17:20:46]

Paul, I am TBR-ing you. This is to merge the change into M53. I verified locally
that Cronet tests are still passing.

[xunjieli, 2016-07-11 17:26:37]

The CQ bit was checked by xunjieli@chromium.org

[commit-bot: I haz the power, 2016-07-11 17:26:54]

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

[commit-bot: I haz the power, 2016-07-11 17:26:56]

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

[commit-bot: I haz the power, 2016-07-11 17:26:57]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[xunjieli, 2016-07-11 17:30:30]

Description was changed from

==========
Remove unnecessary loop and quit in CronetChunkedOutputStream

The MessageLoop.loop() in CronetChunkedOutputStream.close()
is potentially problematic. If the user closes the
OutputStream unintentionally when MessageLoop is not going
to be quit, it might wait forever. This removes the
corresponding unnecessary quit().

BUG=626653
TBR=pauljensen@chromium.org

Review-Url: https://codereview.chromium.org/1657103002
Cr-Commit-Position: refs/heads/master@{#404391}
(cherry picked from commit 578950706701e926c468cc1ab8128e94bd032e08)
==========

to

==========
Remove unnecessary loop and quit in CronetChunkedOutputStream

The MessageLoop.loop() in CronetChunkedOutputStream.close()
is potentially problematic. If the user closes the
OutputStream unintentionally when MessageLoop is not going
to be quit, it might wait forever. This removes the
corresponding unnecessary quit().

BUG=626653
TBR=pauljensen@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/1657103002
Cr-Commit-Position: refs/heads/master@{#404391}
(cherry picked from commit 578950706701e926c468cc1ab8128e94bd032e08)
==========

[xunjieli, 2016-07-11 17:30:33]

The CQ bit was checked by xunjieli@chromium.org

[commit-bot: I haz the power, 2016-07-11 17:31:48]

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

[commit-bot: I haz the power, 2016-07-11 17:33:57]

Description was changed from

==========
Remove unnecessary loop and quit in CronetChunkedOutputStream

The MessageLoop.loop() in CronetChunkedOutputStream.close()
is potentially problematic. If the user closes the
OutputStream unintentionally when MessageLoop is not going
to be quit, it might wait forever. This removes the
corresponding unnecessary quit().

BUG=626653
TBR=pauljensen@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/1657103002
Cr-Commit-Position: refs/heads/master@{#404391}
(cherry picked from commit 578950706701e926c468cc1ab8128e94bd032e08)
==========

to

==========
Remove unnecessary loop and quit in CronetChunkedOutputStream

The MessageLoop.loop() in CronetChunkedOutputStream.close()
is potentially problematic. If the user closes the
OutputStream unintentionally when MessageLoop is not going
to be quit, it might wait forever. This removes the
corresponding unnecessary quit().

BUG=626653
TBR=pauljensen@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/1657103002
Cr-Commit-Position: refs/heads/master@{#404391}
(cherry picked from commit 578950706701e926c468cc1ab8128e94bd032e08)
==========

[commit-bot: I haz the power, 2016-07-11 17:33:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-11 17:34:07]

CQ bit was unchecked.