Disable "OK" button in edit dialog of bookmark button when name is not valid...

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 49 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;
+  // Get a ping when the URL or name text fields change;
   // trigger an initial ping to set things up.
+  [nameField_ setDelegate:self];
   [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];
@@ skipping 47 matching lines @@
   }
   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()) {
+  NSString* name = [nameField_ stringValue];
+
+  // if empty or only whitespace, name is not valid.
+  bool name_valid = true;
+  if (([name length] == 0) ||
+      ([[name stringByTrimmingCharactersInSet:[NSCharacterSet
+              whitespaceAndNewlineCharacterSet]] length] == 0)) {
+    name_valid = false;
+  }
+
+  if (newURL.is_valid() && name_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];
@@ skipping 25 matching lines @@
 
   [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.
+  [nameField_ setDelegate:nil];
   [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];
+  [self controlTextDidChange:nil];
 }
 
 - (void)setDisplayURL:(NSString*)name {
   [urlField_ setStringValue:name];
   [self controlTextDidChange:nil];
 }
 
 - (BOOL)okButtonEnabled {
   return [okButton_ isEnabled];
 }
 
 @end  // BookmarkEditorController