DO NOT SUBMIT

Source/WebKit/chromium/src/WebFrameImpl.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 67 matching lines @@
 #include "Console.h"
 #include "DOMUtilitiesPrivate.h"
 #include "DOMWindow.h"
 #include "Document.h"
 #include "DocumentFragment.h" // Only needed for ReplaceSelectionCommand.h :(
 #include "DocumentLoader.h"
 #include "DocumentMarker.h"
 #include "DocumentMarkerController.h"
 #include "Editor.h"
 #include "EventHandler.h"
+#include "FocusController.h"
 #include "FormState.h"
 #include "FrameLoadRequest.h"
 #include "FrameLoader.h"
 #include "FrameTree.h"
 #include "FrameView.h"
 #include "HTMLCollection.h"
 #include "HTMLFormElement.h"
 #include "HTMLFrameOwnerElement.h"
 #include "HTMLHeadElement.h"
 #include "HTMLInputElement.h"
 #include "HTMLLinkElement.h"
 #include "HTMLNames.h"
 #include "HistoryItem.h"
+#include "HitTestResult.h"
 #include "InspectorController.h"
 #include "Page.h"
 #include "painting/GraphicsContextBuilder.h"
 #include "Performance.h"
 #include "PlatformBridge.h"
 #include "PluginDocument.h"
 #include "PrintContext.h"
 #include "RenderFrame.h"
 #include "RenderObject.h"
 #include "RenderTreeAsText.h"
@@ skipping 994 matching lines @@
     if ((location + length < location) && (location + length))
         length = 0;
 
     Element* selectionRoot = frame()->selection()->rootEditableElement();
     Element* scope = selectionRoot ? selectionRoot : frame()->document()->documentElement();
     RefPtr<Range> range = TextIterator::rangeFromLocationAndLength(scope, location, length);
     if (!range)
         return false;
     IntRect intRect = frame()->editor()->firstRectForRange(range.get());
     rect = WebRect(intRect.x(), intRect.y(), intRect.width(), intRect.height());
+    // When inside an text control, don't adjust the range.
+    if (!selectionRoot)
+        rect = frame()->view()->contentsToWindow(rect);

[James Su, 2011/03/03 04:08:03]

IMHO, we should return frame based position in WebFrame implementation.
And add functions that return window based position into WebWidget/WebView.

[Robert Sesek, 2011/03/18 21:33:36]

Why? That seems unnecessary and makes this CL more complicated than it needs to
be.

 
     return true;
 }
 
+unsigned WebFrameImpl::characterIndexForPoint(const WebPoint& webPoint) const
+{
+    if (!frame())
+        return notFound;
+
+    IntPoint point = frame()->view()->windowToContents(webPoint);

[James Su, 2011/03/03 04:08:03]

Same as above.

+    HitTestResult result = frame()->eventHandler()->hitTestResultAtPoint(point, false);
+    Range* range = characterRangeAtPoint(result.point());
+    if (!range)
+        return notFound;
+
+    unsigned location, length;
+    getLocationAndLengthFromRange(range, location, length);
+    return location;
+}
+
 bool WebFrameImpl::executeCommand(const WebString& name)
 {
     ASSERT(frame());
 
     if (name.length() <= 2)
         return false;
 
     // Since we don't have NSControl, we will convert the format of command
     // string and call the function on Editor directly.
     String command = name;
@@ skipping 1138 matching lines @@
     ScriptValue result = m_frame->script()->executeScript(script, true);
 
     String scriptResult;
     if (!result.getString(scriptResult))
         return;
 
     if (!m_frame->navigationScheduler()->locationChangePending())
         m_frame->document()->loader()->writer()->replaceDocument(scriptResult);
 }
 
+// This function is copied from /WebKit2/WebPage/mac/WebPageMac.mm.
+Range* WebFrameImpl::characterRangeAtPoint(const IntPoint& point) const
+{
+    VisiblePosition position = frame()->visiblePositionForPoint(point);
+    if (position.isNull())
+        return NULL;
+
+    VisiblePosition previous = position.previous();
+    if (previous.isNotNull()) {
+        Range* previousCharacterRange = makeRange(previous, position).get();
+        IntRect rect = frame()->editor()->firstRectForRange(previousCharacterRange);
+        if (rect.contains(point))
+            return previousCharacterRange;
+    }
+
+    VisiblePosition next = position.next();
+    if (next.isNotNull()) {
+        Range* nextCharacterRange = makeRange(position, next).get();
+        IntRect rect = frame()->editor()->firstRectForRange(nextCharacterRange);
+        if (rect.contains(point))
+            return nextCharacterRange;
+    }
+
+    return NULL;
+}
+
+// This function is copied from /WebKit2/WebPage/WebPage.cpp.
+bool WebFrameImpl::getLocationAndLengthFromRange(Range* range, unsigned& location, unsigned& length) const
+{
+    location = notFound;
+    length = 0;
+
+    if (!range || !range->startContainer())
+        return false;
+
+    Element* selectionRoot = range->ownerDocument()->frame()->selection()->rootEditableElement();
+    Element* scope = selectionRoot ? selectionRoot : range->ownerDocument()->documentElement();
+
+    // Mouse events may cause TSM to attempt to create an NSRange for a portion of the view
+    // that is not inside the current editable region.  These checks ensure we don't produce
+    // potentially invalid data when responding to such requests.
+    if (range->startContainer() != scope && !range->startContainer()->isDescendantOf(scope))
+        return false;
+    if (range->endContainer() != scope && !range->endContainer()->isDescendantOf(scope))
+        return false;
+
+    RefPtr<Range> testRange = Range::create(scope->document(), scope, 0, range->startContainer(), range->startOffset());
+    ASSERT(testRange->startContainer() == scope);
+    location = TextIterator::rangeLength(testRange.get());
+    
+    ExceptionCode ec;
+    testRange->setEnd(range->endContainer(), range->endOffset(), ec);
+    ASSERT(testRange->startContainer() == scope);
+    length = TextIterator::rangeLength(testRange.get()) - location;
+    return true;
+}
+
 } // namespace WebKit