New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 848510 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jun 2018
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 759230



Sign in to add a comment

Workers should be able to access file URLs when created from a frame loaded from a file URL

Project Member Reported by jcivelli@chromium.org, May 31 2018

Issue description

Workers should be able to access file URLs when created from a frame loaded from a file URL when the network service is enabled.
This is going to be important when the file access is disabled by default in the network service.
 
Blocking: 759230
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 4 2018

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

commit e3d769d06fd4a3a94a8a08bdba9f1f0f618520e6
Author: Jay Civelli <jcivelli@google.com>
Date: Mon Jun 04 18:08:56 2018

Add file URL support to shared workers with the network service

Shared workers should have access to files if they are created from a frame
based off a file URL.
This is going to be important when file access is disabled by default in
the network service.

Bug:  848510 
Change-Id: Ie2a56ba5fd86b0dc8ee241da87184f4a7a22eb06
Reviewed-on: https://chromium-review.googlesource.com/1081657
Commit-Queue: Jay Civelli <jcivelli@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564152}
[modify] https://crrev.com/e3d769d06fd4a3a94a8a08bdba9f1f0f618520e6/content/browser/shared_worker/shared_worker_service_impl.cc
[modify] https://crrev.com/e3d769d06fd4a3a94a8a08bdba9f1f0f618520e6/content/browser/shared_worker/worker_browsertest.cc
[modify] https://crrev.com/e3d769d06fd4a3a94a8a08bdba9f1f0f618520e6/content/test/data/workers/single_worker.html

Status: Fixed (was: Untriaged)

Sign in to add a comment