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

Issue 789644 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 19
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 1
Type: Bug
Proj-Servicification

Blocking:
issue 598073
issue 789640
issue 792776


Show other hotlists

Hotlists containing this issue:
XXX


Sign in to add a comment

Configure CookieStore and ChannelIDStore when using the network service

Project Member Reported by mmenke@chromium.org, Nov 29 2017

Issue description

The CookieStore and ChannelIDStore need to be configurable when the network service is enabled.

Beyond just where to store them, QuotaPolicyCookieStore and QuotaPolicyChannelIDStore allow the embedder to configure per-origin policies.  We also use encrypted storage of some sort.
 

Comment 1 by mmenke@chromium.org, Nov 29 2017

Cc: mmenke@chromium.org rdsmith@chromium.org jam@chromium.org yzshen@chromium.org
 Issue 757960  has been merged into this issue.
Blocking: 792776
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 12 2017

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

commit a796fa5605119535041802efe09b9e98099273d1
Author: John Abd-El-Malek <jam@chromium.org>
Date: Tue Dec 12 17:12:32 2017

Fix session restore with network service.

This involved:
-persisting cookies with network service
-converting the browser tests to work with the network service

Bug:  789644 ,  769401 ,  776589 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I70320d5550e433049a910c64f055c1809aecbff1
Reviewed-on: https://chromium-review.googlesource.com/818486
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523462}
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/android_webview/browser/cookie_manager.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/net/errorpage_browsertest.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/net/profile_network_context_service.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/profiles/off_the_record_profile_io_data.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/profiles/profile.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/profiles/profile.h
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/profiles/profile_impl.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/profiles/profile_impl.h
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/profiles/profile_impl_io_data.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/profiles/profile_impl_io_data.h
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chrome/browser/sessions/better_session_restore_browsertest.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/chromecast/browser/url_request_context_factory.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/components/safe_browsing/browser/safe_browsing_url_request_context_getter.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/browser/net/quota_policy_cookie_store.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/browser/notification_service_impl.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/network/BUILD.gn
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/network/DEPS
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/network/network_context.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/public/browser/cookie_store_factory.h
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/public/browser/notification_service.h
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/public/common/network_service.mojom
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/public/test/url_loader_interceptor.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/public/test/url_loader_interceptor.h
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/content/utility/utility_main.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/headless/lib/browser/headless_url_request_context_getter.cc
[modify] https://crrev.com/a796fa5605119535041802efe09b9e98099273d1/testing/buildbot/filters/mojo.fyi.network_browser_tests.filter

Owner: reillyg@chromium.org
Cc: -rdsmith@chromium.org
Cc: morlovich@chromium.org

Comment 7 by dxie@chromium.org, May 22 2018

Labels: -Pri-3 Proj-Servicification-Canary OS-Android OS-Chrome OS-Linux OS-Mac OS-Windows Pri-1

Comment 8 by mmenke@chromium.org, Jun 18 2018

[reillyg]:  Looks like https://chromium-review.googlesource.com/c/chromium/src/+/1090035 landed last week, which looks to handle the cookie bits of session cleanup magic.
Cc: cduvall@chromium.org
cduvall@, since you have some context here can you look at what other aspects of cookie (and channel ID) storage remain unconfigured when we enable the Network Service?
Owner: cduvall@chromium.org
Status: Started (was: Assigned)
The remaining things for CookieStore and ChannelIDStore that I know of are:
- ChannelID session cleanup, similar to the cookie session cleanup
- Encrypted storage for cookies

I'll take this bug since I have some context about whats going on and needs to be done.
Project Member

Comment 11 by bugdroid1@chromium.org, Jun 20 2018

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

commit a7501a27f3ca3ccfca8afe34ed9b12d6b7763137
Author: Clark DuVall <cduvall@chromium.org>
Date: Wed Jun 20 17:34:02 2018

Network Service: Handle deleting session only channel IDs

SessionCleanupChannelIDStore and tests are mostly copied from
QuotaPolicyChannelIDStore.


