Fix WeakPtrFactory member placement

chrome/browser/android/most_visited_sites.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_ANDROID_MOST_VISITED_SITES_H_
 #define CHROME_BROWSER_ANDROID_MOST_VISITED_SITES_H_
 
 #include <jni.h>
 
 #include "base/android/scoped_java_ref.h"
@@ skipping 38 matching lines @@
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   // ProfileSyncServiceObserver implementation.
   virtual void OnStateChanged() OVERRIDE;
 
   // Registers JNI methods.
   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();
 
   // Initialize the query to Top Sites. Called if the SuggestionsService is not
   // enabled, or if it returns no data.
   void InitiateTopSitesQuery();
 
   // Callback for when data is available from TopSites.
   void OnMostVisitedURLsAvailable(
       base::android::ScopedJavaGlobalRef<jobject>* j_observer,
@@ skipping 43 matching lines @@
   int num_local_thumbs_;
   // Number of tiles for which a server thumbnail is provided.
   int num_server_thumbs_;
   // Number of tiles for which no thumbnail is found/specified and a gray tile
   // is used as the main tile.
   int num_empty_thumbs_;
 
   // Copy of the server suggestions (if enabled). Used for logging.
   suggestions::SuggestionsProfile server_suggestions_;
 
+  content::NotificationRegistrar registrar_;
+
   // 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_;
-
   DISALLOW_COPY_AND_ASSIGN(MostVisitedSites);
 };
 
 #endif  // CHROME_BROWSER_ANDROID_MOST_VISITED_SITES_H_