[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

rniwa at webkit.org rniwa at webkit.org
Wed Dec 22 14:57:18 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit 7862b1c3d5ae4e1173df4976795c7dea9c9ee4a9
Author: rniwa at webkit.org <rniwa at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Tue Oct 26 00:07:00 2010 +0000

    2010-10-25  Ryosuke Niwa  <rniwa at webkit.org>
    
            Reviewed by Tony Chang.
    
            execCommand('removeFormat') adds extra styles inside tables
            https://bugs.webkit.org/show_bug.cgi?id=29164
    
            Added a test to ensure RemoveFormat command does not add erroneous style attribute
            when removing styles inside a table cell.
    
            * editing/execCommand/remove-format-in-table-cell-expected.txt: Added.
            * editing/execCommand/remove-format-in-table-cell.html: Added.
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@70507 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
index c231b90..84cc67d 100644
--- a/LayoutTests/ChangeLog
+++ b/LayoutTests/ChangeLog
@@ -1,3 +1,16 @@
+2010-10-25  Ryosuke Niwa  <rniwa at webkit.org>
+
+        Reviewed by Tony Chang.
+
+        execCommand('removeFormat') adds extra styles inside tables
+        https://bugs.webkit.org/show_bug.cgi?id=29164
+
+        Added a test to ensure RemoveFormat command does not add erroneous style attribute
+        when removing styles inside a table cell.
+
+        * editing/execCommand/remove-format-in-table-cell-expected.txt: Added.
+        * editing/execCommand/remove-format-in-table-cell.html: Added.
+
 2010-10-25  Kenneth Russell  <kbr at google.com>
 
         Unreviewed, test expectations update. Temporarily WebGL layout
diff --git a/LayoutTests/editing/execCommand/remove-format-in-table-cell-expected.txt b/LayoutTests/editing/execCommand/remove-format-in-table-cell-expected.txt
new file mode 100644
index 0000000..56ef939
--- /dev/null
+++ b/LayoutTests/editing/execCommand/remove-format-in-table-cell-expected.txt
@@ -0,0 +1,9 @@
+This tests that RemoveFormat does not add extra span tags when executed inside a table cell.
+| <table>
+|   <tbody>
+|     <tr>
+|       <td>
+|         "<#selection-anchor>hello<#selection-focus>"
+|     <tr>
+|       <td>
+|         "world"
diff --git a/LayoutTests/editing/execCommand/remove-format-in-table-cell.html b/LayoutTests/editing/execCommand/remove-format-in-table-cell.html
new file mode 100644
index 0000000..ffdf47f
--- /dev/null
+++ b/LayoutTests/editing/execCommand/remove-format-in-table-cell.html
@@ -0,0 +1,15 @@
+<!DOCTYPE html>
+<html>
+<body>
+<script src="../../resources/dump-as-markup.js"></script>
+<div id="test" contenteditable><table><tr><td><b>hello</b></td></tr><tr><td>world</td></tr></table></div>
+<script>
+
+Markup.description('This tests that RemoveFormat does not add extra span tags when executed inside a table cell.');
+window.getSelection().selectAllChildren(document.getElementsByTagName('b')[0]);
+document.execCommand('RemoveFormat', false, null);
+Markup.dump('test');
+
+</script>
+</body>
+</html>

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list