Bug:  789644 ,  789632 
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: I1178b774273ac39ae86b1aff3be94be1e1b00058
Reviewed-on: https://chromium-review.googlesource.com/1107132
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568901}
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/chrome/browser/net/DEPS
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/chrome/browser/net/quota_policy_channel_id_store.cc
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/chrome/browser/net/quota_policy_channel_id_store.h
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/components/content_settings/core/browser/cookie_settings.cc
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/components/content_settings/core/browser/cookie_settings.h
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/components/content_settings/core/common/cookie_settings_base.cc
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/components/content_settings/core/common/cookie_settings_base.h
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/components/content_settings/core/common/cookie_settings_base_unittest.cc
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/services/network/BUILD.gn
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/services/network/cookie_manager.cc
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/services/network/cookie_manager.h
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/services/network/cookie_manager_unittest.cc
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/services/network/network_context.cc
[modify] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/services/network/network_context.h
[add] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/services/network/session_cleanup_channel_id_store.cc
[add] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/services/network/session_cleanup_channel_id_store.h
[add] https://crrev.com/a7501a27f3ca3ccfca8afe34ed9b12d6b7763137/services/network/session_cleanup_channel_id_store_unittest.cc

Project Member

Comment 12 by bugdroid1@chromium.org, Jun 26 2018

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

commit 7b387c3562029b6633db551855fdebc9f121d90d
Author: Clark DuVall <cduvall@chromium.org>
Date: Tue Jun 26 19:08:47 2018

Network Service: Configure CookieManager using ProfileNetworkContextService

As suggested by mmenke@ on
https://chromium-review.googlesource.com/c/chromium/src/+/1100105, moved
cookie settings syncing logic to ProfileNetworkContextService. This also
syncs settings even when network service isn't enabled, which will allow
us to unify the logic for OnCanGetCookies and OnCanSetCookie from
ChromeNetworkDelegate and NetworkServiceNetworkDelegate as described in
 crbug.com/789636#c8 .


Bug:  789644 
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: Ia503145887a5ba877713a2b019791bc56f79ae8a
Reviewed-on: https://chromium-review.googlesource.com/1112726
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570482}
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/chrome/browser/net/network_context_configuration_browsertest.cc
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/chrome/browser/net/profile_network_context_service.cc
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/chrome/browser/net/profile_network_context_service.h
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/chrome/browser/profiles/profile_impl.cc
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/chrome/browser/profiles/profile_impl.h
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/services/network/cookie_manager.cc
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/services/network/cookie_manager.h
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/services/network/cookie_manager_unittest.cc
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/services/network/network_context.cc
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/services/network/public/mojom/cookie_manager.mojom
[modify] https://crrev.com/7b387c3562029b6633db551855fdebc9f121d90d/services/network/public/mojom/network_context.mojom

Cc: dougt@chromium.org
 Issue 848361  has been merged into this issue.
Project Member

Comment 14 by bugdroid1@chromium.org, Jul 4

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

commit 87135a31e034842e4e2030b309276e8851008cc4
Author: Clark DuVall <cduvall@chromium.org>
Date: Wed Jul 04 01:49:54 2018

Network Service: Enable encrypted cookies

Cookies will now be stored encrypted by default in the network service.
Tested that  crbug.com/848361  is fixed with this change.

Bug:  789632 ,  848361 ,  789644 
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: Ic627501ddf1c5030bbf2a203f005f5ebca92dfd8
Reviewed-on: https://chromium-review.googlesource.com/1104791
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Maks Orlovich <morlovich@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Christos Froussios <cfroussios@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572444}
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/chrome/browser/chrome_browser_main_linux.cc
[add] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/chrome/browser/chrome_network_service_browsertest.cc
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/chrome/browser/net/system_network_context_manager.cc
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/chrome/test/BUILD.gn
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/components/safe_browsing/browser/safe_browsing_network_context.cc
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/services/network/BUILD.gn
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/services/network/DEPS
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/services/network/network_context.cc
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/services/network/network_service.cc
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/services/network/network_service.h
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/services/network/network_service_unittest.cc
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/services/network/public/mojom/BUILD.gn
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/services/network/public/mojom/network_context.mojom
[modify] https://crrev.com/87135a31e034842e4e2030b309276e8851008cc4/services/network/public/mojom/network_service.mojom

Project Member

Comment 15 by bugdroid1@chromium.org, Jul 4

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

commit ea0521dd24669f34f0d64927eef0fa847b677eb0
Author: Makoto Shimazu <shimazu@chromium.org>
Date: Wed Jul 04 06:04:00 2018

Revert "Network Service: Enable encrypted cookies"

This reverts commit 87135a31e034842e4e2030b309276e8851008cc4.

Reason for revert: ChromeNetworkServiceBrowserTest.EncryptedCookies
keeps failing on Mac10.11 bot.
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.11%20Tests/27835

