Store a stacktrace of PrefService destruction in PrefChangeRegistrar

base/prefs/pref_change_registrar.h

Index: base/prefs/pref_change_registrar.h
diff --git a/base/prefs/pref_change_registrar.h b/base/prefs/pref_change_registrar.h
index a914bea7d48cec7c4cf2149d7525c54dc10a76b2..f7c40c5392823e2e336f46a72086337c26a4a076 100644
--- a/base/prefs/pref_change_registrar.h
+++ b/base/prefs/pref_change_registrar.h
@@ -66,6 +66,9 @@ class BASE_PREFS_EXPORT PrefChangeRegistrar : public PrefObserver {
   // PrefObserver:
   virtual void OnPreferenceChanged(PrefService* service,
                                    const std::string& pref_name) OVERRIDE;
+  // TODO(battre): Remove function (debugging tool for crbug.com/373435).

[engedy, 2014/05/16 15:19:58]

nit: I am not sure saying "debugging tool" is too informative. The same way you
are doing it in some other places, could we just consistently say everywhere:
 
// TODO(battre): Remove function. See: crbug.com/373435.

And add some details to the bug itself. Anyone interested can then look up the
bug for a detailed explanation.

[battre, 2014/05/16 15:58:37]

Done.

+  virtual void SetPrefServiceDestructionTrace(
+      const std::string& stacktrace) OVERRIDE;
 
   static void InvokeUnnamedCallback(const base::Closure& callback,
                                     const std::string& pref_name);
@@ -74,6 +77,8 @@ class BASE_PREFS_EXPORT PrefChangeRegistrar : public PrefObserver {
 
   ObserverMap observers_;
   PrefService* service_;
+  // TODO(battre): Remove attribute (debugging tool for crbug.com/373435).
+  std::string pref_service_destruction_;
 
   DISALLOW_COPY_AND_ASSIGN(PrefChangeRegistrar);
 };