Wires up sorting of bookmarks to the 'organize menu' in the bookmark...

chrome/browser/bookmarks/bookmark_table_model_unittest.cc

 // Copyright (c) 2006-2008 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.
 
 #include "base/string_util.h"
 #include "base/time.h"
 #include "chrome/browser/bookmarks/bookmark_table_model.h"
 #include "chrome/test/testing_profile.h"
 #include "grit/generated_resources.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 125 matching lines @@
   VerifyAndClearOberserverCounts(0, 0, 1, 0);
   ASSERT_EQ(4, model_->RowCount());
   EXPECT_TRUE(new_node == model_->GetNodeForRow(0));
 
   // Add to the bookmark bar, this shouldn't generate an event.
   bookmark_model()->AddURL(bookmark_model()->GetBookmarkBarNode(), 0, L"new",
                            url1_);
   VerifyAndClearOberserverCounts(0, 0, 0, 0);
 }
 
+// Verifies sort sends out notification and results in a sort.
+TEST_F(BookmarkTableModelTest, SortFolder) {
+  BookmarkNode* other = bookmark_model()->other_node();
+  SetModel(BookmarkTableModel::CreateBookmarkTableModelForFolder(
+           bookmark_model(), other));
+  ASSERT_EQ(3, model_->RowCount());
+  bookmark_model()->SortChildren(other);
+
+  // Sorting should trigger change notification.
+  VerifyAndClearOberserverCounts(1, 0, 0, 0);
+
+  // Make sure things reordered.
+  EXPECT_TRUE(other->GetChild(0) == model_->GetNodeForRow(0));
+  EXPECT_TRUE(other->GetChild(1) == model_->GetNodeForRow(1));
+  EXPECT_TRUE(other->GetChild(2) == model_->GetNodeForRow(2));
+}
+
 // Verifies removing an item from folder model generates the correct event.
 TEST_F(BookmarkTableModelTest, RemoveFromFolder) {
   BookmarkNode* other = bookmark_model()->other_node();
   SetModel(BookmarkTableModel::CreateBookmarkTableModelForFolder(
       bookmark_model(), other));
   bookmark_model()->Remove(other, 0);
 
   // Should have gotten notification of the remove.
   VerifyAndClearOberserverCounts(0, 0, 0, 1);
   EXPECT_EQ(2, model_->RowCount());
@@ skipping 133 matching lines @@
   // Change a folder, this shouldn't change the model.
   bookmark_model()->SetTitle(bookmark_model()->other_node()->GetChild(1),
                              L"new");
   VerifyAndClearOberserverCounts(0, 0, 0, 0);
 
   // Change a url that isn't in the model, this shouldn't send change.
   bookmark_model()->SetTitle(bookmark_model()->other_node()->GetChild(0),
                              L"new");
   VerifyAndClearOberserverCounts(0, 0, 0, 0);
 }