Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(482)

Issue 2525723002: Limit Link header based SW installations for fetches made by SW. (Closed)

Created:
4 years, 1 month ago by Marijn Kruisselbrink
Modified:
4 years, 1 month ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

Limit Link header based SW installations for fetches made by SW. To prevent a service worker from spawning new service workers in the background, only process link headers on requests made by service workers if the service worker is controlling at least one client. BUG=662443 Review-Url: https://codereview.chromium.org/2512103003 Cr-Commit-Position: refs/heads/master@{#433489} (cherry picked from commit e8313bb499c439b0a5091225faf97b06f4cf1182) Committed: https://chromium.googlesource.com/chromium/src/+/f89f2f4beec7dc806420025f9b76d2c731f918b6

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+77 lines, -11 lines) Patch
M content/browser/service_worker/link_header_support.cc View 1 chunk +11 lines, -2 lines 0 comments Download
M content/browser/service_worker/link_header_support_unittest.cc View 6 chunks +65 lines, -9 lines 0 comments Download
M content/browser/service_worker/service_worker_provider_host.h View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
Marijn Kruisselbrink
4 years, 1 month ago (2016-11-22 19:23:12 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
f89f2f4beec7dc806420025f9b76d2c731f918b6.

Powered by Google App Engine
This is Rietveld 408576698