Closing a guest session clears all browsing history.

chrome/test/base/testing_profile.cc

Index: chrome/test/base/testing_profile.cc
diff --git a/chrome/test/base/testing_profile.cc b/chrome/test/base/testing_profile.cc
index e2870aa1c0a23964ec95bf4248c4182f1eceb261..5a76e5606a9f04ebb2cc4bf4014e41d86c2a2007 100644
--- a/chrome/test/base/testing_profile.cc
+++ b/chrome/test/base/testing_profile.cc
@@ -845,7 +845,9 @@ net::URLRequestContextGetter* TestingProfile::GetRequestContextForRenderProcess(
 }
 
 net::URLRequestContextGetter* TestingProfile::GetMediaRequestContext() {
-  return NULL;
+  // We don't test the media context here yet, so returning the same dummy
+  // context is sufficient for now.
+  return GetRequestContext();

[Peter Kasting, 2015/02/10 21:43:36]

Does this change really belong in this CL?

[Mike Lerman, 2015/02/10 21:53:32]

If I don't add it here then the
BrowserCommandControllerTest.NewAvatarMenuEnabledInGuestMode test will fail, as
it exercises the new code path, and the BrowsingDataRemover runs on a
TestingProfile, and will fail without this URLRequestContextGetter. So Yes :)

 }
 
 net::URLRequestContextGetter*