Remove the Extensions URLRequestContext

android_webview/browser/net/aw_url_request_context_getter.cc

Index: android_webview/browser/net/aw_url_request_context_getter.cc
diff --git a/android_webview/browser/net/aw_url_request_context_getter.cc b/android_webview/browser/net/aw_url_request_context_getter.cc
index eed948f0b17e84f6ad2ebf261ac7bf9486b93985..ac6441f6ae71167d128ab6e91909b44fb08d2176 100644
--- a/android_webview/browser/net/aw_url_request_context_getter.cc
+++ b/android_webview/browser/net/aw_url_request_context_getter.cc
@@ -18,6 +18,7 @@
 #include "content/public/browser/content_browser_client.h"
 #include "content/public/browser/cookie_store_factory.h"
 #include "content/public/common/content_client.h"
+#include "content/public/common/content_constants.h"
 #include "content/public/common/url_constants.h"
 #include "net/base/cache_type.h"
 #include "net/cookies/cookie_store.h"
@@ -54,12 +55,13 @@ AwURLRequestContextGetter::~AwURLRequestContextGetter() {
 void AwURLRequestContextGetter::Init() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
-  cookie_store_ = content::CreatePersistentCookieStore(
-      browser_context_->GetPath().Append(FILE_PATH_LITERAL("Cookies")),
-      true,
-      NULL,
-      NULL);
-  cookie_store_->GetCookieMonster()->SetPersistSessionCookies(true);
+  // TODO(ajwong): Do we never drop session cookies?

[joth, 2013/08/16 17:45:05]

We don't - it's up to the embedder to do this via
http://developer.android.com/reference/android/webkit/CookieManager.html#remo...
call.

Maybe add a comment // by default session cookies are always restored. An
application can control the policy e.g. by calling
CookieManager.removeSessionCookie() when Activity.onCreate() is called with
savedInstanceState == null.

[awong, 2013/08/17 00:32:52]

Done.

+  cookie_store_ = content::CreateCookieStore(
+      content::CookieStoreConfig(
+          browser_context_->GetPath().Append(content::kCookieFilename),
+          content::CookieStoreConfig::RESTORED_SESSION_COOKIES,
+          NULL,
+          NULL));
 
   // The CookieMonster must be passed here so it happens synchronously to
   // the main thread initialization (to avoid race condition in another