Changeset 70448 in webkit


Ignore:
Timestamp:
Oct 25, 2010 5:31:13 AM (13 years ago)
Author:
commit-queue@webkit.org
Message:

2010-10-25 Benjamin Kalman <​kalman@chromium.org>

Reviewed by Kent Tamura.

Make editing/deleting/5390681.html and
editing/deleting/delete-mixed-editable-content-001.html use Markup
instead of render tree.
​https://bugs.webkit.org/show_bug.cgi?id=48112

  • editing/deleting/5390681-expected.txt: Added.
  • editing/deleting/5390681.html: Updated to use Markup instead of render tree.
  • editing/deleting/delete-mixed-editable-content-001-expected.txt: Added.
  • editing/deleting/delete-mixed-editable-content-001.html: Updated to use Markup instead of render tree.
  • platform/chromium-linux/editing/deleting/5390681-expected.checksum: Removed.
  • platform/chromium-linux/editing/deleting/5390681-expected.png: Removed.
  • platform/chromium-linux/editing/deleting/delete-mixed-editable-content-001-expected.checksum: Removed.
  • platform/chromium-linux/editing/deleting/delete-mixed-editable-content-001-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5390681-expected.checksum: Removed.
  • platform/chromium-win/editing/deleting/5390681-expected.png: Removed.
  • platform/chromium-win/editing/deleting/5390681-expected.txt: Removed.
  • platform/chromium-win/editing/deleting/delete-mixed-editable-content-001-expected.checksum: Removed.
  • platform/chromium-win/editing/deleting/delete-mixed-editable-content-001-expected.png: Removed.
  • platform/chromium-win/editing/deleting/delete-mixed-editable-content-001-expected.txt: Removed.
  • platform/gtk/editing/deleting/5390681-expected.txt: Removed.
  • platform/mac-leopard/editing/deleting/5390681-expected.checksum: Removed.
  • platform/mac-leopard/editing/deleting/5390681-expected.png: Removed.
  • platform/mac/editing/deleting/5390681-expected.checksum: Removed.
  • platform/mac/editing/deleting/5390681-expected.png: Removed.
  • platform/mac/editing/deleting/5390681-expected.txt: Removed.
  • platform/mac/editing/deleting/delete-mixed-editable-content-001-expected.checksum: Removed.
  • platform/mac/editing/deleting/delete-mixed-editable-content-001-expected.png: Removed.
  • platform/mac/editing/deleting/delete-mixed-editable-content-001-expected.txt: Removed.
  • platform/qt/editing/deleting/5390681-expected.txt: Removed.
  • platform/qt/editing/deleting/delete-mixed-editable-content-001-expected.txt: Removed.
Location:
trunk/LayoutTests
Files:
2 added
21 deleted
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r70447 r70448  
     12010-10-25  Benjamin Kalman  <kalman@chromium.org>
     2
     3        Reviewed by Kent Tamura.
     4
     5        Make editing/deleting/5390681.html and
     6        editing/deleting/delete-mixed-editable-content-001.html use Markup
     7        instead of render tree.
     8        https://bugs.webkit.org/show_bug.cgi?id=48112
     9
     10        * editing/deleting/5390681-expected.txt: Added.
     11        * editing/deleting/5390681.html: Updated to use Markup instead of
     12        render tree.
     13        * editing/deleting/delete-mixed-editable-content-001-expected.txt: Added.
     14        * editing/deleting/delete-mixed-editable-content-001.html: Updated to
     15        use Markup instead of render tree.
     16        * platform/chromium-linux/editing/deleting/5390681-expected.checksum: Removed.
     17        * platform/chromium-linux/editing/deleting/5390681-expected.png: Removed.
     18        * platform/chromium-linux/editing/deleting/delete-mixed-editable-content-001-expected.checksum: Removed.
     19        * platform/chromium-linux/editing/deleting/delete-mixed-editable-content-001-expected.png: Removed.
     20        * platform/chromium-win/editing/deleting/5390681-expected.checksum: Removed.
     21        * platform/chromium-win/editing/deleting/5390681-expected.png: Removed.
     22        * platform/chromium-win/editing/deleting/5390681-expected.txt: Removed.
     23        * platform/chromium-win/editing/deleting/delete-mixed-editable-content-001-expected.checksum: Removed.
     24        * platform/chromium-win/editing/deleting/delete-mixed-editable-content-001-expected.png: Removed.
     25        * platform/chromium-win/editing/deleting/delete-mixed-editable-content-001-expected.txt: Removed.
     26        * platform/gtk/editing/deleting/5390681-expected.txt: Removed.
     27        * platform/mac-leopard/editing/deleting/5390681-expected.checksum: Removed.
     28        * platform/mac-leopard/editing/deleting/5390681-expected.png: Removed.
     29        * platform/mac/editing/deleting/5390681-expected.checksum: Removed.
     30        * platform/mac/editing/deleting/5390681-expected.png: Removed.
     31        * platform/mac/editing/deleting/5390681-expected.txt: Removed.
     32        * platform/mac/editing/deleting/delete-mixed-editable-content-001-expected.checksum: Removed.
     33        * platform/mac/editing/deleting/delete-mixed-editable-content-001-expected.png: Removed.
     34        * platform/mac/editing/deleting/delete-mixed-editable-content-001-expected.txt: Removed.
     35        * platform/qt/editing/deleting/5390681-expected.txt: Removed.
     36        * platform/qt/editing/deleting/delete-mixed-editable-content-001-expected.txt: Removed.
     37
    1382010-10-25  Ariya Hidayat  <ariya@sencha.com>
    239

  • trunk/LayoutTests/editing/deleting/5390681.html

    r24948 r70448  
    1 <p>This tests for a bug where expansion for smart delete would not consider editable boundaries.  Only 'foo' should be deleted.  You should see ' bar'. <b>There is a bug: while the non-editable space isn't deleted, deletion inserts a placeholder when it shouldn't.</b></p>
     1<script src="../../resources/dump-as-markup.js"></script>
    22<div contenteditable="true" id="div">foo<span contenteditable="false"> bar</span></div>
    3 
    43<script>
    54var div = document.getElementById("div");
    … …  
    87sel.modify("extend", "forward", "word");
    98document.execCommand("Delete");
     9Markup.description("This tests for a bug where expansion for smart delete would not consider editable boundaries.  Only 'foo' should be deleted.  You should see ' bar'.  There is a bug: while the non-editable space isn't deleted, deletion inserts a placeholder when it shouldn't.");
     10Markup.dump(div);
    1011</script>

  • trunk/LayoutTests/editing/deleting/delete-mixed-editable-content-001.html

    r15587 r70448  
     1<script src="../../resources/dump-as-markup.js"></script>
    12<script src=../editing.js language="JavaScript" type="text/JavaScript" ></script>
    23<script>
    34function editingTest() {
    4         selectAllCommand();
    5         deleteCommand();
     5    selectAllCommand();
     6    deleteCommand();
    67}
    78</script>
    … …  
    1314</div>
    1415<script>
    15 runEditingTest();
     16runDumpAsTextEditingTest(false);
     17Markup.description("Confirm no crash when selecting all text with mixed editability.");
     18Markup.dump(document.getElementsByTagName("div")[0]);
    1619</script>
Note: See TracChangeset for help on using the changeset viewer.