Skip logging spurious errors for ServiceWorker URLRequests that were spawned

Skip logging spurious errors for ServiceWorker URLRequests that were spawned
  through ServiceWorkerWriteToCacheJob.
This CL does not fix the underlying issue, it is just suppressing a not so
  useful error log.

BUG=568788
Test=Load an extension page with ServiceWorker, observe no
"Allowing load of ..." error messages anymore.

Committed: https://crrev.com/16b16fba24fa3cac9e5c852cb707a5317c34ddad
Cr-Commit-Position: refs/heads/master@{#381517}

[falken, 2016-03-14 00:49:58]

falken@chromium.org changed reviewers:
+ falken@chromium.org

[falken, 2016-03-14 00:49:58]

lgtm

[lazyboy, 2016-03-15 11:02:58]

lazyboy@chromium.org changed reviewers:
+ asargent@chromium.org

[lazyboy, 2016-03-15 11:02:59]

+asargent for OWNERS review.

[asargent_no_longer_on_chrome, 2016-03-15 16:41:29]

lgtm

https://codereview.chromium.org/1790993002/diff/1/extensions/browser/extensio...
File extensions/browser/extension_protocols.cc (right):

https://codereview.chromium.org/1790993002/diff/1/extensions/browser/extensio...
extensions/browser/extension_protocols.cc:333: // does not have
ResourceRequestInfo associated with them. So skip logging
grammar nit: should be "... do not have..."

[lazyboy, 2016-03-16 19:47:08]

The CQ bit was checked by lazyboy@chromium.org

[lazyboy, 2016-03-16 19:47:08]

The patchset sent to the CQ was uploaded after l-g-t-m from falken@chromium.org,
asargent@chromium.org
Link to the patchset: https://codereview.chromium.org/1790993002/#ps20001
(title: "address comments from asargent@")

[commit-bot: I haz the power, 2016-03-16 19:47:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1790993002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1790993002/20001

[lazyboy, 2016-03-16 19:48:26]

https://codereview.chromium.org/1790993002/diff/1/extensions/browser/extensio...
File extensions/browser/extension_protocols.cc (right):

https://codereview.chromium.org/1790993002/diff/1/extensions/browser/extensio...
extensions/browser/extension_protocols.cc:333: // does not have
ResourceRequestInfo associated with them. So skip logging
On 2016/03/15 16:41:29, Antony Sargent wrote:
> grammar nit: should be "... do not have..."

Done.

[commit-bot: I haz the power, 2016-03-16 20:10:02]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-16 20:11:53]

Description was changed from

==========
Skip logging spurious errors for ServiceWorker URLRequests that were spawned
  through ServiceWorkerWriteToCacheJob.
This CL does not fix the underlying issue, it is just suppressing a not so
  useful error log.

BUG=568788
Test=Load an extension page with ServiceWorker, observe no
"Allowing load of ..." error messages anymore.
==========

to

==========
Skip logging spurious errors for ServiceWorker URLRequests that were spawned
  through ServiceWorkerWriteToCacheJob.
This CL does not fix the underlying issue, it is just suppressing a not so
  useful error log.

BUG=568788
Test=Load an extension page with ServiceWorker, observe no
"Allowing load of ..." error messages anymore.

Committed: https://crrev.com/16b16fba24fa3cac9e5c852cb707a5317c34ddad
Cr-Commit-Position: refs/heads/master@{#381517}
==========

[commit-bot: I haz the power, 2016-03-16 20:11:54]

Patchset 2 (id:??) landed as
https://crrev.com/16b16fba24fa3cac9e5c852cb707a5317c34ddad
Cr-Commit-Position: refs/heads/master@{#381517}