Make it clear that WeakPtrFactory is the last data member

sync/internal_api/public/attachments/attachment_service_proxy_for_test.h

 // Copyright 2014 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 SYNC_INTERNAL_API_PUBLIC_ATTACHMENTS_ATTACHMENT_SERVICE_PROXY_FOR_TEST_H_
 #define SYNC_INTERNAL_API_PUBLIC_ATTACHMENTS_ATTACHMENT_SERVICE_PROXY_FOR_TEST_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "sync/base/sync_export.h"
@@ skipping 14 matching lines @@
 
  private:
   // A Core that owns the wrapped AttachmentService.
   class OwningCore : public AttachmentServiceProxy::Core {
    public:
     OwningCore(
         scoped_ptr<AttachmentService>,
         scoped_ptr<base::WeakPtrFactory<AttachmentService> > weak_ptr_factory);
 
    private:
+    virtual ~OwningCore();
+
     scoped_ptr<AttachmentService> wrapped_;
     // WeakPtrFactory for wrapped_.  See Create() for why this is a scoped_ptr.
     scoped_ptr<base::WeakPtrFactory<AttachmentService> > weak_ptr_factory_;
 
-    virtual ~OwningCore();
-
     DISALLOW_COPY_AND_ASSIGN(OwningCore);
   };
 
   AttachmentServiceProxyForTest(
       const scoped_refptr<base::SequencedTaskRunner>& wrapped_task_runner,
       const scoped_refptr<Core>& core);
 };
 
 }  // namespace syncer
 
 #endif  // SYNC_INTERNAL_API_PUBLIC_ATTACHMENTS_ATTACHMENT_SERVICE_PROXY_FOR_TEST_H_