Follow-up clean-up in LoginHandler

components/password_manager/core/browser/password_manager.cc

Index: components/password_manager/core/browser/password_manager.cc
diff --git a/components/password_manager/core/browser/password_manager.cc b/components/password_manager/core/browser/password_manager.cc
index 3529e88657ea53cf9dd9de2a9252978f15a6baf0..d55849868e5d709f64fa1ef477a50333bf2f691e 100644
--- a/components/password_manager/core/browser/password_manager.cc
+++ b/components/password_manager/core/browser/password_manager.cc
@@ -403,6 +403,9 @@ void PasswordManager::AddObserverAndDeliverCredentials(
   observers_.AddObserver(observer);
 
   observer->set_signon_realm(observed_form.signon_realm);
+  // TODO(vabr): Even though the observers do the realm check, this mechanism
+  // will still result in every observer being notified about every form. We
+  // could perhaps do better by registering an observer call-back instead.
 
   std::vector<PasswordForm> observed_forms;
   observed_forms.push_back(observed_form);