New issue
Advanced search Search tips

Issue 732682 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 732681



Sign in to add a comment

Add a //content API for the omnibox to call to signal that a navigation to a URL will likely occur

Project Member Reported by falken@chromium.org, Jun 13 2017

Issue description

The omnibox could call this API and //content could warm up the service worker for the URL.
 

Comment 1 by falken@chromium.org, Jun 13 2017

Blocking: 732681
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 14 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa

commit a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa
Author: Tsuyoshi Horo <horo@chromium.org>
Date: Wed Jun 14 14:47:16 2017

Implement ServiceWorkerContext::StartServiceWorkerForNavigationHint()

Bug:  732682 
Change-Id: Ibc2fba5cc4de4473635dfdb5e0695695f1576411
Reviewed-on: https://chromium-review.googlesource.com/530731
Reviewed-by: Mark Pearson (away June 14) <mpearson@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#479379}
[modify] https://crrev.com/a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa/chrome/browser/chrome_service_worker_browsertest.cc
[modify] https://crrev.com/a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa/content/browser/service_worker/service_worker_context_wrapper.cc
[modify] https://crrev.com/a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa/content/browser/service_worker/service_worker_context_wrapper.h
[modify] https://crrev.com/a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa/content/browser/service_worker/service_worker_metrics.cc
[modify] https://crrev.com/a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa/content/browser/service_worker/service_worker_metrics.h
[modify] https://crrev.com/a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa/content/public/browser/service_worker_context.h
[modify] https://crrev.com/a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa/tools/metrics/histograms/histograms.xml

Project Member

Comment 3 by bugdroid1@chromium.org, Jun 28 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/eb576e6c660bc1ce8e7aaf93bfab70042479a237

commit eb576e6c660bc1ce8e7aaf93bfab70042479a237
Author: Tsuyoshi Horo <horo@chromium.org>
Date: Wed Jun 28 08:04:36 2017

Reuse unmatched service worker processes for navigation

This CL introduces UnmatchedServiceWorkerProcessTracker which tracks the
unmatched service processes. The unmatched service processes will be reused for
the later navigation to the same site.

See the doc:
https://docs.google.com/a/chromium.org/document/d/1iT8xvo18-zmlsU2iHqH2RCIXrWoGJOIaZ6aDASubi1Q/edit?usp=sharing

Bug:  732682 ,600707
Change-Id: I15b970afbdb6801d1124616be316c3e4c6e0e9d1
Reviewed-on: https://chromium-review.googlesource.com/535220
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Charlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#482916}
[modify] https://crrev.com/eb576e6c660bc1ce8e7aaf93bfab70042479a237/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/eb576e6c660bc1ce8e7aaf93bfab70042479a237/content/browser/renderer_host/render_process_host_unittest.cc
[modify] https://crrev.com/eb576e6c660bc1ce8e7aaf93bfab70042479a237/content/browser/service_worker/service_worker_process_manager.cc
[modify] https://crrev.com/eb576e6c660bc1ce8e7aaf93bfab70042479a237/content/browser/site_instance_impl.cc
[modify] https://crrev.com/eb576e6c660bc1ce8e7aaf93bfab70042479a237/content/browser/site_instance_impl.h

Comment 4 by horo@chromium.org, Jul 3 2017

Status: Fixed (was: Started)
Project Member

Comment 5 by bugdroid1@chromium.org, Jul 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/96ca7166963bd085be29f050065dc8bc421f54c3

commit 96ca7166963bd085be29f050065dc8bc421f54c3
Author: Tsuyoshi Horo <horo@chromium.org>
Date: Mon Jul 10 10:24:37 2017

Add NAVIGATION_HINT to ServiceWorkerMetrics.EventType in enum.xml

This is a follow-up to https://crrev.com/a1c6f54f17c58f01f8aed5c5a0d1c5c095ced1aa

Bug:  732682 
Change-Id: I512acd72393b8467cfd801411e6113b14dcfa0aa
Reviewed-on: https://chromium-review.googlesource.com/564913
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485224}
[modify] https://crrev.com/96ca7166963bd085be29f050065dc8bc421f54c3/tools/metrics/histograms/enums.xml

Sign in to add a comment