Move clear cache code from chrome/ (ChromeWVGDelegate) to extensions/ (WVGuest)

extensions/browser/guest_view/web_view/web_view_guest.cc

Index: extensions/browser/guest_view/web_view/web_view_guest.cc
diff --git a/extensions/browser/guest_view/web_view/web_view_guest.cc b/extensions/browser/guest_view/web_view/web_view_guest.cc
index f29e12f468018ec1b3693b94719a200c51325b6b..721cb11cac8d86967d8c61f5e73048f012723e6e 100644
--- a/extensions/browser/guest_view/web_view/web_view_guest.cc
+++ b/extensions/browser/guest_view/web_view/web_view_guest.cc
@@ -7,6 +7,8 @@
 #include "base/message_loop/message_loop.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
+#include "components/browsing_data/storage_partition_http_cache_data_remover.h"
+#include "components/web_cache/browser/web_cache_manager.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/child_process_security_policy.h"
@@ -705,15 +707,25 @@ bool WebViewGuest::ClearData(base::Time remove_since,
     return false;
 
   if (removal_mask & webview::WEB_VIEW_REMOVE_DATA_MASK_CACHE) {
-    if (web_view_guest_delegate_) {
-      // First clear http cache data and then clear the rest in
-      // |ClearDataInternal|.
-      web_view_guest_delegate_->ClearCache(
-          remove_since, base::Bind(&WebViewGuest::ClearDataInternal,
-                                   weak_ptr_factory_.GetWeakPtr(), remove_since,
-                                   removal_mask, callback));
-      return true;
-    }
+    // First clear http cache data and then clear the rest in
+    // |ClearDataInternal|.
+    int render_process_id = web_contents()->GetRenderProcessHost()->GetID();
+    // We need to clear renderer cache separately for our process because
+    // StoragePartitionHttpCacheDataRemover::ClearData() does not clear that.
+    web_cache::WebCacheManager::GetInstance()->Remove(render_process_id);
+    web_cache::WebCacheManager::GetInstance()->ClearCacheForProcess(
+        render_process_id);
+
+    base::Closure cache_removal_done_callback = base::Bind(
+        &WebViewGuest::ClearDataInternal, weak_ptr_factory_.GetWeakPtr(),
+        remove_since, removal_mask, callback);
+    // StoragePartitionHttpCacheDataRemover removes itself when it is done.
+    // components/, move |ClearCache| to WebViewGuest: http//crbug.com/471287.
+    browsing_data::StoragePartitionHttpCacheDataRemover::CreateForRange(
+        partition, remove_since, base::Time::Now())
+        ->Remove(cache_removal_done_callback);
+
+    return true;

[Fady Samuel, 2015/04/26 15:06:07]

I just noticed this early exit. What happens if you want to clear other pieces
of data?

[lazyboy, 2015/04/26 15:16:51]

ClearDataInternal above in |cache_removal_done_callback| will do that.

   }
 
   ClearDataInternal(remove_since, removal_mask, callback);