Remove PrefObserver usage, batch 3.

chrome/browser/api/prefs/pref_member.h

Index: chrome/browser/api/prefs/pref_member.h
diff --git a/chrome/browser/api/prefs/pref_member.h b/chrome/browser/api/prefs/pref_member.h
index 5f3791cf19659585c7663c202e26f5d8ad278596..da249e4bf814e1fa1d04256b7cae82a13348b0cd 100644
--- a/chrome/browser/api/prefs/pref_member.h
+++ b/chrome/browser/api/prefs/pref_member.h
@@ -47,6 +47,10 @@ class PrefMemberBase : public PrefObserver {
   // the pref that is changing.
   typedef base::Callback<void(const std::string&)> NamedChangeCallback;
 
+  const std::string& pref_name() const { return pref_name_; }

[Mattias Nissler (ping if slow), 2012/11/13 15:47:03]

Note that there already is a public GetPrefName in PrefMember below, I think we
only want one.

[Jói, 2012/11/13 16:28:24]

Right you are, sorry I didn't notice.  Moved this back.

+  PrefServiceBase* prefs() { return prefs_; }
+  const PrefServiceBase* prefs() const { return prefs_; }
+
  protected:
   class Internal : public base::RefCountedThreadSafe<Internal> {
    public:
@@ -124,10 +128,6 @@ class PrefMemberBase : public PrefObserver {
   // it hasn't been loaded yet.
   void VerifyPref() const;
 
-  const std::string& pref_name() const { return pref_name_; }
-  PrefServiceBase* prefs() { return prefs_; }
-  const PrefServiceBase* prefs() const { return prefs_; }
-
   virtual Internal* internal() const = 0;
 
   // Used to allow registering plain base::Closure callbacks.