 Chromium Code Reviews
 Chromium Code Reviews Issue 6452011:
  Rework tree APIs to reflect Google style and more const-correctness....  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src/
    
  
    Issue 6452011:
  Rework tree APIs to reflect Google style and more const-correctness....  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src/| Index: views/layout/grid_layout_unittest.cc | 
| =================================================================== | 
| --- views/layout/grid_layout_unittest.cc (revision 74117) | 
| +++ views/layout/grid_layout_unittest.cc (working copy) | 
| @@ -62,9 +62,8 @@ | 
| } | 
| virtual void RemoveAll() { | 
| - for (int i = host.GetChildViewCount() - 1; i >= 0; i--) { | 
| + for (size_t i = host.child_count() - 1; i >= 0; i--) | 
| 
sky
2011/02/08 19:11:54
THis never stops.
PS I don't like size_t
 | 
| host.RemoveChildView(host.GetChildViewAt(i)); | 
| - } | 
| } | 
| void GetPreferredSize() { | 
| @@ -92,9 +91,8 @@ | 
| } | 
| virtual void RemoveAll() { | 
| - for (int i = host.GetChildViewCount() - 1; i >= 0; i--) { | 
| + for (size_t i = host.child_count() - 1; i >= 0; i--) | 
| 
sky
2011/02/08 19:11:54
This never stops.
PS I don't like size_t
 | 
| host.RemoveChildView(host.GetChildViewAt(i)); | 
| - } | 
| } | 
| void TestAlignment(GridLayout::Alignment alignment, gfx::Rect* bounds) { |