Use job id from JobEventDetails to create CupsPrintJobs.

Use job id from JobEventDetails to create CupsPrintJobs.

By using job id from JobEventDetails, we can avoid querying CUPS to
display the initial notification and we don't have to guess if we've
identified the correct job.  This wasn't done originally, because job id
was not present in JobEventDetails.

Additionally, add some rate limiting for queries and check for jobs which
we lose track of.

Speculatively, this fixes 682853 as cupsd can become overwhelmed and stop responding to queries.

BUG=684853, 682853
Review-Url: https://codereview.chromium.org/2699273002
Cr-Commit-Position: refs/heads/master@{#452281}
Committed: https://chromium.googlesource.com/chromium/src/+/6157021217dd1922d6f5dcc8fd00b6e36bbb3a32

[skau, 2017-02-18 02:23:40]

Description was changed from

==========
Use job id from JobEventDetails to crete CupsPrintJobs.

By using job id from JobEventDetails, we can avoid querying CUPS to
display the initial notification and we don't have to guess if we've
identified the correct job.  This wasn't done originally, because job id
was not present in JobEventDetails.

Additionally, add some rate limiting for queries and check for jobs which
we lose track of.

BUG=684853
==========

to

==========
Use job id from JobEventDetails to create CupsPrintJobs.

By using job id from JobEventDetails, we can avoid querying CUPS to
display the initial notification and we don't have to guess if we've
identified the correct job.  This wasn't done originally, because job id
was not present in JobEventDetails.

Additionally, add some rate limiting for queries and check for jobs which
we lose track of.

Speculatively, this fixes 682853 as cupsd can become overwhelmed and stop
responding to queries.

BUG=684853,682853
==========

[skau, 2017-02-18 02:23:43]

skau@chromium.org changed reviewers:
+ justincarlson@chromium.org, xdai@google.com

[skau, 2017-02-18 02:24:22]

Please take a look!

[skau, 2017-02-22 21:44:12]

I guess this got lost.  Can you take a look? :)

[xdai1, 2017-02-22 22:34:18]

On 2017/02/22 21:44:12, skau wrote:
> I guess this got lost.  Can you take a look? :)

Missed it. lgtm. Thanks!

[Carlson, 2017-02-22 23:08:06]

lgtm

Apologies for missing this.

[skau, 2017-02-22 23:13:10]

The CQ bit was checked by skau@chromium.org

[commit-bot: I haz the power, 2017-02-22 23:15:10]

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, 2017-02-22 23:51:56]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487805190043180,
"parent_rev": "e10fdac86b82f2e64a63815a9e08789582e6c8f9", "commit_rev":
"6157021217dd1922d6f5dcc8fd00b6e36bbb3a32"}

[commit-bot: I haz the power, 2017-02-22 23:52:37]

Description was changed from

==========
Use job id from JobEventDetails to create CupsPrintJobs.

By using job id from JobEventDetails, we can avoid querying CUPS to
display the initial notification and we don't have to guess if we've
identified the correct job.  This wasn't done originally, because job id
was not present in JobEventDetails.

Additionally, add some rate limiting for queries and check for jobs which
we lose track of.

Speculatively, this fixes 682853 as cupsd can become overwhelmed and stop
responding to queries.

BUG=684853,682853
==========

to

==========
Use job id from JobEventDetails to create CupsPrintJobs.

By using job id from JobEventDetails, we can avoid querying CUPS to
display the initial notification and we don't have to guess if we've
identified the correct job.  This wasn't done originally, because job id
was not present in JobEventDetails.

Additionally, add some rate limiting for queries and check for jobs which
we lose track of.

Speculatively, this fixes 682853 as cupsd can become overwhelmed and stop
responding to queries.

BUG=684853,682853

Review-Url: https://codereview.chromium.org/2699273002
Cr-Commit-Position: refs/heads/master@{#452281}
Committed:
https://chromium.googlesource.com/chromium/src/+/6157021217dd1922d6f5dcc8fd00...
==========

[commit-bot: I haz the power, 2017-02-22 23:52:37]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/6157021217dd1922d6f5dcc8fd00...