[Autofill] Add user type metrics.

chrome/browser/ui/autofill/autofill_dialog_controller_impl.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/autofill/autofill_dialog_controller_impl.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 1009 matching lines @@
 }
 
 void AutofillDialogControllerImpl::OnDidGetFullWallet(
     scoped_ptr<wallet::FullWallet> full_wallet) {
   NOTIMPLEMENTED();
   WalletRequestCompleted(true);
 }
 
 void AutofillDialogControllerImpl::OnDidGetWalletItems(
     scoped_ptr<wallet::WalletItems> wallet_items) {
+  if (!wallet_items_ && wallet_items) {

[Dan Beam, 2013/03/07 17:16:50]

I don't really know how you'd make this harder to misread, but I definitely
thought this said

  if (!wallet_items && wallet_items)

which would be never true @_@ (just a _ difference is common, I guess... I'm
doing it below)

[Dan Beam, 2013/03/07 17:16:50]

this currently assumes we don't re-request wallet items on the first error.  if
this ever changes this will break.

[Ilya Sherman, 2013/03/09 01:58:57]

Do you have a suggestion on how to better hook this?  Add a boolean that tracks
whether this is the first time reaching here?

[Dan Beam, 2013/03/09 02:02:31]

^ yes, something along these lines, or figure out a way to ask metric_logger_
whether it's already logged this type of metric.

+    // On the first successful response, log any relevant user type metrics.
+    // Consider a user to be an Autofill user if the user has any credit cards
+    // or addresses saved.  Check that the item count is greater than 1 because
+    // an "empty" menu still has the "add new" menu item.
+    bool has_autofill_profiles =
+        suggested_cc_.GetItemCount() > 1 ||
+        suggested_billing_.GetItemCount() > 1;

[Dan Beam, 2013/03/07 17:16:50]

what if there's a network error/wallet error/malformed response/this response
never comes?

[Ilya Sherman, 2013/03/09 01:58:57]

Then we won't get a stat for that user.  Since this should affect all users
roughly equally, I'm not too worried about that.

[Dan Beam, 2013/03/09 02:02:31]

how do you know this will affect users equally? can't you simply move this into
WalletRequestCompleted()?

+    AutofillMetrics::DialogUserTypeMetric user_type;
+    if (SignedInState() == SIGNED_IN) {
+      if (!wallet_items->instruments().empty())
+        user_type = AutofillMetrics::DIALOG_WALLET_USER_SIGNED_IN;
+      else if (has_autofill_profiles)
+        user_type = AutofillMetrics::DIALOG_AUTOFILL_USER_SIGNED_IN;
+      else
+        user_type = AutofillMetrics::DIALOG_NEW_USER_SIGNED_IN;
+    } else {
+      if (has_autofill_profiles)
+        user_type = AutofillMetrics::DIALOG_AUTOFILL_USER_NOT_SIGNED_IN;
+      else
+        user_type = AutofillMetrics::DIALOG_NEW_USER_NOT_SIGNED_IN;
+    }
+    metric_logger_.LogDialogUserType(dialog_type_, user_type);
+  }
+
   bool items_changed = !wallet_items_ || *wallet_items != *wallet_items_;
   wallet_items_ = wallet_items.Pass();
   WalletRequestCompleted(true);
 
   if (items_changed) {
     GenerateSuggestionsModels();
     view_->ModelChanged();
     view_->UpdateAccountChooser();
     view_->UpdateNotificationArea();
   }
@@ skipping 393 matching lines @@
   // If the user is editing or inputting data, ask the view.
   if (item_key.empty() || section_editing_state_[SECTION_SHIPPING])
     return view_->UseBillingForShipping();
 
   // Otherwise, the checkbox should be hidden so its state is irrelevant.
   // Always use the shipping suggestion model.
   return false;
 }
 
 }  // namespace autofill