mac: Almost fix Lion accented character bubble input in web contents.

content/browser/renderer_host/render_widget_host_view_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 "content/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include <QuartzCore/QuartzCore.h>
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
@@ skipping 2460 matching lines @@
   // events, such as the Characters dialog. In this case the text should be
   // sent as an input method event as well.
   // TODO(suzhe): It's hard for us to support replacementRange without accessing
   // the full web content. NOTE: If someone adds support for this, make sure
   // it works with the default range passed in by -insertText: below.
   BOOL isAttributedString = [string isKindOfClass:[NSAttributedString class]];
   NSString* im_text = isAttributedString ? [string string] : string;
   if (handlingKeyDown_) {
     textToBeInserted_.append(base::SysNSStringToUTF16(im_text));
   } else {
+    ui::Range replacement_range(replacementRange);
     renderWidgetHostView_->render_widget_host_->ImeConfirmComposition(
-        base::SysNSStringToUTF16(im_text));
+        base::SysNSStringToUTF16(im_text), replacement_range);
   }
 
   // Inserting text will delete all marked text automatically.
   hasMarkedText_ = NO;
 }
 
 - (void)insertText:(id)string {
   // This is a method on NSTextInput, not NSTextInputClient. But on 10.5, this
   // gets called anyway. Forward to the right method. http://crbug.com/47890
   [self insertText:string replacementRange:NSMakeRange(0, 0)];

[Avi (use Gerrit), 2011/10/11 23:58:36]

Does an NSRange of (0,0) convert to an invalid range on line 2481 above? Now
that we're passing that parameter along we should verify it's correct.

[Avi (use Gerrit), 2011/10/12 00:06:01]

According to ui::Range, this should be NSMakeRange(NSNotFound, 0) to convert to
an invalid range.

[Nico, 2011/10/12 00:10:36]

Great catch, thanks! Done.

 }
 
 - (void)viewDidMoveToWindow {
   if (canBeKeyView_) {
     NSWindow* newWindow = [self window];
     // Pointer comparison only, since we don't know if lastWindow_ is still
     // valid.
     if (newWindow) {
       // If we move into a new window, refresh the frame information. We
       // don't need to do it if it was the same window as it used to be in,
@@ skipping 206 matching lines @@
   if (!string) return NO;
 
   // If the user is currently using an IME, confirm the IME input,
   // and then insert the text from the service, the same as TextEdit and Safari.
   [self confirmComposition];
   [self insertText:string];
   return YES;
 }
 
 @end