chromeos: Avoid crash on thaw failure if freezing failed.

chromeos: Avoid crash on thaw failure if freezing failed.

Avoid an "Unable to thaw renderers." LOG(FATAL) in
RendererFreezer if freezing the renderers also failed
earlier. This is a workaround for unexplained EBADF and
ENOENT errors seen when writing to cgroups.

BUG=chromium:661310
TEST=manual: log in and chmod freezer.state in
     /sys/fs/cgroup/freezer/chrome_renderers/to_be_frozen
     to 444; suspend and resume and check that chrome
     doesn't abort

Committed: https://crrev.com/d66436c2b7a48ae82eda1af894fa79b81a5ffaad
Cr-Commit-Position: refs/heads/master@{#438625}

[Daniel Erat, 2016-12-14 18:51:04]

derat@chromium.org changed reviewers:
+ chirantan@chromium.org

[Daniel Erat, 2016-12-14 18:51:10]

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

[commit-bot: I haz the power, 2016-12-14 18:51:53]

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

[chirantan, 2016-12-14 18:55:20]

chirantan@google.com changed reviewers:
+ chirantan@google.com

[chirantan, 2016-12-14 18:55:20]

lgtm

Thanks!

[commit-bot: I haz the power, 2016-12-14 19:24:52]

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

[commit-bot: I haz the power, 2016-12-14 19:24:53]

Dry run: This issue passed the CQ dry run.

[Daniel Erat, 2016-12-14 21:09:39]

The CQ bit was checked by derat@chromium.org

[commit-bot: I haz the power, 2016-12-14 21:10:42]

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-12-14 21:19:12]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481749779130340, "parent_rev":
"ada57d859f13cec3d38a7221e1876c8da940c694", "commit_rev":
"78c102c8cd7f856290acc8b0dafdda73c5ef412b"}

[commit-bot: I haz the power, 2016-12-14 21:19:43]

Description was changed from

==========
chromeos: Avoid crash on thaw failure if freezing failed.

Avoid an "Unable to thaw renderers." LOG(FATAL) in
RendererFreezer if freezing the renderers also failed
earlier. This is a workaround for unexplained EBADF and
ENOENT errors seen when writing to cgroups.

BUG=chromium:661310
TEST=manual: log in and chmod freezer.state in
     /sys/fs/cgroup/freezer/chrome_renderers/to_be_frozen
     to 444; suspend and resume and check that chrome
     doesn't abort
==========

to

==========
chromeos: Avoid crash on thaw failure if freezing failed.

Avoid an "Unable to thaw renderers." LOG(FATAL) in
RendererFreezer if freezing the renderers also failed
earlier. This is a workaround for unexplained EBADF and
ENOENT errors seen when writing to cgroups.

BUG=chromium:661310
TEST=manual: log in and chmod freezer.state in
     /sys/fs/cgroup/freezer/chrome_renderers/to_be_frozen
     to 444; suspend and resume and check that chrome
     doesn't abort

Review-Url: https://codereview.chromium.org/2575933002
==========

[commit-bot: I haz the power, 2016-12-14 21:19:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-14 21:22:56]

Description was changed from

==========
chromeos: Avoid crash on thaw failure if freezing failed.

Avoid an "Unable to thaw renderers." LOG(FATAL) in
RendererFreezer if freezing the renderers also failed
earlier. This is a workaround for unexplained EBADF and
ENOENT errors seen when writing to cgroups.

BUG=chromium:661310
TEST=manual: log in and chmod freezer.state in
     /sys/fs/cgroup/freezer/chrome_renderers/to_be_frozen
     to 444; suspend and resume and check that chrome
     doesn't abort

Review-Url: https://codereview.chromium.org/2575933002
==========

to

==========
chromeos: Avoid crash on thaw failure if freezing failed.

Avoid an "Unable to thaw renderers." LOG(FATAL) in
RendererFreezer if freezing the renderers also failed
earlier. This is a workaround for unexplained EBADF and
ENOENT errors seen when writing to cgroups.

BUG=chromium:661310
TEST=manual: log in and chmod freezer.state in
     /sys/fs/cgroup/freezer/chrome_renderers/to_be_frozen
     to 444; suspend and resume and check that chrome
     doesn't abort

Committed: https://crrev.com/d66436c2b7a48ae82eda1af894fa79b81a5ffaad
Cr-Commit-Position: refs/heads/master@{#438625}
==========

[commit-bot: I haz the power, 2016-12-14 21:22:57]

Patchset 1 (id:??) landed as
https://crrev.com/d66436c2b7a48ae82eda1af894fa79b81a5ffaad
Cr-Commit-Position: refs/heads/master@{#438625}