diff options
| author | kristianm@chromium.org <kristianm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-08 02:09:45 +0000 |
|---|---|---|
| committer | kristianm@chromium.org <kristianm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-03-08 02:09:45 +0000 |
| commit | 9a086d65b577f62a5c296e710c76a581f606b06b (patch) | |
| tree | 41826e2444488c85b69b2d34c4ee002888e0b0eb /android_webview/native | |
| parent | dc37d385a542ec3f33ce4f4537936f8445b16c0c (diff) | |
| download | chromium_src-9a086d65b577f62a5c296e710c76a581f606b06b.zip chromium_src-9a086d65b577f62a5c296e710c76a581f606b06b.tar.gz chromium_src-9a086d65b577f62a5c296e710c76a581f606b06b.tar.bz2 | |
Added todo's in cookiemanager for possible fixes
Actually fixing them is a low priority since it is an uncommon
usage.
BUG=7162846
Review URL: https://chromiumcodereview.appspot.com/12611010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@186849 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'android_webview/native')
| -rw-r--r-- | android_webview/native/cookie_manager.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/android_webview/native/cookie_manager.cc b/android_webview/native/cookie_manager.cc index 08ebfe3..ab9b92c 100644 --- a/android_webview/native/cookie_manager.cc +++ b/android_webview/native/cookie_manager.cc @@ -230,6 +230,8 @@ void CookieManager::RemoveAllCookie() { base::Unretained(this)), false); } +// TODO(kristianm): Pass a null callback so it will not be invoked +// across threads. void CookieManager::RemoveAllCookieAsyncHelper( base::WaitableEvent* completion) { DCHECK(!completion); @@ -262,6 +264,8 @@ bool CookieManager::HasCookies() { return has_cookies; } +// TODO(kristianm): Simplify this, copying the entire list around +// should not be needed. void CookieManager::HasCookiesAsyncHelper(bool* result, base::WaitableEvent* completion) { cookie_monster_->GetAllCookiesAsync( |