Add bookmark bar/manager actions to track drag and drop.

chrome/browser/ui/views/bookmarks/bookmark_bar_view.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/bookmarks/bookmark_bar_view.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <limits>
@@ skipping 1162 matching lines @@
   } else if (drop_info_->location.on) {
     parent_node = root->GetChild(index);
     index = parent_node->child_count();
   } else {
     parent_node = root;
   }
   const BookmarkNodeData data = drop_info_->data;
   DCHECK(data.is_valid());
   bool copy = drop_info_->location.operation == ui::DragDropTypes::DRAG_COPY;
   drop_info_.reset();
+
+  content::RecordAction(base::UserMetricsAction("BookmarkBar_DragEnd"));

[Alexei Svitkine (slow), 2016/11/18 16:52:15]

Are you missing this in histograms.xml?

[calamity, 2016/11/21 00:49:58]

This already exists in actions.xml, but was only implemented on Mac. Adding it
here implements it on other desktop platforms.

[Alexei Svitkine (slow), 2016/11/21 15:47:06]

Ah, I see.

It does not appear to have an owner or description in actions.xml. If it's
interesting to you, please add yourself as owner and provide a description.

(Otherwise, if it's not interesting, then I suggest removing it in the Mac code
rather than adding it in views.)

[calamity, 2016/11/22 03:18:06]

Done.

   return chrome::DropBookmarks(
       browser_->profile(), data, parent_node, index, copy);
 }
 
 void BookmarkBarView::OnThemeChanged() {
   UpdateAppearanceForTheme();
 }
 
 const char* BookmarkBarView::GetClassName() const {
   return kViewClassName;
@@ skipping 926 matching lines @@
     return;
   apps_page_shortcut_->SetVisible(visible);
   UpdateBookmarksSeparatorVisibility();
   LayoutAndPaint();
 }
 
 void BookmarkBarView::OnShowManagedBookmarksPrefChanged() {
   if (UpdateOtherAndManagedButtonsVisibility())
     LayoutAndPaint();
 }