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

Issue 766082 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Sep 27
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 766299



Sign in to add a comment

Background Fetch in Incognito mode

Project Member Reported by delph...@chromium.org, Sep 18 2017

Issue description

From: https://chromium-review.googlesource.com/c/chromium/src/+/647751/18

Since incognito must not be detectable due to the absence of Background Fetch, then depending on the permission model we end up with for Background Fetch we'll probably end up needing to override GetBrowserContextToUse here to return chrome::GetBrowserContextOwnInstanceInIncognito(context);. We'll additionally need to work with the Download Service to special case such downloads so that they are only stored in memory, and are aborted if/when the browser is exited.

 
David, do you have a tracking bug for DownloadService and Incognito mode that we mark this as blocking on?
Owner: delph...@chromium.org
Blockedon: 766299
Added and marked as blocking.
Components: Privacy
Adding privacy for FYI
Owner: ----
Status: Available (was: Assigned)
Cc: rhalavati@chromium.org
Components: Privacy>Incognito
Labels: BlocksMVP
Project Member

Comment 9 by bugdroid1@chromium.org, Aug 10

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

commit 7c2fdfb867ef81d5f6d1e64579549f594310c948
Author: Rayan Kanso <rayankans@chromium.org>
Date: Fri Aug 10 13:30:09 2018

[Background Fetch] Lazily initialize download service used by the delegate.

If the storage partitions are not fully initialized, the delegate factory
will enter an infinite loop when trying to create an instance of
BackgroundFetchDelegateImpl.

This changes moves the service initialization logic from the constructor
to a lazy accessor function, which is guaranteed to be called after the IO
thread is up and running. This means all Storage Partitions will have
been loaded.

This behavior can be triggered when creating a
BackgroundFetchDelegateImpl in incognito mode.

Bug:  766082 
Change-Id: I320e146634792e16a61514620639ece38b022349
Reviewed-on: https://chromium-review.googlesource.com/1167842
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#582133}
[modify] https://crrev.com/7c2fdfb867ef81d5f6d1e64579549f594310c948/chrome/browser/background_fetch/background_fetch_delegate_impl.cc
[modify] https://crrev.com/7c2fdfb867ef81d5f6d1e64579549f594310c948/chrome/browser/background_fetch/background_fetch_delegate_impl.h

Status: Fixed (was: Available)

Sign in to add a comment