 Chromium Code Reviews
 Chromium Code Reviews Issue 256003003:
  Enable Automatic Passwords Saving  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master
    
  
    Issue 256003003:
  Enable Automatic Passwords Saving  (Closed) 
  Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: components/password_manager/core/browser/password_manager_client.h | 
| diff --git a/components/password_manager/core/browser/password_manager_client.h b/components/password_manager/core/browser/password_manager_client.h | 
| index ef9c8b0f8d19a4447fed86a57a38610750c5e516..1d43f7bd73e327115f325e9491175d5a6c3af773 100644 | 
| --- a/components/password_manager/core/browser/password_manager_client.h | 
| +++ b/components/password_manager/core/browser/password_manager_client.h | 
| @@ -25,6 +25,10 @@ class PasswordManagerClient { | 
| PasswordManagerClient() {} | 
| virtual ~PasswordManagerClient() {} | 
| + virtual bool IsAutomaticPasswordsSavingEnabled() const { | 
| 
vabr (Chromium)
2014/04/28 09:13:14
Please document this method. For example:
"For aut
 | 
| + return false; | 
| 
vabr (Chromium)
2014/04/28 09:13:14
nit: Please only declare the method here, and defi
 
rchtara
2014/04/28 11:28:29
Done.
 
rchtara
2014/04/28 11:28:29
Done.
 | 
| + } | 
| + | 
| // Informs the embedder of a password form that can be saved if the user | 
| // allows it. The embedder is not required to prompt the user if it decides | 
| // that this form doesn't need to be saved. |