Fix WeakPtrFactory member placement

chrome/browser/android/most_visited_sites.h

Index: chrome/browser/android/most_visited_sites.h
diff --git a/chrome/browser/android/most_visited_sites.h b/chrome/browser/android/most_visited_sites.h
index 6609846709ecd1798084ea3795525e1fd000880c..8ea74f60aab386969f038e273dc430889752e583 100644
--- a/chrome/browser/android/most_visited_sites.h
+++ b/chrome/browser/android/most_visited_sites.h
@@ -56,6 +56,13 @@ class MostVisitedSites : public ProfileSyncServiceObserver,
   static bool Register(JNIEnv* env);
 
  private:
+  // The source of the Most Visited sites.
+  enum MostVisitedSource {
+    TOP_SITES,
+    SUGGESTIONS_SERVICE
+  };
+  MostVisitedSource mv_source_;

[Bernhard Bauer, 2014/09/30 13:04:16]

Sorry, this should stay in its original place. According to the style guide,
type declarations come before method declarations, which come before members.
That means that we'll have to separate these.

+
   virtual ~MostVisitedSites();
   void QueryMostVisitedURLs();
 
@@ -119,17 +126,10 @@ class MostVisitedSites : public ProfileSyncServiceObserver,
   // Copy of the server suggestions (if enabled). Used for logging.
   suggestions::SuggestionsProfile server_suggestions_;
 
-  // For callbacks may be run after destruction.
-  base::WeakPtrFactory<MostVisitedSites> weak_ptr_factory_;
-
   content::NotificationRegistrar registrar_;
 
-  // The source of the Most Visited sites.
-  enum MostVisitedSource {
-    TOP_SITES,
-    SUGGESTIONS_SERVICE
-  };
-  MostVisitedSource mv_source_;
+  // For callbacks may be run after destruction.
+  base::WeakPtrFactory<MostVisitedSites> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(MostVisitedSites);
 };