Always close non-null output stream.

Always close non-null output stream.

BUG=595449
Committed: https://crrev.com/17d298aac4611c2838f56c0bce992575eba5c631
Cr-Commit-Position: refs/heads/master@{#384968}

[Peter Wen, 2016-04-04 17:46:57]

wnwen@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[Peter Wen, 2016-04-04 17:46:58]

Debated between surrounding the entire method with the try/catch vs adding two
additional close() calls when returning.

WDYT? 🍍

[gone, 2016-04-04 17:52:23]

I think the correct thing to do is correct the one place where
TabState#saveState() is called and the stream that's passed in isn't closed. 
TabPersister and WebappActivity both properly close the stream, but
TabPersistentStore doesn't.

[Peter Wen, 2016-04-04 17:59:34]

Opps, totally missed that option. Thanks, PTAL. :)

[gone, 2016-04-04 18:00:13]

lgtm, thanks!

[Peter Wen, 2016-04-04 18:00:51]

The CQ bit was checked by wnwen@chromium.org

[commit-bot: I haz the power, 2016-04-04 18:01:08]

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

[commit-bot: I haz the power, 2016-04-04 19:02:56]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-04 19:03:56]

Description was changed from

==========
Always close non-null output stream.

BUG=595449
==========

to

==========
Always close non-null output stream.

BUG=595449

Committed: https://crrev.com/17d298aac4611c2838f56c0bce992575eba5c631
Cr-Commit-Position: refs/heads/master@{#384968}
==========

[commit-bot: I haz the power, 2016-04-04 19:03:57]

Patchset 3 (id:??) landed as
https://crrev.com/17d298aac4611c2838f56c0bce992575eba5c631
Cr-Commit-Position: refs/heads/master@{#384968}