Fixes possible use after free in SessionService

chrome/browser/sessions/base_session_service.h

Index: chrome/browser/sessions/base_session_service.h
diff --git a/chrome/browser/sessions/base_session_service.h b/chrome/browser/sessions/base_session_service.h
index e85799a66e6b2ab7cbccca2d7589294169b8889f..6d259f0d46dce2c95f63aeda1d5eb9e57f6a05d1 100644
--- a/chrome/browser/sessions/base_session_service.h
+++ b/chrome/browser/sessions/base_session_service.h
@@ -156,7 +156,7 @@ class BaseSessionService {
 
   // This posts the task to the SequencedWorkerPool, or run immediately
   // if the SequencedWorkerPool has been shutdown.
-  bool RunTaskOnBackendThread(const tracked_objects::Location& from_here,
+  void RunTaskOnBackendThread(const tracked_objects::Location& from_here,
                               const base::Closure& task);
 
   // Max number of navigation entries in each direction we'll persist.
@@ -164,6 +164,7 @@ class BaseSessionService {
 
  private:
   friend class BetterSessionRestoreCrashTest;
+  friend class SessionServiceTestHelper;

[marja, 2014/08/26 14:48:09]

Afaics this is not needed.

 
   // The profile. This may be null during testing.
   Profile* profile_;