Original change's description:
> Network Service: Enable encrypted cookies
> 
> Cookies will now be stored encrypted by default in the network service.
> Tested that  crbug.com/848361  is fixed with this change.
> 
> Bug:  789632 ,  848361 ,  789644 
> Cq-Include-Trybots: luci.chromium.try:linux_mojo
> Change-Id: Ic627501ddf1c5030bbf2a203f005f5ebca92dfd8
> Reviewed-on: https://chromium-review.googlesource.com/1104791
> Commit-Queue: Clark DuVall <cduvall@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Maks Orlovich <morlovich@chromium.org>
> Reviewed-by: Matt Menke <mmenke@chromium.org>
> Reviewed-by: Christos Froussios <cfroussios@chromium.org>
> Reviewed-by: John Abd-El-Malek <jam@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#572444}

TBR=dcheng@chromium.org,jam@chromium.org,mmenke@chromium.org,cfroussios@chromium.org,morlovich@chromium.org,cduvall@chromium.org

Change-Id: I192a10c6d28a102dcf5ae247cdbda4c602872eb9
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug:  789632 ,  848361 ,  789644 
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Reviewed-on: https://chromium-review.googlesource.com/1125460
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Commit-Queue: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572490}
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/chrome/browser/chrome_browser_main_linux.cc
[delete] https://crrev.com/2de92717271b3a0bd9eaf260bed975b6e9c0e76a/chrome/browser/chrome_network_service_browsertest.cc
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/chrome/browser/net/system_network_context_manager.cc
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/chrome/test/BUILD.gn
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/components/safe_browsing/browser/safe_browsing_network_context.cc
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/services/network/BUILD.gn
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/services/network/DEPS
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/services/network/network_context.cc
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/services/network/network_service.cc
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/services/network/network_service.h
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/services/network/network_service_unittest.cc
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/services/network/public/mojom/BUILD.gn
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/services/network/public/mojom/network_context.mojom
[modify] https://crrev.com/ea0521dd24669f34f0d64927eef0fa847b677eb0/services/network/public/mojom/network_service.mojom

Project Member

Comment 16 by bugdroid1@chromium.org, Jul 6

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

commit 1c6f0e1c732f129237c96e7e01622da7d54bdd7a
Author: Clark DuVall <cduvall@chromium.org>
Date: Fri Jul 06 17:31:34 2018

Reland "Network Service: Enable encrypted cookies"

This is a reland of 87135a31e034842e4e2030b309276e8851008cc4

Original change's description:
> Network Service: Enable encrypted cookies
>
> Cookies will now be stored encrypted by default in the network service.
> Tested that  crbug.com/848361  is fixed with this change.
>
> Bug:  789632 ,  848361 ,  789644 
> Cq-Include-Trybots: luci.chromium.try:linux_mojo
> Change-Id: Ic627501ddf1c5030bbf2a203f005f5ebca92dfd8
> Reviewed-on: https://chromium-review.googlesource.com/1104791
> Commit-Queue: Clark DuVall <cduvall@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Maks Orlovich <morlovich@chromium.org>
> Reviewed-by: Matt Menke <mmenke@chromium.org>
> Reviewed-by: Christos Froussios <cfroussios@chromium.org>
> Reviewed-by: John Abd-El-Malek <jam@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#572444}

TBR=jam@chromium.org,dcheng@chromium.org,mmenke@chromium.org,cfroussios@chromium.org,morlovich@chromium.org

Bug:  789632 ,  848361 ,  789644 
Change-Id: I68ec0249a0f8fba8563c4a24abb6482da33d4ebc
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Reviewed-on: https://chromium-review.googlesource.com/1127960
Reviewed-by: Clark DuVall <cduvall@chromium.org>
Commit-Queue: Clark DuVall <cduvall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#573003}
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/chrome/browser/chrome_browser_main_linux.cc
[add] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/chrome/browser/chrome_network_service_browsertest.cc
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/chrome/browser/net/system_network_context_manager.cc
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/chrome/test/BUILD.gn
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/components/safe_browsing/browser/safe_browsing_network_context.cc
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/services/network/BUILD.gn
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/services/network/DEPS
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/services/network/network_context.cc
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/services/network/network_service.cc
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/services/network/network_service.h
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/services/network/network_service_unittest.cc
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/services/network/public/mojom/BUILD.gn
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/services/network/public/mojom/network_context.mojom
[modify] https://crrev.com/1c6f0e1c732f129237c96e7e01622da7d54bdd7a/services/network/public/mojom/network_service.mojom

