DevTools: Fix Network.clearBrowserCookies

chrome/browser/chrome_content_browser_client.cc

Index: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index c43a1e4b6182f7720b8fec9776d1b9f66d56cb20..022115c42080600014e4512bba71fed5552668e0 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -2742,16 +2742,6 @@ void ChromeContentBrowserClient::ClearCache(RenderFrameHost* rfh) {
                   BrowsingDataRemover::ORIGIN_TYPE_UNPROTECTED_WEB);
 }
 
-void ChromeContentBrowserClient::ClearCookies(RenderFrameHost* rfh) {
-  Profile* profile = Profile::FromBrowserContext(
-      rfh->GetSiteInstance()->GetProcess()->GetBrowserContext());
-  BrowsingDataRemover* remover =
-      BrowsingDataRemoverFactory::GetForBrowserContext(profile);
-  int remove_mask = ChromeBrowsingDataRemoverDelegate::DATA_TYPE_SITE_DATA;

[dgozman, 2017/03/31 22:57:55]

This looks like more than cookies, doesn't it? Let's double check we do
equivalent things.

[phulce, 2017/03/31 23:54:32]

Oh yeah forgot to mention this was clearly a bug too :) When you clear cookies
currently in Network panel context menu, it deletes all localStorage/cache/etc

-  remover->Remove(base::Time(), base::Time::Max(), remove_mask,
-                  BrowsingDataRemover::ORIGIN_TYPE_UNPROTECTED_WEB);
-}
-
 void ChromeContentBrowserClient::ClearSiteData(
     content::BrowserContext* browser_context,
     const url::Origin& origin,