Reland r50296 which removes some uses of CookieMonster on the UI thread.

chrome/browser/extensions/extension_cookies_helpers.h

Index: chrome/browser/extensions/extension_cookies_helpers.h
diff --git a/chrome/browser/extensions/extension_cookies_helpers.h b/chrome/browser/extensions/extension_cookies_helpers.h
index 889a8c08e437722d41f61d9c64ee943f605d360f..77284c6ffc56413e9b2d8da3a342979962495549 100644
--- a/chrome/browser/extensions/extension_cookies_helpers.h
+++ b/chrome/browser/extensions/extension_cookies_helpers.h
@@ -46,6 +46,7 @@ DictionaryValue* CreateCookieStoreValue(Profile* profile,
 
 // Retrieves all cookies from the given cookie store corresponding to the given
 // URL. If the URL is empty, all cookies in the cookie store are retrieved.
+// This can only be called on the IO thread.
 net::CookieMonster::CookieList GetCookieListFromStore(
     net::CookieStore* cookie_store, const GURL& url);
 
@@ -60,7 +61,8 @@ GURL GetURLFromCookiePair(
 // match list all the cookies that both match the given URL and cookie details
 // and are allowed by extension host permissions.
 void AppendMatchingCookiesToList(
-    net::CookieStore* cookie_store, const std::string& store_id,
+    const net::CookieMonster::CookieList& all_cookies,
+    const std::string& store_id,
     const GURL& url, const DictionaryValue* details,
     const Extension* extension,
     ListValue* match_list);