Commit 5345c6a4 authored by rdevlin.cronin's avatar rdevlin.cronin Committed by Commit bot

Fix bug in browser action overflow container

Fix a bug where reording actions in the browser action overflow container can
cause them all to become visible.

BUG=414177

Review URL: https://codereview.chromium.org/570403002

Cr-Commit-Position: refs/heads/master@{#295108}
parent 0e3b3a6c
...@@ -239,7 +239,7 @@ int BrowserActionOverflowMenuController::OnPerformDrop( ...@@ -239,7 +239,7 @@ int BrowserActionOverflowMenuController::OnPerformDrop(
// If the extension was moved to the overflow menu from the main bar, notify // If the extension was moved to the overflow menu from the main bar, notify
// the owner. // the owner.
if (drop_index >= owner_->VisibleBrowserActions()) if (drop_data.index() < owner_->VisibleBrowserActions())
owner_->NotifyActionMovedToOverflow(); owner_->NotifyActionMovedToOverflow();
if (for_drop_) if (for_drop_)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment