Style fixes for previous review

webkit/glue/webmenurunner_mac.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 "webkit/glue/webmenurunner_mac.h"
 
 #include "base/sys_string_conversions.h"
 
+@interface WebMenuRunner (PrivateAPI)
+
+// Worker function used during initialization.
+- (void)addItem:(const WebMenuItem&)item;
+
+// A callback for the menu controller object to call when an item is selected
+// from the menu. This is not called if the menu is dismissed without a
+// selection.
+- (void)menuItemSelected:(id)sender;
+
+@end  // WebMenuRunner (PrivateAPI)
+
 @implementation WebMenuRunner
 
 - (id)initWithItems:(const std::vector<WebMenuItem>&)items {
   if ((self = [super init])) {
-    menu_ = [[NSMenu alloc] initWithTitle:@""];
+    menu_.reset([[NSMenu alloc] initWithTitle:@""]);
     [menu_ setAutoenablesItems:NO];
-    menuItemWasChosen_ = NO;
     index_ = -1;
     for (int i = 0; i < static_cast<int>(items.size()); ++i)
       [self addItem:items[i]];
   }
   return self;
 }
 
-- (void)dealloc {
-  [menu_ release];
-  [super dealloc];
-}
-
 - (void)addItem:(const WebMenuItem&)item {
   if (item.type == WebMenuItem::SEPARATOR) {
     [menu_ addItem:[NSMenuItem separatorItem]];
     return;
   }
 
   NSString* title = base::SysUTF16ToNSString(item.label);
-  NSMenuItem* menu_item = [menu_ addItemWithTitle:title
-                                           action:@selector(menuItemSelected:)
-                                    keyEquivalent:@""];
-  [menu_item setEnabled:(item.enabled && item.type != WebMenuItem::GROUP)];
-  [menu_item setTarget:self];
+  NSMenuItem* menuItem = [menu_ addItemWithTitle:title
+                                          action:@selector(menuItemSelected:)
+                                   keyEquivalent:@""];
+  [menuItem setEnabled:(item.enabled && item.type != WebMenuItem::GROUP)];
+  [menuItem setTarget:self];
 }
 
 // Reflects the result of the user's interaction with the popup menu. If NO, the
 // menu was dismissed without the user choosing an item, which can happen if the
 // user clicked outside the menu region or hit the escape key. If YES, the user
 // selected an item from the menu.
 - (BOOL)menuItemWasChosen {
   return menuItemWasChosen_;
 }
 
@@ skipping 19 matching lines @@
   if ([self menuItemWasChosen])
     index_ = [button indexOfSelectedItem];
 }
 
 - (int)indexOfSelectedItem {
   return index_;
 }
 
 @end  // WebMenuRunner
 
+namespace webkit_glue {
+
 // Helper function for manufacturing input events to send to WebKit.
-NSEvent* CreateEventForMenuAction(BOOL item_chosen, int window_num,
-                                  int item_height, int selected_index,
-                                  NSRect menu_bounds, NSRect view_bounds) {
+NSEvent* EventWithMenuAction(BOOL item_chosen, int window_num,
+                             int item_height, int selected_index,
+                             NSRect menu_bounds, NSRect view_bounds) {
   NSEvent* event = nil;
   double event_time = (double)(AbsoluteToDuration(UpTime())) / 1000.0;
 
   if (item_chosen) {
     // Construct a mouse up event to simulate the selection of an appropriate
     // menu item.
     NSPoint click_pos;
     click_pos.x = menu_bounds.size.width / 2;
 
     // This is going to be hard to calculate since the button is painted by
@@ skipping 28 matching lines @@
                          windowNumber:window_num
                               context:nil
                            characters:@""
           charactersIgnoringModifiers:@""
                             isARepeat:NO
                               keyCode:0x1B];
   }
 
   return event;
 }
+
+}  // namespace webkit_glue