Changeset 44491

Timestamp:

06/07/09 19:19:19 (9 months ago)

Author:

mitz@apple.com

Message:

Reviewed by Sam Weinig.

• fix < rdar://problem/6931661> -[WebView _selectionIsAll] returns YES when the selection is inside a text field.

• editing/VisibleSelection.cpp: (WebCore::VisibleSelection::isAll): Return false if the selection is in a shadow tree.

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• editing/VisibleSelection.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-06-07  Dan Bernstein  <mitz@apple.com>
+
+        Reviewed by Sam Weinig.
+
+        - fix <rdar://problem/6931661> -[WebView _selectionIsAll] returns YES
+          when the selection is inside a text field.
+
+        * editing/VisibleSelection.cpp:
+        (WebCore::VisibleSelection::isAll): Return false if the selection is in
+        a shadow tree.
+
 2009-06-07  Gustavo Noronha Silva  <gustavo.noronha@collabora.co.uk>
 

trunk/WebCore/editing/VisibleSelection.cpp

@@ -225 +225 @@
 bool VisibleSelection::isAll(StayInEditableContent stayInEditableContent) const
 {
-    return visibleStart().previous(stayInEditableContent).isNull() && visibleEnd().next(stayInEditableContent).isNull();
+    return !shadowTreeRootNode() && visibleStart().previous(stayInEditableContent).isNull() && visibleEnd().next(stayInEditableContent).isNull();
 }