Integrate the Spelling service to Chrome.

chrome/browser/tab_contents/render_view_context_menu_mac.mm

 // Copyright (c) 2011 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 "chrome/browser/tab_contents/render_view_context_menu_mac.h"
 
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_nsobject.h"
 #include "base/message_loop.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #import "chrome/browser/ui/cocoa/browser_window_controller.h"
 #import "chrome/browser/ui/cocoa/menu_controller.h"
 #include "grit/generated_resources.h"
 
+namespace {
+
+// Retrieves an NSMenuItem which has the specified command_id. This function
+// traverses the given |model| in the depth-first order. When this function
+// finds an item whose command_id is the same as the given |command_id|, it
+// returns the NSMenuItem associated with the item. This function emulates
+// views::MenuItemViews::GetMenuItemByID() for Mac.
+NSMenuItem* GetMenuItemByID(ui::MenuModel* model,
+                            NSMenu* menu,
+                            int command_id) {
+  for (int i = 0; i < model->GetItemCount(); ++i) {
+    NSMenuItem* item = [menu itemAtIndex:i];
+    if (model->GetCommandIdAt(i) == command_id)
+      return item;
+
+    ui::MenuModel* submenu = model->GetSubmenuModelAt(i);
+    if (submenu && [item hasSubmenu]) {
+      NSMenuItem* subitem = GetMenuItemByID(submenu,
+                                            [item submenu],
+                                            command_id);
+      if (subitem)
+        return subitem;
+    }
+  }
+  return nil;
+}
+
+}  // namespace
+
 // Obj-C bridge class that is the target of all items in the context menu.
 // Relies on the tag being set to the command id.
 
 RenderViewContextMenuMac::RenderViewContextMenuMac(
     TabContents* web_contents,
     const ContextMenuParams& params,
     NSView* parent_view)
     : RenderViewContextMenu(web_contents, params),
       parent_view_(parent_view) {
 }
@@ skipping 70 matching lines @@
   NSString* text = base::SysUTF16ToNSString(params_.selection_text);
   NSPasteboard* pboard = [NSPasteboard pasteboardWithUniqueName];
   // 10.5 and earlier require declareTypes before setData.
   // See the documentation on [NSPasteboard declareTypes].
   NSArray* toDeclare = [NSArray arrayWithObject:NSStringPboardType];
   [pboard declareTypes:toDeclare owner:nil];
   BOOL ok = [pboard setString:text forType:NSStringPboardType];
   if (ok)
     NSPerformService(@"Look Up in Dictionary", pboard);
 }
+
+void RenderViewContextMenuMac::UpdateMenuItem(int command_id,
+                                              bool enabled,
+                                              const string16& title) {
+  NSMenuItem* item = GetMenuItemByID(&menu_model_, [menuController_ menu],
+                                     command_id);
+  if (!item)
+    return;
+
+  // Update the returned NSMenuItem directly so we can update it immediately.
+  [item setEnabled:enabled ? YES : NO];

[Avi (use Gerrit), 2011/08/23 18:38:19]

It's safe to just pass in enabled here; no need to convert it.

[Hironori Bono, 2011/08/25 11:18:58]

Done. Thank you for noticing it. I misunderstood our coding style.

+  [item setTitle:SysUTF16ToNSString(title)];
+  [[item menu] itemChanged:item];
+}