Status: Fixed (was: Started)
I believe this is all working now, feel free to reopen if there is anything left.
Labels: TE-Verified-69.0.3493.0 TE-Verified-M69
As duped issue-848361 is working as intended on windows using latest canary-69.0.3493.0,tested the same on Mac , debian &adding TE Verified labels here.

Thanks..!
Cc: pbomm...@chromium.org
Status: Assigned (was: Fixed)
I see that we still have this issue and below are steps which reproduces the issue :


Steps to reproduce :
1. Install and launch Chrome Canary 
2. From Chrome://flags -->Enable "Enable network service" flag from "About://flags"
3. Relaunch Chrome
4. Visit "https://www.gmail.com" and signin.
5. Make sure you are signed in and also verify that "chrome://settings/content/cookies --> Keep local data only until you quit your browser" isn't enabled.
6. From Chrome://flags -->Disable "Enable network service" flag from "About://flags"

What is the expected result?
Should still be signed in to gmail.com 

What happens instead?
Kicked me out of gmail.


Since I am still able to reproduce this issue changing the status back to assigned, please let me know if above issue should be tracked separately.
I don't think the channel ID store was ever hooked up.
I'm not able to reproduce this, what machine and Chrome version are you using?
Tried this on Windows 7 and 10 with Chrome version 69.0.3495.0.
cduvall kindly pointed out to me that the channel ID store is apparently not encrypted, unlike the cookie store, so I'm not sure why this wouldn't be working.
Looked into this a bit more, I can repro by:

1. Enable network service
2. Sign in to gmail
3. Restart chrome

It looks like network service isn't persisting cookies for some reason.
After some more investigation, network service seems to be persisting cookies fine, but this is an issue with Chrome signin. I noticed that when logging into gmail without network service, it also logs into Chrome. This does not happen with network service enabled. Even if you try to log in by clicking the account button -> "Sign in" with network service enabled, you get a 404. I believe this is the root cause.
My best guess is that signin::FixAccountConsistencyRequestHeader is no longer being run on that header when network service is disabled, which is causing some issues.
Status: Fixed (was: Assigned)
I'm going to close this, I believe the bug we are seeing is signin not working with network service:  crbug.com/789670 
Project Member

Comment 28 by bugdroid1@chromium.org, Jul 27

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

commit 41e3c57070c2c1ad1aed15c4b0be89a8df4083f7
Author: Matt Menke <mmenke@chromium.org>
Date: Fri Jul 27 19:05:47 2018

NetworkContext: Always create the CookieStore and ChannelIDStore

This CL makes NetworkContexts create CookieStores and ChannelIDStores in
the hybrid URLRequestContext creation path, where the embedder starts
configuring a URLRequestContextBuilder, and the NetworkContext finishes
configuring it.

The reduces redundant code, lets us test the new path in production
before we ship the network service, and will allow Isolated App
URLRequestContexts to need less code when they're updated to use the
hybrid configuration path (Which I plan to do in a followup CL).

The NetworkContext constructor that takes an already-initialized
URLRequestContext is not modified in this CL.

Bug:  789644 
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: Ibf40ae6497e7be06b0a666bc664ea943e145cae9
Reviewed-on: https://chromium-review.googlesource.com/1152083
Reviewed-by: Maks Orlovich <morlovich@chromium.org>
Reviewed-by: Clark DuVall <cduvall@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578740}
[modify] https://crrev.com/41e3c57070c2c1ad1aed15c4b0be89a8df4083f7/chrome/browser/io_thread.cc
[modify] https://crrev.com/41e3c57070c2c1ad1aed15c4b0be89a8df4083f7/chrome/browser/profiles/off_the_record_profile_io_data.cc
[modify] https://crrev.com/41e3c57070c2c1ad1aed15c4b0be89a8df4083f7/chrome/browser/profiles/profile_impl_io_data.cc
[modify] https://crrev.com/41e3c57070c2c1ad1aed15c4b0be89a8df4083f7/services/network/network_context.cc
[modify] https://crrev.com/41e3c57070c2c1ad1aed15c4b0be89a8df4083f7/services/network/network_context.h
[modify] https://crrev.com/41e3c57070c2c1ad1aed15c4b0be89a8df4083f7/services/network/network_context_unittest.cc
[modify] https://crrev.com/41e3c57070c2c1ad1aed15c4b0be89a8df4083f7/services/network/network_service.cc
[modify] https://crrev.com/41e3c57070c2c1ad1aed15c4b0be89a8df4083f7/services/network/network_service.h

