Service Worker: Use more specific errors when StartWorker fails
commit9573d4d33bf81518dd38c732bd18bfb8851bb89a
authorfalken <falken@chromium.org>
Wed, 15 Apr 2015 16:49:39 +0000 (15 09:49 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 15 Apr 2015 16:50:08 +0000 (15 16:50 +0000)
tree8c57c651e265ad14eef64928fccf20342b3777fd
parentece9b5e4d3e45d874d893f8422be0796c26bb702
Service Worker: Use more specific errors when StartWorker fails

UMA shows StartWorker fails more than expected, but most errors are just
the generic START_WORKER_FAILED. Use a more specific error when appropriate.

This has some user-visible behavior change in the error message that register()
rejects with. For example, a syntax error used to fail with "ServiceWorker failed
to install"; now it fails with "ServiceWorker script evaluation failed".

A Blink layout test is added for error messages, which didn't have test
coverage previously.

BUG=448003
TEST=https://codereview.chromium.org/1058323004

Review URL: https://codereview.chromium.org/1063823005

Cr-Commit-Position: refs/heads/master@{#325261}
15 files changed:
content/browser/notifications/notification_event_dispatcher_impl.cc
content/browser/push_messaging/push_messaging_message_filter.cc
content/browser/push_messaging/push_messaging_router.cc
content/browser/service_worker/embedded_worker_instance.cc
content/browser/service_worker/service_worker_browsertest.cc
content/browser/service_worker/service_worker_context_wrapper.cc
content/browser/service_worker/service_worker_process_manager.cc
content/browser/service_worker/service_worker_process_manager.h
content/browser/service_worker/service_worker_register_job.cc
content/browser/service_worker/service_worker_registration_status.cc
content/browser/service_worker/service_worker_version.cc
content/browser/service_worker/service_worker_version.h
content/common/service_worker/service_worker_status_code.cc
content/common/service_worker/service_worker_status_code.h
tools/metrics/histograms/histograms.xml