More context for ServiceWorker related trace events |
||
Issue descriptionIt would be quite useful to have more context on ServiceWorker related trace events when doing deep dives (i.e. being able to follow a request for a specific assets across SW, CacheStorage, Network, Renderer, etc.) Use case: being able to explain where/how time is spent for resource request when a service worker is involved regardless of the underlying strategy (i.e. cache first, networkOnly, etc.). Some examples: - URL in Renderer=>SW=>ServiceWorkerContextClient::DispatchFetchEvent - URL in Renderer=>SW=>CacheStorageMsg_CacheMatchSuccess - URL in Browser=>IOThread=>ServiceWorkerHostMsg_FetchEventResponse - URL in Browser=>ServiceWorkerVersion=>ServiceWorkerVersion::Request There might other places where context would be useful. UI support to facilitate following the request across process/threads would be nice but just getting enough context should be useful. Attached a trace as a reference.
,
Jan 27 2017
(bug triaging process) kindly ping for an expired NextAction bug:)
,
Feb 10 2017
ping. shall we update NextAction?
,
Feb 10 2017
I guess this should be a tracking bug than one with a deadline. People should feel empowered to add missing trace information when they discover difficulties analyzing traces. And link to this bug.
,
Jun 14 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6e67d953ba38b4172a8bcd1928e49f65e86e8a1b commit 6e67d953ba38b4172a8bcd1928e49f65e86e8a1b Author: Makoto Shimazu <shimazu@chromium.org> Date: Thu Jun 14 09:25:57 2018 ServiceWorker: More traces for events dispatched to service workers Bug: 852202 , 669772, 806658 Change-Id: Ife9c41fbe08c1f414bb5459cba29d98d7e1fb892 Reviewed-on: https://chromium-review.googlesource.com/1098650 Commit-Queue: Makoto Shimazu <shimazu@chromium.org> Reviewed-by: Matt Falkenhagen <falken@chromium.org> Cr-Commit-Position: refs/heads/master@{#567205} [modify] https://crrev.com/6e67d953ba38b4172a8bcd1928e49f65e86e8a1b/content/common/service_worker/service_worker_utils.cc [modify] https://crrev.com/6e67d953ba38b4172a8bcd1928e49f65e86e8a1b/content/common/service_worker/service_worker_utils.h [modify] https://crrev.com/6e67d953ba38b4172a8bcd1928e49f65e86e8a1b/content/renderer/service_worker/service_worker_context_client.cc [modify] https://crrev.com/6e67d953ba38b4172a8bcd1928e49f65e86e8a1b/content/renderer/service_worker/web_service_worker_provider_impl.cc |
||
►
Sign in to add a comment |
||
Comment 1 by falken@chromium.org
, Dec 15 2016Owner: falken@chromium.org
Status: Assigned (was: Untriaged)