Changeset 57876 in webkit

Timestamp:

Apr 20, 2010 12:15:03 AM (14 years ago)

Author:

tkent@chromium.org

Message:

2010-04-20 Kent Tamura <​tkent@chromium.org>

Reviewed by Shinichiro Hamaji.

[DRT/Chromium] Fix some unexpected results of editing
​https://bugs.webkit.org/show_bug.cgi?id=37843

This change fixes about 70 unexpected results.
The original test_webview_delegate.cc doesn't have this bug.
The bug was introduced when I ported it to WebKit tree.

• DumpRenderTree/chromium/WebViewHost.cpp: (printRangeDescription): Replace the latter startContainer() with endContainer().

Location:

trunk/WebKitTools

Files:

• ChangeLog (modified) (1 diff)
• DumpRenderTree/chromium/WebViewHost.cpp (modified) (1 diff)

trunk/WebKitTools/ChangeLog

@@ +1 @@
+2010-04-20  Kent Tamura  <tkent@chromium.org>
+
+        Reviewed by Shinichiro Hamaji.
+
+        [DRT/Chromium] Fix some unexpected results of editing
+        https://bugs.webkit.org/show_bug.cgi?id=37843
+
+        This change fixes about 70 unexpected results.
+        The original test_webview_delegate.cc doesn't have this bug.
+        The bug was introduced when I ported it to WebKit tree.
+
+        * DumpRenderTree/chromium/WebViewHost.cpp:
+        (printRangeDescription): Replace the latter startContainer() with endContainer().
+
 2010-04-19  Dirk Pranke  <dpranke@chromium.org>
 

trunk/WebKitTools/DumpRenderTree/chromium/WebViewHost.cpp

@@ -221 +221 @@
     printNodeDescription(startNode, exception);
     printf(" to %d of ", range.endOffset());
-    WebNode endNode = range.startContainer(exception);
+    WebNode endNode = range.endContainer(exception);
     printNodeDescription(endNode, exception);
 }