Project Member

Comment 29 by bugdroid1@chromium.org, Jul 27

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

commit 77605f5e56f0e385a420268db712cb060e04a1b2
Author: Matt Menke <mmenke@chromium.org>
Date: Fri Jul 27 21:36:16 2018

Revert "NetworkContext: Always create the CookieStore and ChannelIDStore"

This reverts commit 41e3c57070c2c1ad1aed15c4b0be89a8df4083f7.

Reason for revert:  https://crbug.com/868536  - DCHECK being hit because OSCrypt isn't set up.

Original change's description:
> NetworkContext: Always create the CookieStore and ChannelIDStore
> 
> This CL makes NetworkContexts create CookieStores and ChannelIDStores in
> the hybrid URLRequestContext creation path, where the embedder starts
> configuring a URLRequestContextBuilder, and the NetworkContext finishes
> configuring it.
> 
> The reduces redundant code, lets us test the new path in production
> before we ship the network service, and will allow Isolated App
> URLRequestContexts to need less code when they're updated to use the
> hybrid configuration path (Which I plan to do in a followup CL).
> 
> The NetworkContext constructor that takes an already-initialized
> URLRequestContext is not modified in this CL.
> 
> Bug:  789644 
> Cq-Include-Trybots: luci.chromium.try:linux_mojo
> Change-Id: Ibf40ae6497e7be06b0a666bc664ea943e145cae9
> Reviewed-on: https://chromium-review.googlesource.com/1152083
> Reviewed-by: Maks Orlovich <morlovich@chromium.org>
> Reviewed-by: Clark DuVall <cduvall@chromium.org>
> Commit-Queue: Matt Menke <mmenke@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578740}

TBR=mmenke@chromium.org,morlovich@chromium.org,cduvall@chromium.org

Change-Id: Ia9f914fac8b0fa624210363dd55333a0dfa20531
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug:  789644 
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Reviewed-on: https://chromium-review.googlesource.com/1153232
Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578808}
[modify] https://crrev.com/77605f5e56f0e385a420268db712cb060e04a1b2/chrome/browser/io_thread.cc
[modify] https://crrev.com/77605f5e56f0e385a420268db712cb060e04a1b2/chrome/browser/profiles/off_the_record_profile_io_data.cc
[modify] https://crrev.com/77605f5e56f0e385a420268db712cb060e04a1b2/chrome/browser/profiles/profile_impl_io_data.cc
[modify] https://crrev.com/77605f5e56f0e385a420268db712cb060e04a1b2/services/network/network_context.cc
[modify] https://crrev.com/77605f5e56f0e385a420268db712cb060e04a1b2/services/network/network_context.h
[modify] https://crrev.com/77605f5e56f0e385a420268db712cb060e04a1b2/services/network/network_context_unittest.cc
[modify] https://crrev.com/77605f5e56f0e385a420268db712cb060e04a1b2/services/network/network_service.cc
[modify] https://crrev.com/77605f5e56f0e385a420268db712cb060e04a1b2/services/network/network_service.h

Project Member

Comment 30 by bugdroid1@chromium.org, Jul 31

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

commit ef8aab106496a6b738170e26be9603be1fa15fa4
Author: Matt Menke <mmenke@chromium.org>
Date: Tue Jul 31 14:42:17 2018

Convert os_crypt from a source_set to a component

services/network and chrome/ both depend on os_crypt, which creates
problems with the component build when the NetworkService is
instantiated in-process, and caused
https://chromium.googlesource.com/chromium/src/+/41e3c57070c2 to be
reverted.

It also makes the component build to link os_crypt at least twice,
which seems not great.

Bug:  789644 ,  868536 
Change-Id: Ie964c51ea2b85a7ace63f76bb39524537fe69e2f
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Reviewed-on: https://chromium-review.googlesource.com/1155309
Reviewed-by: Christos Froussios <cfroussios@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579403}
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/BUILD.gn
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/ie7_password_win.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/key_storage_config_linux.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/key_storage_keyring.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/key_storage_kwallet.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/key_storage_libsecret.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/key_storage_linux.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/key_storage_util_linux.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/keychain_password_mac.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/keyring_util_linux.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/kwallet_dbus.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/libsecret_util_linux.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/os_crypt.h
[modify] https://crrev.com/ef8aab106496a6b738170e26be9603be1fa15fa4/components/os_crypt/os_crypt_switches.h

