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

chrome/browser/views/bookmark_bar_view.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 "chrome/browser/views/bookmark_bar_view.h"
 
 #include <limits>
 
 #include "base/string_util.h"
 #include "base/base_drag_source.h"
@@ skipping 1311 matching lines @@
   ConfigureButton(node, button);
   gfx::Size new_pref = button->GetPreferredSize();
   if (old_pref.width() != new_pref.width()) {
     Layout();
     SchedulePaint();
   } else if (button->IsVisible()) {
     button->SchedulePaint();
   }
 }
 
+void BookmarkBarView::BookmarkNodeChildrenReordered(BookmarkModel* model,
+                                                    BookmarkNode* node) {

[ncarter (slow), 2009/02/27 21:22:34]

Please confirm that it's OK to not worry about the throbbing state here (because
the bubble disappears when it loses focus?)

[sky, 2009/02/27 21:42:06]

I added a stop, but no start with a description of why.

+  NotifyModelChanged();
+  if (node != model_->GetBookmarkBarNode())
+    return;  // We only care about reordering of the bookmark bar node.
+
+  // Remove the existing buttons.
+  while (GetBookmarkButtonCount()) {
+    views::View* button = GetChildViewAt(0);
+    RemoveChildView(button);
+    MessageLoop::current()->DeleteSoon(FROM_HERE, button);
+  }
+
+  // Create the new buttons.
+  for (int i = 0; i < node->GetChildCount(); ++i)
+    AddChildView(i, CreateBookmarkButton(node->GetChild(i)));
+
+  Layout();
+  SchedulePaint();
+}
+
 void BookmarkBarView::BookmarkNodeFavIconLoaded(BookmarkModel* model,
                                                 BookmarkNode* node) {
   if (menu_runner_.get())
     menu_runner_->FavIconLoaded(node);
   if (drop_menu_runner_.get())
     drop_menu_runner_->FavIconLoaded(node);
   BookmarkNodeChangedImpl(model, node);
 }
 
 void BookmarkBarView::WriteDragData(View* sender,
@@ skipping 518 matching lines @@
 }
 
 void BookmarkBarView::StopThrobbing(bool immediate) {
   if (!throbbing_view_)
     return;
 
   // If not immediate, cycle through 2 more complete cycles.
   throbbing_view_->StartThrobbing(immediate ? 0 : 4);
   throbbing_view_ = NULL;
 }