Save password functionality added to the save password bubble

chrome/browser/content_settings/tab_specific_content_settings.h

 // 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.
 
 #ifndef CHROME_BROWSER_CONTENT_SETTINGS_TAB_SPECIFIC_CONTENT_SETTINGS_H_
 #define CHROME_BROWSER_CONTENT_SETTINGS_TAB_SPECIFIC_CONTENT_SETTINGS_H_
 
 #include <set>
 #include <string>
 
@@ skipping 277 matching lines @@
       int64 frame_id,
       int64 parent_frame_id,
       bool is_main_frame,
       const GURL& validated_url,
       bool is_error_page,
       bool is_iframe_srcdoc,
       content::RenderViewHost* render_view_host) OVERRIDE;
   virtual void AppCacheAccessed(const GURL& manifest_url,
                                 bool blocked_by_policy) OVERRIDE;
 
-  // If user clicks on 'save password' this will have the password saved upon
-  // the next navigation.
+  // Called when the user clicks 'save password' button. Instructs
+  // |form_to_save| to save the password when the next navigation occurs or
+  // when the tab is closed.
   bool PasswordAccepted();

[Peter Kasting, 2013/09/10 23:04:41]

These two functions should also be replaced by a single SetPasswordAction()
routine.

[npentrel, 2013/09/11 09:10:42]

Done.

 
-  // If user clicks on 'never save password for this site' this have the
-  // password blacklisted upon the next navigation.
+  // Called when the user clicks 'never for this site' button. Instructs
+  // |form_to_save| to blacklist the password when the next navigation occurs
+  // or when the tab is closed.
   bool PasswordFormBlacklisted();
 
   // Message handlers. Public for testing.
   void OnContentBlocked(ContentSettingsType type,
                         const std::string& resource_identifier);
   void OnContentAllowed(ContentSettingsType type);
 
   // These methods are invoked on the UI thread by the static functions above.
   // Public for testing.
   void OnCookiesRead(const GURL& url,
@@ skipping 21 matching lines @@
                                   bool allowed);
 
   // This method is called to update the status about the microphone and
   // camera stream access. |request_permissions| contains a list of requested
   // media stream types and the permission for each type.
   void OnMediaStreamPermissionSet(
       const GURL& request_origin,
       const MediaStreamDevicesController::MediaStreamTypePermissionMap&
           request_permissions);
 
-  // This method is called to pass the |form_to_save| on a successful password
-  // submission. It also updates the status of the save password content
-  // setting.
+  // Called when a password form is submitted. |form_to_save| is the submitted
+  // form. This form can save or blacklist the login credentials via
+  // PasswordAccepted() and PasswordFormBlacklisted respectively.
   void OnPasswordSubmitted(PasswordFormManager* form_to_save);
 
   // There methods are called to update the status about MIDI access.
   void OnMIDISysExAccessed(const GURL& reqesting_origin);
   void OnMIDISysExAccessBlocked(const GURL& requesting_origin);
 
   // Adds the given |SiteDataObserver|. The |observer| is notified when a
   // locale shared object, like for example a cookie, is accessed.
   void AddSiteDataObserver(SiteDataObserver* observer);
 
@@ skipping 71 matching lines @@
   GURL media_stream_access_origin_;
 
   // The PasswordFormManager managing the form we're asking the user about,
   // and should update as per the decision.
   scoped_ptr<PasswordFormManager> form_to_save_;
 
   DISALLOW_COPY_AND_ASSIGN(TabSpecificContentSettings);
 };
 
 #endif  // CHROME_BROWSER_CONTENT_SETTINGS_TAB_SPECIFIC_CONTENT_SETTINGS_H_