On bookmark edit, the OK button is now disabled if the entered URL is...

chrome/browser/cocoa/bookmark_editor_controller.mm

 // Copyright (c) 2009 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.
 
 #include "base/logging.h"
 #include "base/mac_util.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_editor.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/profile.h"
@@ skipping 54 matching lines @@
     std::string url_string = node_->GetURL().possibly_invalid_spec();
     initialUrl_.reset([[NSString stringWithUTF8String:url_string.c_str()]
                         retain]);
   } else {
     initialName_.reset([@"" retain]);
     initialUrl_.reset([@"" retain]);
   }
   [nameField_ setStringValue:initialName_];
   [urlField_ setStringValue:initialUrl_];
 
+  // Get a ping when the URL text changes;
+  // trigger an initial ping to set things up.
+  [urlField_ setDelegate:self];
+  [self controlTextDidChange:nil];
+
   if (configuration_ == BookmarkEditor::SHOW_TREE) {
     // build the tree et al
     NOTIMPLEMENTED();
   } else {
     // Remember the NSBrowser's height; we will shrink our frame by that
     // much.
     NSRect frame = [[self window] frame];
     CGFloat browserHeight = [browser_ frame].size.height;
     frame.size.height -= browserHeight;
     frame.origin.y += browserHeight;
@@ skipping 27 matching lines @@
 
 // TODO(jrg)
 - (IBAction)newFolder:(id)sender {
   NOTIMPLEMENTED();
 }
 
 - (IBAction)cancel:(id)sender {
   [NSApp endSheet:[self window]];
 }
 
+// If possible, return a valid GURL from the URL text field.
+- (GURL)GURLFromUrlField {
+  NSString *url = [urlField_ stringValue];
+  GURL newURL = GURL([url UTF8String]);
+  if (!newURL.is_valid()) {
+    // Mimic observed friendliness from Windows
+    newURL = GURL([[NSString stringWithFormat:@"http://%@", url] UTF8String]);
+  }
+  return newURL;
+}
+
+// When the URL changes we may enable or disable the OK button.
+// We set ourselves as the delegate of urlField_ so this gets called.
+// (Yes, setting ourself as a delegate automatically registers us for
+// the notification.)
+- (void)controlTextDidChange:(NSNotification *)aNotification {
+  GURL newURL = [self GURLFromUrlField];
+  if (newURL.is_valid()) {
+    [okButton_ setEnabled:YES];
+  } else {
+    [okButton_ setEnabled:NO];
+  }
+}
+
 // TODO(jrg): Once the tree is available edits may be more extensive
 // than just name/url.
 - (IBAction)ok:(id)sender {
   NSString *name = [nameField_ stringValue];
   NSString *url = [urlField_ stringValue];
 
   if ((![name isEqual:initialName_]) ||
       (![url isEqual:initialUrl_])) {
     std::wstring newTitle = base::SysNSStringToWide(name);
-    GURL newURL = GURL([url UTF8String]);
+    GURL newURL = [self GURLFromUrlField];
     if (!newURL.is_valid()) {
-      // Mimic observed friendliness from Windows
-      newURL = GURL([[NSString stringWithFormat:@"http://%@", url] UTF8String]);
-    }
-    if (!newURL.is_valid()) {
-      // Silently ignoring a bad URL is unfriendly.
-      newURL = GURL();
+      // Shouldn't be reached -- OK button disabled if not valid!
+      NOTREACHED();
+      return;
     }
     int index = 0;
     BookmarkModel* model = profile_->GetBookmarkModel();
     if (node_) {
       index = parentNode_->IndexOfChild(node_);
       model->Remove(parentNode_, index);
     } else {
       index = parentNode_->GetChildCount();
     }
     const BookmarkNode* node = model->AddURL(parentNode_, index,
                                              newTitle, newURL);
     // Honor handler semantics: callback on node creation
     if (handler_.get())
       handler_->NodeCreated(node);
   }
 
   [NSApp endSheet:[self window]];
 }
 
 - (void)didEndSheet:(NSWindow*)sheet
          returnCode:(int)returnCode
         contextInfo:(void*)contextInfo {
+  // This is probably unnecessary but it feels cleaner since the
+  // delegate of a text field can be automatically registered for
+  // notifications.
+  [urlField_ setDelegate:nil];
+
   [[self window] orderOut:self];
 
   // BookmarkEditor::Show() will create us then run away.  Unusually
   // for a controller, we are responsible for deallocating ourself.
   [self autorelease];
 }
 
 
 - (NSString*)displayName {
   return [nameField_ stringValue];
 }
 
 - (NSString*)displayURL {
   return [urlField_ stringValue];
 }
 
 - (void)setDisplayName:(NSString*)name {
   [nameField_ setStringValue:name];
 }
 
 - (void)setDisplayURL:(NSString*)name {
   [urlField_ setStringValue:name];
+  [self controlTextDidChange:nil];
+}
+
+- (BOOL)okButtonEnabled {
+  return [okButton_ isEnabled];
 }
 
 @end  // BookmarkEditorController