Fix 2 issues with incognito extensions on linux:

- Drag and drop of browser actions wouldn't work properly from an incognito
window.
- Installing an extension from an incognito window raised a DCHECK.

Review URL: http://codereview.chromium.org/983004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41764 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 5 insertions, 3 deletions

diff --git a/chrome/browser/gtk/browser_actions_toolbar_gtk.cc b/chrome/browser/gtk/browser_actions_toolbar_gtk.cc
index 4e2aa8a..7a9ecdf 100644
--- a/chrome/browser/gtk/browser_actions_toolbar_gtk.cc
+++ b/chrome/browser/gtk/browser_actions_toolbar_gtk.cc
@@ -582,8 +582,6 @@ gboolean BrowserActionsToolbarGtk::OnDragMotion(GtkWidget* widget,
     return FALSE;
 
   drop_index_ = x < kButtonSize ? 0 : x / (kButtonSize + kButtonPadding);
-  if (profile_->IsOffTheRecord())
-    drop_index_ = model_->IncognitoIndexToOriginal(drop_index_);
 
   // We will go ahead and reorder the child in order to provide visual feedback
   // to the user. We don't inform the model that it has moved until the drag
@@ -597,8 +595,12 @@ gboolean BrowserActionsToolbarGtk::OnDragMotion(GtkWidget* widget,
 
 void BrowserActionsToolbarGtk::OnDragEnd(GtkWidget* button,
                                          GdkDragContext* drag_context) {
-  if (drop_index_ != -1)
+  if (drop_index_ != -1) {
+    if (profile_->IsOffTheRecord())
+      drop_index_ = model_->IncognitoIndexToOriginal(drop_index_);
+
     model_->MoveBrowserAction(drag_button_->extension(), drop_index_);
+  }
 
   drag_button_ = NULL;
   drop_index_ = -1;