Remove FontSizePrefs destroy() method as its a singleton class.

chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java

Index: chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java b/chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java
index 6878cfe7225c41c8d24bc595660bc05f1ed48963..d4d532af327be3dfec1263bc3d31d15645416290 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java
@@ -173,6 +173,7 @@ public class FontSizePrefs implements OnSharedPreferenceChangeListener {
     public void destroy() {
         mSharedPreferences.unregisterOnSharedPreferenceChangeListener(this);
         nativeDestroy(mFontSizePrefsAndroidPtr);
+        mObserverMap.clear();

[Ted C, 2014/09/15 18:42:37]

destroy seems like an odd API to have on a singleton.  The singleton would now
be in an invalid state and can never be reused.  That isn't a comment about this
change per say, but it's just general confusion about this.

Now for this, are the observers being cleaned up as part of the native destroy
call?  To me, it looks like this could end up leaking the native observer
references.  If that is the case, we should be iterating over the map and
calling destroy on them to make sure that doesn't happen.

Also, I can't find any references to this method and I suspect it is never
actually called (nor does it really need to as destroy would only come when
android is about to destroy our application and kill the process).

[nyquist, 2014/09/15 19:56:50]

Yes, .clear() would be wrong. I think it would be better to just remove this
method alltogether.

[wajahat, 2014/09/16 09:30:36]

As this is a singleton class and there are no references to destroy() hence
removing this method altogether.

On 2014/09/15 18:42:37, Ted C wrote:

[wajahat, 2014/09/16 09:30:36]

Done.

     }
 
     private native void nativeAddObserver(long nativeFontSizePrefsAndroid,