New issue
Advanced search Search tips

Issue 899276 link

Starred by 1 user
Status: Fixed
Owner:
hanxi@chromium.org
Closed: Oct 26
Cc:
jam@chromium.org
qin...@chromium.org
hnakashima@chromium.org
yfried...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 729596



Sign in to add a comment

Set default cookie policy in early startup

Project Member Reported by hanxi@chromium.org, Oct 26 
The default cookie policy should be set before any url request. Therefore, the  SetDefaultCookiePolicyToBlock() should be called in early startup, before the network service is running.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 26 

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

commit 778cb84a2f6c541c5ba81efa951d4a568dbc2720
Author: Xi Han <hanxi@google.com>
Date: Fri Oct 26 18:18:31 2018

Set default cookie policy early.

SetDefaultCookiePolicyToBlock() should be called before any url request. Call it
it in the early startup.

Bug:  899276 
Change-Id: Idf747013cd19d057506d7e0314a2d8c64c413c29
Reviewed-on: https://chromium-review.googlesource.com/c/1301842
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Xi Han <hanxi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#603151}
[modify] https://crrev.com/778cb84a2f6c541c5ba81efa951d4a568dbc2720/chrome/app/chrome_main_delegate.cc
[modify] https://crrev.com/778cb84a2f6c541c5ba81efa951d4a568dbc2720/chrome/browser/chrome_browser_main.cc

Comment 2 by hanxi@chromium.org, Oct 26

Status: Fixed (was: Started)
►

Sign in to add a comment