ABANDONED: chromeos: Resume displays when suspend is cancelled.

chromeos: Resume displays when suspend is cancelled.

Fix a race where:
- A suspend attempt starts, resulting in displays getting
  reconfigured for suspend asynchronously.
- Before the config request finishes, the suspend attempt is
  cancelled/completed and we attempt to resume displays
  (which ends up being a no-op, as they're not yet
  suspended).
- Displays are finally suspended and remain that way even
  though the system has resumed.

Instead, note when a suspend attempt completes while the
suspend-displays task is still running and then resume the
displays when the task completes.

BUG=620208

[Daniel Erat, 2016-08-30 20:30:49]

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

[commit-bot: I haz the power, 2016-08-30 20:31:43]

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-08-30 21:26:41]

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

[commit-bot: I haz the power, 2016-08-30 21:26:42]

Dry run: This issue passed the CQ dry run.

[Daniel Erat, 2016-08-30 23:12:43]

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

[commit-bot: I haz the power, 2016-08-30 23:13:30]

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

[Daniel Erat, 2016-08-30 23:29:51]

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

[commit-bot: I haz the power, 2016-08-30 23:30:30]

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

[Daniel Erat, 2016-08-30 23:31:38]

derat@chromium.org changed reviewers:
+ djkurtz@chromium.org, dnicoara@chromium.org

[Daniel Erat, 2016-08-30 23:31:39]

calling all dans!

djkurtz@, does this resolve the problem you were seeing? please let me know if
my description of the race in the CL description is incorrect.

[commit-bot: I haz the power, 2016-08-31 00:14:27]

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

[commit-bot: I haz the power, 2016-08-31 00:14:28]

Dry run: This issue passed the CQ dry run.

[Daniel Erat, 2016-08-31 16:01:14]

On 2016/08/30 23:31:39, Daniel Erat wrote:
> calling all dans!
> 
> djkurtz@, does this resolve the problem you were seeing? please let me know if
> my description of the race in the CL description is incorrect.

closing, as it sounds like https://codereview.chromium.org/2154743003 makes this
unnecessary.

[Daniel Erat, 2016-08-31 16:01:27]

Description was changed from

==========
chromeos: Resume displays when suspend is cancelled.

Fix a race where:
- A suspend attempt starts, resulting in displays getting
  reconfigured for suspend asynchronously.
- Before the config request finishes, the suspend attempt is
  cancelled/completed and we attempt to resume displays
  (which ends up being a no-op, as they're not yet
  suspended).
- Displays are finally suspended and remain that way even
  though the system has resumed.

Instead, note when a suspend attempt completes while the
suspend-displays task is still running and then resume the
displays when the task completes.

BUG=620208
==========

to

==========
chromeos: Resume displays when suspend is cancelled.

Fix a race where:
- A suspend attempt starts, resulting in displays getting
  reconfigured for suspend asynchronously.
- Before the config request finishes, the suspend attempt is
  cancelled/completed and we attempt to resume displays
  (which ends up being a no-op, as they're not yet
  suspended).
- Displays are finally suspended and remain that way even
  though the system has resumed.

Instead, note when a suspend attempt completes while the
suspend-displays task is still running and then resume the
displays when the task completes.

BUG=620208
==========