Autofill OSX: Disable inputs and Verify button after permanent error.

chrome/browser/ui/cocoa/autofill/card_unmask_prompt_view_bridge.mm

Index: chrome/browser/ui/cocoa/autofill/card_unmask_prompt_view_bridge.mm
diff --git a/chrome/browser/ui/cocoa/autofill/card_unmask_prompt_view_bridge.mm b/chrome/browser/ui/cocoa/autofill/card_unmask_prompt_view_bridge.mm
index 6a1aa5695463f5c4922ef6db1d42854cddcd7c4a..365c888998652561f421558e71384ed3d1b9f42d 100644
--- a/chrome/browser/ui/cocoa/autofill/card_unmask_prompt_view_bridge.mm
+++ b/chrome/browser/ui/cocoa/autofill/card_unmask_prompt_view_bridge.mm
@@ -199,6 +199,13 @@ void CardUnmaskPromptViewBridge::PerformClose() {
   [self updateVerifyButtonEnabled];
 }
 
+- (void)setInputsEnabled:(BOOL)enabled {

[bondd, 2015/03/26 19:50:06]

Only ever called with enabled == NO. I could rename it to -disableInputs if you
prefer.

[groby-ooo-7-16, 2015/03/26 20:17:45]

No, I'd leave it, for API symmetry. If you leave this as a separate call.

but - why not:

for (NSView view in [[self view] subviews])
  [view setEnabled:enabled];

which could be neatly inlined.

[bondd, 2015/03/26 20:33:12]

We want to leave the Cancel button enabled.

[bondd, 2015/03/26 20:46:52]

Also, unless I'm missing something this will give you compiler
warning-treated-as-error
"error: 'NSView' may not respond to 'setEnabled:'"
and will not touch any of these UI elements because they are all contained in
other NSViews (inputRowView_ and storageView_). But like I said, maybe I am
missing something here...it's certainly happened before. :)

+  [cvcInput_ setEnabled:enabled];
+  [monthPopup_ setEnabled:enabled];
+  [yearPopup_ setEnabled:enabled];
+  [storageCheckbox_ setEnabled:enabled];
+}
+
 - (void)setRetriableErrorMessage:(const base::string16&)text {
   NSAttributedString* attributedString =
       constrained_window::GetAttributedLabelString(
@@ -235,11 +242,14 @@ void CardUnmaskPromptViewBridge::PerformClose() {
   }
 
   [permanentErrorBox_ setHidden:text.empty()];
+  [self setInputsEnabled:NO];
+  [self updateVerifyButtonEnabled];
   [self setRetriableErrorMessage:base::string16()];
 }
 
 - (void)updateVerifyButtonEnabled {
   BOOL enable = ![inputRowView_ isHidden] &&
+                ![[permanentErrorLabel_ stringValue] length] &&

[bondd, 2015/03/26 19:50:06]

Alternatively, this line could be "[cvcInput_ isEnabled] &&"

[groby-ooo-7-16, 2015/03/26 20:17:45]

I like the current one better, because it's more general. The cvcInput will
break if there are ever additional cases that can set an error

[bondd, 2015/03/26 20:33:12]

Acknowledged.

                 bridge_->GetController()->InputCvcIsValid(
                     base::SysNSStringToUTF16([cvcInput_ stringValue])) &&
                 [self expirationDateIsValid];