Adding UMA stats.

chrome/browser/ui/gtk/password_generation_bubble_gtk.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/gtk/password_generation_bubble_gtk.h"
 
-#include "base/utf_string_conversions.h"
+#include "base/metrics/histogram.h"
 #include "chrome/browser/autofill/password_generator.h"
 #include "chrome/browser/password_manager/password_manager.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/gtk/bubble/bubble_gtk.h"
 #include "chrome/browser/ui/gtk/gtk_chrome_link_button.h"
 #include "chrome/browser/ui/gtk/gtk_theme_service.h"
 #include "chrome/browser/ui/gtk/gtk_util.h"
 #include "chrome/browser/ui/tab_contents/tab_contents.h"
 #include "chrome/common/autofill_messages.h"
@@ skipping 68 matching lines @@
   gtk_widget_grab_focus(text_field_);
 
   bubble_ = BubbleGtk::Show(tab->web_contents()->GetContentNativeView(),
                             &anchor_rect,
                             content,
                             BubbleGtk::ARROW_LOCATION_TOP_LEFT,
                             BubbleGtk::MATCH_SYSTEM_THEME |
                                 BubbleGtk::POPUP_WINDOW |
                                 BubbleGtk::GRAB_INPUT,
                             GtkThemeService::GetFrom(tab_->profile()),
-                            NULL);  // delegate
+                            this);  // delegate
 
   g_signal_connect(content, "destroy",
                    G_CALLBACK(&OnDestroyThunk), this);
   g_signal_connect(accept_button, "clicked",
                    G_CALLBACK(&OnAcceptClickedThunk), this);
   g_signal_connect(text_field_, "icon-press",
                    G_CALLBACK(&OnRegenerateClickedThunk), this);
+  g_signal_connect(text_field_, "changed",
+                   G_CALLBACK(&OnPasswordEditedThunk), this);
   g_signal_connect(learn_more_link, "clicked",
                    G_CALLBACK(OnLearnMoreLinkClickedThunk), this);
 }
 
 PasswordGenerationBubbleGtk::~PasswordGenerationBubbleGtk() {}
 
+void PasswordGenerationBubbleGtk::BubbleClosing(
+    BubbleGtk* bubble, bool closed_by_escape) {
+  // Send the password generation status to the renderer.
+  RenderViewHost* render_view_host = tab_->web_contents()->GetRenderViewHost();
+  render_view_host->Send(new AutofillMsg_PasswordGenerationBubbleClosed(
+      render_view_host->GetRoutingID(),
+      password_generation_status_));

[Ilya Sherman, 2012/07/18 05:36:15]

What's the advantage of sending the status to the renderer, rather than logging
the histograms from the browser process?  In general, we prefer to (a) do more
in the browser, and less in the renderer; and (b) send less data from one
process to the other.  In this case, I'm not seeing the gain from pushing this
work to the renderer.

[Ilya Sherman, 2012/07/18 05:38:14]

Reading back through the comments on this CL, I think you might be doing this
based on Garrett's comment that "Some of the stats (bubble creation, password
accepted) can be pushed into the renderer".  I don't think this is what he
meant, though I'll let him clarify in case I'm misunderstanding.

[Garrett Casto, 2012/07/18 17:20:03]

Yeah, I just meant that for those particular stats you can already tell in the
renderer that they are happening so you wouldn't need to edit each individual UI
file. Since we want to keep track of other stats as well I don't know if it
actually makes sense to log them there, I was more musing about possible ways to
cut down on code duplication. Given the setup now, I think that just having a
helper function that logs a PasswordGenerationStatus would be better.

[zysxqn, 2012/07/18 19:16:46]

Done.

+}
+
 void PasswordGenerationBubbleGtk::OnDestroy(GtkWidget* widget) {
   // We are self deleting, we have a destroy signal setup to catch when we are
   // destroyed (via the BubbleGtk being destroyed), and delete ourself.
   delete this;
 }
 
 void PasswordGenerationBubbleGtk::OnAcceptClicked(GtkWidget* widget) {
-  RenderViewHost* render_view_host = tab_->web_contents()->GetRenderViewHost();
-  render_view_host->Send(new AutofillMsg_GeneratedPasswordAccepted(
-      render_view_host->GetRoutingID(),
-      UTF8ToUTF16(gtk_entry_get_text(GTK_ENTRY(text_field_)))));
+  password_generation_status_.password_accepted = true;
+  password_generation_status_.generated_password =
+      gtk_entry_get_text(GTK_ENTRY(text_field_));
   tab_->password_manager()->SetFormHasGeneratedPassword(form_);
   bubble_->Close();
 }
 
 void PasswordGenerationBubbleGtk::OnRegenerateClicked(
     GtkWidget* widget,
     GtkEntryIconPosition icon_pos,
     GdkEvent* event) {
   gtk_entry_set_text(GTK_ENTRY(text_field_),
                      password_generator_->Generate().c_str());
+  password_generation_status_.password_regenerated = true;
+}
+
+void PasswordGenerationBubbleGtk::OnPasswordEdited(GtkWidget* widget) {
+  password_generation_status_.password_edited = true;
 }
 
 void PasswordGenerationBubbleGtk::OnLearnMoreLinkClicked(GtkButton* button) {
+  password_generation_status_.learn_more_visited = true;
   Browser* browser = browser::FindBrowserWithWebContents(tab_->web_contents());
   content::OpenURLParams params(
       GURL(chrome::kAutoPasswordGenerationLearnMoreURL), content::Referrer(),
       NEW_FOREGROUND_TAB, content::PAGE_TRANSITION_LINK, false);
   browser->OpenURL(params);
   bubble_->Close();
 }