Added of new UMA signals in order to be able to discover early if the "save password" feature gets …

chrome/browser/password_manager/password_manager.h

Index: chrome/browser/password_manager/password_manager.h
diff --git a/chrome/browser/password_manager/password_manager.h b/chrome/browser/password_manager/password_manager.h
index 786548d29d806b9e6cb03a4081222d1eb8e42567..1d0add69c224848f43d7a36f5f4c997c0d9872e2 100644
--- a/chrome/browser/password_manager/password_manager.h
+++ b/chrome/browser/password_manager/password_manager.h
@@ -113,8 +113,10 @@ class PasswordManager : public LoginModel,
     MAX_FAILURE_VALUE
   };
 
-  // Log failure for UMA
-  void RecordFailure(ProvisionalSaveFailure failure);
+  // Log failure for UMA. |form_origin| is used in order to know if the save
+  // password prompt behavior is monitored for the current host.

[Ilya Sherman, 2013/09/12 04:45:45]

I find the second sentence in this comment somewhat hard to follow.  One
possible rephrasing that I think would be clearer: "Logs additional metrics if
the |form_origin| corresponds to one of the top, explicitly monitored websites."

[jdomingos, 2013/09/12 12:01:01]

Done.

+  void RecordFailure(ProvisionalSaveFailure failure,
+                     const std::string& form_origin);
 
   // Possibly set up FieldTrial for testing other possible usernames. This only
   // happens if there are other_possible_usernames to be shown and the