Implement Android WebView.clearCache

Implement part of Android WebView.clearCache

For velocity, not refactoring BrowsingDataRemover code into content/.
This method is similar to BrowsingDataRemover::REMOVE_CACHE mask. Still need to implement clearing http disk cache.

BUG=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=159827

[boliu, 2012-10-02 21:19:10]

Work in progress.

#1 in downstream bug done. Still need to clear disk cache and add test. And
clean up.

[joth, 2012-10-02 21:36:38]

lg with changes you already mention. Seems OK to land this as first step and do
the disk caches in another CL.

copied in jam@ just as an FYI / in case you have other input on whether content
API should provide functions for WebKit cache clearing?
(c.f. duplication of WebCache::clear() calls in this CL vs
https://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/chrome_rende...)

IMO - our requirements are driven by sufficiently different factors (long term
api compat vs evolving browser product evolution) so I think it's fine to have
this sort of level of "duplication"

https://codereview.chromium.org/11051003/diff/1/android_webview/common/render...
File android_webview/common/render_view_messages.h (left):

https://codereview.chromium.org/11051003/diff/1/android_webview/common/render...
android_webview/common/render_view_messages.h:37: // These are messages sent
from the renderer to the browser process.
why u remove me?

https://codereview.chromium.org/11051003/diff/1/android_webview/renderer/aw_r...
File android_webview/renderer/aw_render_process_observer.h (right):

https://codereview.chromium.org/11051003/diff/1/android_webview/renderer/aw_r...
android_webview/renderer/aw_render_process_observer.h:14: class
AwRenderProcessObserver : public content::RenderProcessObserver {
class comment? something about handling render-process wide IPCs

[boliu, 2012-10-02 22:06:59]

Prepare for submitting incomplete implementation. Added some TODOs. Left
incomplete test there since it shows the code path runs without crashing.

https://codereview.chromium.org/11051003/diff/1/android_webview/common/render...
File android_webview/common/render_view_messages.h (left):

https://codereview.chromium.org/11051003/diff/1/android_webview/common/render...
android_webview/common/render_view_messages.h:37: // These are messages sent
from the renderer to the browser process.
On 2012/10/02 21:36:38, joth wrote:
> why u remove me?

Doh! misread, thought the comment was the same as above.

Added back.

https://codereview.chromium.org/11051003/diff/1/android_webview/renderer/aw_r...
File android_webview/renderer/aw_render_process_observer.h (right):

https://codereview.chromium.org/11051003/diff/1/android_webview/renderer/aw_r...
android_webview/renderer/aw_render_process_observer.h:14: class
AwRenderProcessObserver : public content::RenderProcessObserver {
On 2012/10/02 21:36:38, joth wrote:
> class comment? something about handling render-process wide IPCs

Done.

[joth, 2012-10-02 23:01:52]

lgtm

https://codereview.chromium.org/11051003/diff/5001/android_webview/renderer/a...
File android_webview/renderer/aw_render_process_observer.h (right):

https://codereview.chromium.org/11051003/diff/5001/android_webview/renderer/a...
android_webview/renderer/aw_render_process_observer.h:14: // An
RenderProcessObserver implementation used for handling android_webview
nit: An -> A

[commit-bot: I haz the power, 2012-10-02 23:07:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/boliu@chromium.org/11051003/11004

[commit-bot: I haz the power, 2012-10-03 01:44:48]

Change committed as 159827