Don't call WebCacheManager::Remove(RPH_id) to clear webview's cache.

Don't call WebCacheManager::Remove(RPH_id) to clear webview's cache.

We already call WebCacheManager::ClearCacheForProcess(RPH_id), which
seems to be sufficient in my local testing.
I remember that clearing webview cache didn't seem to work without calling
WCM::Remove() when I implemented it.

After r380598, Calling WCM::Remove also deletes mojo service associated
with the RPH, so the next call to WCM::Remove with the same RPH will
result in a nullptr access.

BUG=615429
Test=Added a test, see http://crbug.com/615429#c2 for repro.

Committed: https://crrev.com/d0dae71f785f35d100d1454e3a7331a896bdf2f2
Cr-Commit-Position: refs/heads/master@{#397803}

[lazyboy, 2016-06-03 01:18:57]

lazyboy@chromium.org changed reviewers:
+ lfg@chromium.org

[lfg, 2016-06-03 18:26:53]

lgtm

[lazyboy, 2016-06-03 21:13:57]

The CQ bit was checked by lazyboy@chromium.org

[commit-bot: I haz the power, 2016-06-03 21:14:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2033993002/1

[commit-bot: I haz the power, 2016-06-03 21:20:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-03 21:21:35]

Description was changed from

==========
Don't call WebCacheManager::Remove(RPH_id) to clear webview's cache.

We already call WebCacheManager::ClearCacheForProcess(RPH_id), which
seems to be sufficient in my local testing.
I remember that clearing webview cache didn't seem to work without calling
WCM::Remove() when I implemented it.

After r380598, Calling WCM::Remove also deletes mojo service associated
with the RPH, so the next call to WCM::Remove with the same RPH will
result in a nullptr access.

BUG=615429
Test=Added a test, see http://crbug.com/615429#c2 for repro.
==========

to

==========
Don't call WebCacheManager::Remove(RPH_id) to clear webview's cache.

We already call WebCacheManager::ClearCacheForProcess(RPH_id), which
seems to be sufficient in my local testing.
I remember that clearing webview cache didn't seem to work without calling
WCM::Remove() when I implemented it.

After r380598, Calling WCM::Remove also deletes mojo service associated
with the RPH, so the next call to WCM::Remove with the same RPH will
result in a nullptr access.

BUG=615429
Test=Added a test, see http://crbug.com/615429#c2 for repro.

Committed: https://crrev.com/d0dae71f785f35d100d1454e3a7331a896bdf2f2
Cr-Commit-Position: refs/heads/master@{#397803}
==========

[commit-bot: I haz the power, 2016-06-03 21:21:36]

Patchset 1 (id:??) landed as
https://crrev.com/d0dae71f785f35d100d1454e3a7331a896bdf2f2
Cr-Commit-Position: refs/heads/master@{#397803}