Fix TODO style in password_generation_agent.mm

ios/chrome/browser/passwords/password_generation_agent.mm

 // Copyright 2014 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.
 
 #import "ios/chrome/browser/passwords/password_generation_agent.h"
 
 #include <stddef.h>
 
 #import "base/ios/weak_nsobject.h"
 #include "base/mac/foundation_util.h"
@@ skipping 29 matching lines @@
 // an account creation form.
 const size_t kMinimumTextFieldsForAccountCreation = 3;
 
 // Returns true if |urls| contains |url|.
 bool VectorContainsURL(const std::vector<GURL>& urls, const GURL& url) {
   return std::find(urls.begin(), urls.end(), url) != urls.end();
 }
 
 // Returns whether |field| should be considered a text field. Implementation
 // mirrors that of password_controller.js.
-// TODO(dconnelly): Figure out how (and if) to determine if |field| is visible.
-// http://crbug.com/433856
+// TODO(crbug.com/433856): Figure out how to determine if |field| is visible.
 bool IsTextField(const autofill::FormFieldData& field) {
   return field.form_control_type == "text" ||
          field.form_control_type == "email" ||
          field.form_control_type == "number" ||
          field.form_control_type == "tel" || field.form_control_type == "url" ||
          field.form_control_type == "search" ||
          field.form_control_type == "password";
 }
 
 }  // namespace
@@ skipping 16 matching lines @@
 
 // Returns a list of all password fields in |form|.
 - (std::vector<autofill::FormFieldData>)passwordFieldsInForm:
     (const autofill::PasswordForm&)form;
 
 // Merges the data from local heuristics, the autofill server, and the password
 // manager to find the field that should trigger the password generation UI
 // when selected by the user. The resulting field is stored in
 // |_passwordGenerationField|. This logic is nearly identical to that of the
 // upstream autofill::PasswordGenerationAgent::DetermineGenerationElement.
-// TODO(dconnelly): Try to find a way to share this code with the upstream
-// implementation, even though it lives in the renderer.
-// http://crbug.com/434679.
 - (void)determinePasswordGenerationField;
 
 // Returns YES if the specified form and field should trigger the
 // password generation UI.
 - (BOOL)isGenerationForm:(const base::string16&)formName
                    field:(const base::string16&)fieldName;
 
 // The name of the form identified as an account creation form, if it exists.
 - (NSString*)passwordGenerationFormName;
 
@@ skipping 158 matching lines @@
   return passwordFields;
 }
 
 - (void)allowPasswordGenerationForForm:(const autofill::PasswordForm&)form {
   _allowedGenerationFormOrigins.push_back(form.origin);
   [self determinePasswordGenerationField];
 }
 
 - (void)processParsedPasswordForms:
     (const std::vector<autofill::PasswordForm>&)forms {
-  // TODO(dconnelly): Find a way to share some of this logic with the desktop
-  // agent. http://crbug.com/434679.
   for (const auto& passwordForm : forms) {
     if ([self formHasGAIARealm:passwordForm])
       continue;
     if (![self formHasEnoughTextFieldsForAccountCreation:passwordForm])
       continue;
     std::vector<autofill::FormFieldData> passwordFields(
         [self passwordFieldsInForm:passwordForm]);
     if (passwordFields.empty())
       continue;
     // This form checks out.
@@ skipping 164 matching lines @@
   DCHECK(_accessoryViewReadyCompletion);
   _accessoryViewReadyCompletion.get()([self currentAccessoryView], self);
 }
 
 - (BOOL)getLogKeyboardAccessoryMetrics {
   // Only store metrics for regular Autofill, not passwords.
   return NO;
 }
 
 @end