commit d5706f0f6f16662881005a80bbbb9945a255d7aa
author Geoffrey Pitsch <gpitsch@google.com> Fri Oct 27 15:47:35 2017 -0400
committer Geoffrey Pitsch <gpitsch@google.com> Fri Oct 27 15:47:35 2017 -0400
tree bcf5107b4fbe17f57465cbc9534899a3815d2dae
parent 028b263084b9ab70e15f8f2ed1df9545032b7dd0

Fix potential race bug where notification may persist after job finish.

The job's OnFinished callback occurs on a separate thread from the
JobWatcher.  It's hypothetically possible that OnFinished is called
after the JobWatcher checks isFinished but before it calls notify().
The onFinished call's actions cancel the notification, but JobWatcher
could then reinstate it as a non-foreground notification.

Bug: 64974697
Test: docsui unit tests
Change-Id: Id5b9e4ae6ca31b560c243022283ea7be211ece98