REGRESSION (r167652): Broke fast/regions/cssom/region-range-for-box-crash.html in...
authorrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Apr 2014 19:48:29 +0000 (19:48 +0000)
committerrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Apr 2014 19:48:29 +0000 (19:48 +0000)
https://bugs.webkit.org/show_bug.cgi?id=131982

Reviewed by David Hyatt.

Source/WebCore:
The problem was creating the Range from the arguments received at RenderView::setSelection(). Specifically
in this test endPos is 1 when the element has not children which creates an invalid Range.

* rendering/RenderView.cpp:
(WebCore::RenderView::splitSelectionBetweenSubtrees): Pass 0 as startPos and endPos to Range::create() as we are
not interested in the positions at this point.

LayoutTests:
* TestExpectations: Unskip fast/regions/cssom/region-range-for-box-crash.html.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@167675 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/TestExpectations
Source/WebCore/ChangeLog
Source/WebCore/rendering/RenderView.cpp

index 9165623..fd57085 100644 (file)
@@ -1,3 +1,12 @@
+2014-04-22  Manuel Rego Casasnovas  <rego@igalia.com>
+
+        REGRESSION (r167652): Broke fast/regions/cssom/region-range-for-box-crash.html in debug mode
+        https://bugs.webkit.org/show_bug.cgi?id=131982
+
+        Reviewed by David Hyatt.
+
+        * TestExpectations: Unskip fast/regions/cssom/region-range-for-box-crash.html.
+
 2014-04-22  Brent Fulgham  <bfulgham@apple.com>
 
         Check (rather than assume) element is a RenderTableSection before using it
index 1a20ebc..dcd95fc 100644 (file)
@@ -106,5 +106,3 @@ webkit.org/b/131318 http/tests/inspector/replay/document-last-modified-fallback-
 
 # Expando properties on attribute nodes disappear
 webkit.org/b/88045 fast/dom/gc-attribute-node.html [ Failure Pass ]
-
-webkit.org/b/131982 [ Debug ] fast/regions/cssom/region-range-for-box-crash.html [ Crash ]
index 5be0e95..d9e792e 100644 (file)
@@ -1,3 +1,17 @@
+2014-04-22  Manuel Rego Casasnovas  <rego@igalia.com>
+
+        REGRESSION (r167652): Broke fast/regions/cssom/region-range-for-box-crash.html in debug mode
+        https://bugs.webkit.org/show_bug.cgi?id=131982
+
+        Reviewed by David Hyatt.
+
+        The problem was creating the Range from the arguments received at RenderView::setSelection(). Specifically
+        in this test endPos is 1 when the element has not children which creates an invalid Range.
+
+        * rendering/RenderView.cpp:
+        (WebCore::RenderView::splitSelectionBetweenSubtrees): Pass 0 as startPos and endPos to Range::create() as we are
+        not interested in the positions at this point.
+
 2014-04-22  Brady Eidson  <beidson@apple.com>
 
         Change Image Controls replacement to use selection and paste
index d4426b2..9c6d4ac 100644 (file)
@@ -799,7 +799,7 @@ void RenderView::splitSelectionBetweenSubtrees(RenderObject* start, int startPos
         renderSubtreesMap.set(namedFlowThread, nullptr);
 
     if (start && end) {
-        RefPtr<Range> initialRange = Range::create(document(), start->node(), startPos, end->node(), endPos);
+        RefPtr<Range> initialRange = Range::create(document(), start->node(), 0, end->node(), 0);
 
         Node* startNode = initialRange->startContainer();
         Node* endNode = initialRange->endContainer();