[Tab Groups] Add user action metrics for when a group is renamed through

the editor bubble.

Bug: 1040657
Change-Id: I532ffcc9257e038292f7a3d2d33ae31a50744558
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2028161Reviewed-by: Connie Wan <connily@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Commit-Queue: Charlene Yan <cyan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#737477}

chrome/browser/ui/views/tabs/tab_group_editor_bubble_view.cc

@@ -80,8 +80,12 @@ TabGroupEditorBubbleView::TabGroupEditorBubbleView(
 
   DialogDelegate::set_buttons(ui::DIALOG_BUTTON_NONE);
 
-  const auto* layout_provider = ChromeLayoutProvider::Get();
   const base::string16 title = tab_controller_->GetGroupTitle(group_);
+  title_at_opening_ = title;
+  DialogDelegate::set_close_callback(base::BindOnce(
+      &TabGroupEditorBubbleView::OnBubbleClose, base::Unretained(this)));
+
+  const auto* layout_provider = ChromeLayoutProvider::Get();
   const int horizontal_spacing = layout_provider->GetDistanceMetric(
       views::DISTANCE_RELATED_CONTROL_HORIZONTAL);
   const int vertical_menu_spacing = layout_provider->GetDistanceMetric(
@@ -233,6 +237,13 @@ void TabGroupEditorBubbleView::UpdateGroup() {
   tab_controller_->SetVisualDataForGroup(group_, new_data);
 }
 
+void TabGroupEditorBubbleView::OnBubbleClose() {
+  if (title_at_opening_ != title_field_->GetText()) {
+    base::RecordAction(
+        base::UserMetricsAction("TabGroups_TabGroupBubble_NameChanged"));
+  }
+}
+
 void TabGroupEditorBubbleView::TitleFieldController::ContentsChanged(
     views::Textfield* sender,
     const base::string16& new_contents) {

chrome/browser/ui/views/tabs/tab_group_editor_bubble_view.h

@@ -55,6 +55,8 @@ class TabGroupEditorBubbleView : public views::BubbleDialogDelegateView {
 
   SkColor background_color() const { return color(); }
 
+  void OnBubbleClose();
+
   TabController* const tab_controller_;
   const tab_groups::TabGroupId group_;
 
@@ -102,6 +104,8 @@ class TabGroupEditorBubbleView : public views::BubbleDialogDelegateView {
   // Creates the set of tab group colors to display and returns the color that
   // is initially selected.
   SkColor InitColorSet();
+
+  base::string16 title_at_opening_;
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_TABS_TAB_GROUP_EDITOR_BUBBLE_VIEW_H_

tools/metrics/actions/actions.xml

@@ -22330,6 +22330,15 @@ should be able to be added at any place in this file.
   </description>
 </action>
 
+<action name="TabGroups_TabGroupBubble_NameChanged">
+  <owner>connily@chromium.org</owner>
+  <owner>cyan@chromium.org</owner>
+  <owner>chrome-desktop-ui-sea@google.com</owner>
+  <description>
+    User changed the name of a group through the editor bubble.
+  </description>
+</action>
+
 <action name="TabGroups_TabGroupBubble_NewTabInGroup">
   <owner>connily@chromium.org</owner>
   <owner>cyan@chromium.org</owner>