Declaring the weak_ptr_factory in proper order.

ash/wm/workspace/workspace_window_resizer.h

 // Copyright (c) 2012 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 ASH_WM_WORKSPACE_WINDOW_RESIZER_H_
 #define ASH_WM_WORKSPACE_WINDOW_RESIZER_H_
 
 #include <vector>
 
 #include "ash/wm/window_resizer.h"
@@ skipping 199 matching lines @@
   // Used to verify |magnetism_window_| is still valid.
   aura::WindowTracker window_tracker_;
 
   // If |magnetism_window_| is non-NULL this indicates how the two windows
   // should attach.
   MatchedEdge magnetism_edge_;
 
   // Dock container window layout manager.
   DockedWindowLayoutManager* dock_layout_;
 
+  // Current instance for use by the WorkspaceWindowResizerTest.
+  static WorkspaceWindowResizer* instance_;

[oshima, 2014/08/26 15:31:05]

can you move this to anonymous namespace in .cc ?

[Anand Ratn (left samsung), 2014/08/28 06:15:07]

Done.

+
   // Used to determine if this has been deleted during a drag such as when a tab
   // gets dragged into another browser window.
   base::WeakPtrFactory<WorkspaceWindowResizer> weak_ptr_factory_;
 
-  // Current instance for use by the WorkspaceWindowResizerTest.
-  static WorkspaceWindowResizer* instance_;
-
   DISALLOW_COPY_AND_ASSIGN(WorkspaceWindowResizer);
 };
 
 }  // namespace ash
 
 #endif  // ASH_WM_WORKSPACE_WINDOW_RESIZER_H_