tree 68bef9bca2ba9c3dfda933b9f265a918168a1e71
parent 38e3d616bc62fe5e0627f5b3878c824c4c174a16
author Miranda Kephart <mkephart@google.com> 1591300775 -0400
committer Miranda Kephart <mkephart@google.com> 1591303729 -0400

Only show error if screenshot process dies within timeout period

Currently, we show a "Couldn't take screenshot" error any time the
screenshot process disconnects. However, if the screenshot process
remains up, this can result in strange superfluous notifications,
where hours later the screenshot process gets frozen/killed due to
memory pressure, and we show the notification despite the user not
trying to take a screenshot at that point.

This change adds a check so that we only show the error notification
if the screenshot process is disconnected within the timeout period
(10 seconds).

Bug: 156064134
Fix: 156064134
Test: manual -- killed the process with 'adb shell ps | grep screenshot'
and then 'adb shell kill <pid>' to verify that the error notification
appeared, then made sure it did not appear post-change.

Change-Id: I2419634ad888300abc8b50aadab5888ce653e363