Project Member

Comment 31 by bugdroid1@chromium.org, Jul 31

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

commit df45365eaf5d9a3474356bc3d18a401645cbdf5c
Author: Matt Menke <mmenke@chromium.org>
Date: Tue Jul 31 17:33:10 2018

Reland "NetworkContext: Always create the CookieStore and ChannelIDStore"

This reverts commit 77605f5e56f0e385a420268db712cb060e04a1b2.

Reason for revert: The CL was reverted because two components
were depending on the os_crypt source_set, resulting in issues
with the component build.  I made os_crypt into a component
itself in https://chromium-review.googlesource.com/1155309,
which should fix the problem, and make the CL safe to re-land,
without modification.

Original change's description:
> Revert "NetworkContext: Always create the CookieStore and ChannelIDStore"
>
> This reverts commit 41e3c57070c2c1ad1aed15c4b0be89a8df4083f7.
>
> Reason for revert:  https://crbug.com/868536  - DCHECK being hit because OSCrypt isn't set up.
>
> Original change's description:
> > NetworkContext: Always create the CookieStore and ChannelIDStore
> >
> > This CL makes NetworkContexts create CookieStores and ChannelIDStores in
> > the hybrid URLRequestContext creation path, where the embedder starts
> > configuring a URLRequestContextBuilder, and the NetworkContext finishes
> > configuring it.
> >
> > The reduces redundant code, lets us test the new path in production
> > before we ship the network service, and will allow Isolated App
> > URLRequestContexts to need less code when they're updated to use the
> > hybrid configuration path (Which I plan to do in a followup CL).
> >
> > The NetworkContext constructor that takes an already-initialized
> > URLRequestContext is not modified in this CL.
> >
> > Bug:  789644 
> > Cq-Include-Trybots: luci.chromium.try:linux_mojo
> > Change-Id: Ibf40ae6497e7be06b0a666bc664ea943e145cae9
> > Reviewed-on: https://chromium-review.googlesource.com/1152083
> > Reviewed-by: Maks Orlovich <morlovich@chromium.org>
> > Reviewed-by: Clark DuVall <cduvall@chromium.org>
> > Commit-Queue: Matt Menke <mmenke@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#578740}
>
> TBR=mmenke@chromium.org,morlovich@chromium.org,cduvall@chromium.org
>
> Change-Id: Ia9f914fac8b0fa624210363dd55333a0dfa20531
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug:  789644 
> Cq-Include-Trybots: luci.chromium.try:linux_mojo
> Reviewed-on: https://chromium-review.googlesource.com/1153232
> Reviewed-by: Matt Menke <mmenke@chromium.org>
> Commit-Queue: Matt Menke <mmenke@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578808}

TBR=mmenke@chromium.org,morlovich@chromium.org,cduvall@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug:  789644 
Change-Id: I958d52e4d3ef343b88f89f410bd009a166d00ef5
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Reviewed-on: https://chromium-review.googlesource.com/1156664
Reviewed-by: Matt Menke <mmenke@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#579456}
[modify] https://crrev.com/df45365eaf5d9a3474356bc3d18a401645cbdf5c/chrome/browser/io_thread.cc
[modify] https://crrev.com/df45365eaf5d9a3474356bc3d18a401645cbdf5c/chrome/browser/profiles/off_the_record_profile_io_data.cc
[modify] https://crrev.com/df45365eaf5d9a3474356bc3d18a401645cbdf5c/chrome/browser/profiles/profile_impl_io_data.cc
[modify] https://crrev.com/df45365eaf5d9a3474356bc3d18a401645cbdf5c/services/network/network_context.cc
[modify] https://crrev.com/df45365eaf5d9a3474356bc3d18a401645cbdf5c/services/network/network_context.h
[modify] https://crrev.com/df45365eaf5d9a3474356bc3d18a401645cbdf5c/services/network/network_context_unittest.cc
[modify] https://crrev.com/df45365eaf5d9a3474356bc3d18a401645cbdf5c/services/network/network_service.cc
[modify] https://crrev.com/df45365eaf5d9a3474356bc3d18a401645cbdf5c/services/network/network_service.h

Sign in to add a comment