Move setToolTipText related functions from RenderView to RenderWidget.

webkit/glue/webmenurunner_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 "webkit/glue/webmenurunner_mac.h"
 
 #include "base/sys_string_conversions.h"
 
 #if !defined(MAC_OS_X_VERSION_10_6) || \
     MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_6
@@ skipping 48 matching lines @@
 - (void)addItem:(const WebMenuItem&)item {
   if (item.type == WebMenuItem::SEPARATOR) {
     [menu_ addItem:[NSMenuItem separatorItem]];
     return;
   }
 
   NSString* title = base::SysUTF16ToNSString(item.label);
   NSMenuItem* menuItem = [menu_ addItemWithTitle:title
                                           action:@selector(menuItemSelected:)
                                    keyEquivalent:@""];
+  NSString* toolTip = base::SysUTF16ToNSString(item.toolTip);
+  [menuItem setToolTip:toolTip];

[Nico, 2011/05/25 17:12:58]

Do you only want to do this if the toolTip isn't the empty string?

[honten.org, 2011/05/25 17:25:49]

You are right.

We don't have to set the string, if it's empty.

I'll add check routine.

On 2011/05/25 17:12:58, Nico wrote:

   [menuItem setEnabled:(item.enabled && item.type != WebMenuItem::GROUP)];
   [menuItem setTarget:self];
 
   // Set various alignment/language attributes. Note that many (if not most) of
   // these attributes are functional only on 10.6 and above.
   scoped_nsobject<NSMutableDictionary> attrs(
       [[NSMutableDictionary alloc] initWithCapacity:3]);
   scoped_nsobject<NSMutableParagraphStyle> paragraphStyle(
       [[NSMutableParagraphStyle alloc] init]);
   [paragraphStyle setAlignment:rightAligned_ ? NSRightTextAlignment
@@ skipping 127 matching lines @@
                            characters:@""
           charactersIgnoringModifiers:escape_str
                             isARepeat:NO
                               keyCode:0x1B];
   }
 
   return event;
 }
 
 }  // namespace webkit_glue