60620 2011-05-11 05:05:28 -0700 [Windows] REGRESSION(r65868): Tables from Excel are pasted as plain text 2011-11-25 15:54:13 -0800 1 1 1 Unclassified WebKit DOM 528+ (Nightly build) Unspecified Unspecified NEW InRadar P1 Major --- 60644 62112 41115 1 rniwa webkit-unassigned abarth aestes ap aroben dcheng enrica eric komoroske tony oldest_to_newest 401911 0 rniwa 2011-05-11 05:05:28 -0700 On WebKit TOT, copying & pasting table cells from Microsoft Excel result in table cells being pasted as plain text. This is due to the fact we're stripping tr, td, and other table cell elements when we're parsing the following HTML fragment (no table element): <col width=64 span=2 style='width:48pt'> <tr height=20 style='height:15.0pt'> <td height=20 class=xl65 width=64 style='height:15.0pt;width:48pt'>hello</span></td> <td class=xl65 width=64 style='border-left:none;width:48pt'>world</td> </tr> <tr height=20 style='height:15.0pt'> <td height=20 class=xl65 style='height:15.0pt;border-top:none'>webkit</td> <td class=xl65 style='border-top:none;border-left:none'>&nbsp;</td> </tr> http://crbug.com/19360 401922 1 rniwa 2011-05-11 05:35:07 -0700 Unfortunately, nighties between this range crash on start an I cannot test the behavior. But I suspect that http://trac.webkit.org/changeset/65868 is the cause. 401923 2 rniwa 2011-05-11 05:36:41 -0700 To reproduce this bug, insert the markup in comment #1 by execCommand('insertHTML'); WebKit should insert tr's and td's. 401925 3 93111 rniwa 2011-05-11 05:39:04 -0700 Created attachment 93111 HTML content to be pasted in step 2 To reproduce the original bug, you must follow the lengthly steps below: Reproduction steps: 1. Download Windows Clipboard Viewer from http://www.peterbuettner.de/develop/tools/clipview/ 2. Launch the program and copy & paste the attached content 3. Type in "HTML Format" (without quotations) into a box right of "Push in Clip" 4. Press "Push in Clip" 5. Open http://www.mozilla.org/editor/midasdemo/ in WebKit Windows port. 6. Paste in the content editable region of the page Expected result: "hello", "world", and "WebKit" are in table cells Actual result: "hello world WebKit" is pasted as a plain text. 401930 4 aroben 2011-05-11 06:11:03 -0700 <rdar://problem/9420024> 401934 5 93113 rniwa 2011-05-11 06:24:24 -0700 Created attachment 93113 test for bisection 401951 6 aroben 2011-05-11 07:22:12 -0700 Bisection indicates that r65868 is to blame, as suspected. 401952 7 aroben 2011-05-11 07:22:38 -0700 r65868 is "Use new HTML5 TreeBuilder for fragment parsing" (bug 44475). 402097 8 eric 2011-05-11 10:37:53 -0700 What does firefox do? Should we not be using fragment parsing for copy/paste? 402103 9 rniwa 2011-05-11 10:44:00 -0700 (In reply to comment #8) > What does firefox do? Should we not be using fragment parsing for copy/paste? They seem to have a special parsing algorithm just to deal with CF HTML :( 402107 10 tony 2011-05-11 11:00:15 -0700 (In reply to comment #9) > (In reply to comment #8) > > What does firefox do? Should we not be using fragment parsing for copy/paste? > > They seem to have a special parsing algorithm just to deal with CF HTML :( Ryosuke and I were looking at the Moz code yesterday. He's saying there's a parsing algorithm that parses CF_HTML to generate HTML, which I assume gets passed to their HTML5 parser. WebKit has similar code for parsing CF_HTML into HTML, but it's very simplistic. 509284 11 rniwa 2011-11-25 15:54:13 -0800 This has been fixed on Chromium Windows. 93111 2011-05-11 05:39:04 -0700 2011-05-11 05:39:04 -0700 HTML content to be pasted in step 2 excel-cp.txt text/plain 2401 rniwa VmVyc2lvbjoxLjANClN0YXJ0SFRNTDowMDAwMDAwMTA1DQpFbmRIVE1MOjAwMDAwMDI0MDENClN0 YXJ0RnJhZ21lbnQ6MDAwMDAwMTg5OQ0KRW5kRnJhZ21lbnQ6MDAwMDAwMjM0OQ0KDQo8aHRtbCB4 bWxuczp2PSJ1cm46c2NoZW1hcy1taWNyb3NvZnQtY29tOnZtbCINCnhtbG5zOm89InVybjpzY2hl bWFzLW1pY3Jvc29mdC1jb206b2ZmaWNlOm9mZmljZSINCnhtbG5zOng9InVybjpzY2hlbWFzLW1p Y3Jvc29mdC1jb206b2ZmaWNlOmV4Y2VsIg0KeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnL1RSL1JF Qy1odG1sNDAiPg0KDQo8aGVhZD4NCjxtZXRhIGh0dHAtZXF1aXY9Q29udGVudC1UeXBlIGNvbnRl bnQ9InRleHQvaHRtbDsgY2hhcnNldD11dGYtOCI+DQo8bWV0YSBuYW1lPVByb2dJZCBjb250ZW50 PUV4Y2VsLlNoZWV0Pg0KPG1ldGEgbmFtZT1HZW5lcmF0b3IgY29udGVudD0iTWljcm9zb2Z0IEV4 Y2VsIDE0Ij4NCjxsaW5rIGlkPU1haW4tRmlsZSByZWw9TWFpbi1GaWxlDQpocmVmPSJmaWxlOi8v L0M6XFVzZXJzXHRlc3RpbmdcQXBwRGF0YVxMb2NhbFxUZW1wXG1zb2h0bWxjbGlwMVwwMVxjbGlw Lmh0bSI+DQo8bGluayByZWw9RmlsZS1MaXN0DQpocmVmPSJmaWxlOi8vL0M6XFVzZXJzXHRlc3Rp bmdcQXBwRGF0YVxMb2NhbFxUZW1wXG1zb2h0bWxjbGlwMVwwMVxjbGlwX2ZpbGVsaXN0LnhtbCI+ DQo8c3R5bGU+DQo8IS0tdGFibGUNCgl7bXNvLWRpc3BsYXllZC1kZWNpbWFsLXNlcGFyYXRvcjoi XC4iOw0KCW1zby1kaXNwbGF5ZWQtdGhvdXNhbmQtc2VwYXJhdG9yOiJcLCI7fQ0KQHBhZ2UNCgl7 bWFyZ2luOi43NWluIC43aW4gLjc1aW4gLjdpbjsNCgltc28taGVhZGVyLW1hcmdpbjouM2luOw0K CW1zby1mb290ZXItbWFyZ2luOi4zaW47fQ0KdHINCgl7bXNvLWhlaWdodC1zb3VyY2U6YXV0bzt9 DQpjb2wNCgl7bXNvLXdpZHRoLXNvdXJjZTphdXRvO30NCmJyDQoJe21zby1kYXRhLXBsYWNlbWVu dDpzYW1lLWNlbGw7fQ0KdGQNCgl7cGFkZGluZy10b3A6MXB4Ow0KCXBhZGRpbmctcmlnaHQ6MXB4 Ow0KCXBhZGRpbmctbGVmdDoxcHg7DQoJbXNvLWlnbm9yZTpwYWRkaW5nOw0KCWNvbG9yOmJsYWNr Ow0KCWZvbnQtc2l6ZToxMS4wcHQ7DQoJZm9udC13ZWlnaHQ6NDAwOw0KCWZvbnQtc3R5bGU6bm9y bWFsOw0KCXRleHQtZGVjb3JhdGlvbjpub25lOw0KCWZvbnQtZmFtaWx5OkNhbGlicmksIHNhbnMt c2VyaWY7DQoJbXNvLWZvbnQtY2hhcnNldDowOw0KCW1zby1udW1iZXItZm9ybWF0OkdlbmVyYWw7 DQoJdGV4dC1hbGlnbjpnZW5lcmFsOw0KCXZlcnRpY2FsLWFsaWduOmJvdHRvbTsNCglib3JkZXI6 bm9uZTsNCgltc28tYmFja2dyb3VuZC1zb3VyY2U6YXV0bzsNCgltc28tcGF0dGVybjphdXRvOw0K CW1zby1wcm90ZWN0aW9uOmxvY2tlZCB2aXNpYmxlOw0KCXdoaXRlLXNwYWNlOm5vd3JhcDsNCglt c28tcm90YXRlOjA7fQ0KLnhsNjUNCgl7Ym9yZGVyOi41cHQgc29saWQgd2luZG93dGV4dDt9DQot LT4NCjwvc3R5bGU+DQo8L2hlYWQ+DQoNCjxib2R5IGxpbms9Ymx1ZSB2bGluaz1wdXJwbGU+DQo8 c3BhbiBzdHlsZT0nb3JwaGFuczogMjt0ZXh0LWFsaWduOmF1dG87d2lkb3dzOiAyOy13ZWJraXQt Ym9yZGVyLWhvcml6b250YWwtc3BhY2luZzogMHB4Ow0KLXdlYmtpdC1ib3JkZXItdmVydGljYWwt c3BhY2luZzogMHB4Oy13ZWJraXQtdGV4dC1kZWNvcmF0aW9ucy1pbi1lZmZlY3Q6IG5vbmU7DQot d2Via2l0LXRleHQtc2l6ZS1hZGp1c3Q6IGF1dG87LXdlYmtpdC10ZXh0LXN0cm9rZS13aWR0aDog MHB4Jz4NCg0KPHRhYmxlIGJvcmRlcj0wIGNlbGxwYWRkaW5nPTAgY2VsbHNwYWNpbmc9MCB3aWR0 aD0xMjggc3R5bGU9J2JvcmRlci1jb2xsYXBzZToNCiBjb2xsYXBzZTt3aWR0aDo5NnB0Jz4NCjwh LS1TdGFydEZyYWdtZW50LS0+DQogPGNvbCB3aWR0aD02NCBzcGFuPTIgc3R5bGU9J3dpZHRoOjQ4 cHQnPg0KIDx0ciBoZWlnaHQ9MjAgc3R5bGU9J2hlaWdodDoxNS4wcHQnPg0KICA8dGQgaGVpZ2h0 PTIwIGNsYXNzPXhsNjUgd2lkdGg9NjQgc3R5bGU9J2hlaWdodDoxNS4wcHQ7d2lkdGg6NDhwdCc+ aGVsbG88L3NwYW4+PC90ZD4NCiAgPHRkIGNsYXNzPXhsNjUgd2lkdGg9NjQgc3R5bGU9J2JvcmRl ci1sZWZ0Om5vbmU7d2lkdGg6NDhwdCc+d29ybGQ8L3RkPg0KIDwvdHI+DQogPHRyIGhlaWdodD0y MCBzdHlsZT0naGVpZ2h0OjE1LjBwdCc+DQogIDx0ZCBoZWlnaHQ9MjAgY2xhc3M9eGw2NSBzdHls ZT0naGVpZ2h0OjE1LjBwdDtib3JkZXItdG9wOm5vbmUnPndlYmtpdDwvdGQ+DQogIDx0ZCBjbGFz cz14bDY1IHN0eWxlPSdib3JkZXItdG9wOm5vbmU7Ym9yZGVyLWxlZnQ6bm9uZSc+Jm5ic3A7PC90 ZD4NCiA8L3RyPg0KPCEtLUVuZEZyYWdtZW50LS0+DQo8L3RhYmxlPg0KDQo8L2JvZHk+DQoNCjwv aHRtbD4NCg== 93113 2011-05-11 06:24:24 -0700 2011-05-11 06:24:24 -0700 test for bisection bug60620.html text/html 742 rniwa PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8Ym9keT4KPGRpdiBpZD0idGVzdCIgY29udGVudGVkaXRh YmxlPjwvZGl2Pgo8c2NyaXB0PgoKdmFyIHRlc3QgPSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgn dGVzdCcpOwp0ZXN0LmZvY3VzKCk7CmRvY3VtZW50LmV4ZWNDb21tYW5kKCJJbnNlcnRIVE1MIiwg ZmFsc2UsICI8Y29sIHdpZHRoPTY0IHNwYW49MiBzdHlsZT0nd2lkdGg6NDhwdCc+IisKIjx0ciBo ZWlnaHQ9MjAgc3R5bGU9J2hlaWdodDoxNS4wcHQnPiIrCiIgPHRkIGhlaWdodD0yMCBjbGFzcz14 bDY1IHdpZHRoPTY0IHN0eWxlPSdoZWlnaHQ6MTUuMHB0O3dpZHRoOjQ4cHQnPmhlbGxvPC9zcGFu PjwvdGQ+IisKIiA8dGQgY2xhc3M9eGw2NSB3aWR0aD02NCBzdHlsZT0nYm9yZGVyLWxlZnQ6bm9u ZTt3aWR0aDo0OHB0Jz53b3JsZDwvdGQ+IisKIjwvdHI+IisKIjx0ciBoZWlnaHQ9MjAgc3R5bGU9 J2hlaWdodDoxNS4wcHQnPiIrCiIgPHRkIGhlaWdodD0yMCBjbGFzcz14bDY1IHN0eWxlPSdoZWln aHQ6MTUuMHB0O2JvcmRlci10b3A6bm9uZSc+d2Via2l0PC90ZD4iKwoiIDx0ZCBjbGFzcz14bDY1 IHN0eWxlPSdib3JkZXItdG9wOm5vbmU7Ym9yZGVyLWxlZnQ6bm9uZSc+Jm5ic3A7PC90ZD4iKwoi PC90cj4iKTsKdGVzdC5pbm5lckhUTUwgPSBkb2N1bWVudC5nZXRFbGVtZW50c0J5VGFnTmFtZSgn dGQnKS5sZW5ndGggPyAnUEFTUycgOiAnRkFJTCc7Cgo8L3NjcmlwdD4KPC9ib2R5Pgo8L2h0bWw+ Cg==