Fix bug in browser action overflow container

chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.cc

 // 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.
 
 #include "chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.h"
 
 #include "base/message_loop/message_loop.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_action.h"
 #include "chrome/browser/extensions/extension_context_menu_model.h"
@@ skipping 221 matching lines @@
       extensions::ExtensionRegistry::Get(browser_->profile())->
           enabled_extensions().GetByID(drop_data.id());
   extensions::ExtensionToolbarModel* toolbar_model =
       extensions::ExtensionToolbarModel::Get(browser_->profile());
   if (browser_->profile()->IsOffTheRecord())
     drop_index = toolbar_model->IncognitoIndexToOriginal(drop_index);
   toolbar_model->MoveExtensionIcon(extension, drop_index);
 
   // If the extension was moved to the overflow menu from the main bar, notify
   // the owner.
-  if (drop_index >= owner_->VisibleBrowserActions())
+  if (drop_data.index() < owner_->VisibleBrowserActions())
     owner_->NotifyActionMovedToOverflow();
 
   if (for_drop_)
     delete this;
   return ui::DragDropTypes::DRAG_MOVE;
 }
 
 bool BrowserActionOverflowMenuController::CanDrag(views::MenuItemView* menu) {
   return true;
 }
@@ skipping 10 matching lines @@
     views::MenuItemView* sender) {
   return ui::DragDropTypes::DRAG_MOVE;
 }
 
 size_t BrowserActionOverflowMenuController::IndexForId(int id) const {
   // The index of the view being dragged (GetCommand gives a 1-based index into
   // the overflow menu).
   DCHECK_GT(owner_->VisibleBrowserActions() + id, 0u);
   return owner_->VisibleBrowserActions() + id - 1;
 }