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

Issue 687082 link

Starred by 3 users
Status: Fixed
Owner:
lfg@chromium.org
Closed: Jun 2017
Cc:
█ dskaram@chromium.org
lfg@chromium.org
wjmaclean@chromium.org
mcnee@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Feature



Sign in to add a comment

Support removal of webview session cookies

Project Member Reported by dskaram@chromium.org, Jan 31 2017 
Today session cookies are not cleared from memory when the webview process is killed. They are still alive even if the app process is killed. The only two ways to clear session cookies are:
    a. Manually invoke Webview’s clearData() with cookie flag set
    b. Reboot chromebook

Doing (a) however clears both in memory as well persistent cookies which is not the desired outcome.


Partners need a way to clear *only* the session cookies leaving behind persistent cookies. As per documentation there is no special flag to clear only the session related cookies.

Comment 1 by lfg@chromium.org, Jan 31 2017

Components: Platform>Apps>BrowserTag
Labels: M-58 OS-Linux OS-Mac OS-Windows
Targeting M58.
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 16 2017 

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

commit 66cdb5cf76f67ac76c426998dece842e7b739209
Author: lfg <lfg@chromium.org>
Date: Thu Feb 16 22:25:50 2017

Support the removal of only session cookies or persistent cookies
through the <webview> cleardata API.

BUG= 687082 

Review-Url: https://codereview.chromium.org/2700473003
Cr-Commit-Position: refs/heads/master@{#451119}

[modify] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/chrome/browser/apps/guest_view/web_view_browsertest.cc
[modify] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/chrome/common/extensions/api/webview_tag.json
[add] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/chrome/test/data/extensions/platform_apps/web_view/common/cleardata_persistent/bootstrap.js
[add] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/chrome/test/data/extensions/platform_apps/web_view/common/cleardata_persistent/guest.html
[add] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/chrome/test/data/extensions/platform_apps/web_view/common/cleardata_session/bootstrap.js
[add] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/chrome/test/data/extensions/platform_apps/web_view/common/cleardata_session/guest.html
[modify] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/extensions/browser/api/guest_view/web_view/web_view_internal_api.cc
[modify] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/extensions/browser/guest_view/web_view/web_view_constants.cc
[modify] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/extensions/browser/guest_view/web_view/web_view_constants.h
[modify] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/extensions/browser/guest_view/web_view/web_view_guest.cc
[modify] https://crrev.com/66cdb5cf76f67ac76c426998dece842e7b739209/extensions/common/api/web_view_internal.json

Comment 3 by dskaram@chromium.org, Feb 17 2017 

Will take a few days for Chrome changes to merge into Chrome OS canary/dev. Will update the bug once changes hit Chrome OS (right now they're only on Chrome).

Comment 4 by lfg@chromium.org, Jun 5 2017

Status: Fixed (was: Assigned)
Closing it as fixed, since this has now reached the stable channel.

Comment 5 by akhashn...@gmail.com, Jun 9 2017 

Verified. Used sessionCookies and persistentCookie attributes and works as expected. Tested on Stable 58 and Beta 59.
►

Sign in to add a comment