RESOLVED FIXED119698
Remove Node::attach() and ContainerNode::attach() 
https://bugs.webkit.org/show_bug.cgi?id=119698
Summary Remove Node::attach() and ContainerNode::attach()
Antti Koivisto
Reported 2013-08-12 14:03:15 PDT
Currently attach() is a virtual function on Node. However only Elements have style and so a corresponding independent render object. Attaching should be made Element level concept.
Attachments
possibly patch (43.41 KB, patch)
2013-08-12 14:22 PDT, Antti Koivisto 		buildbot: commit-queue-
DetailsFormatted DiffDiff
another (43.41 KB, patch)
2013-08-12 16:16 PDT, Antti Koivisto 		buildbot: commit-queue-
DetailsFormatted DiffDiff
Archive of layout-test-results from webkit-ews-08 for mac-mountainlion (377.91 KB, application/zip)
2013-08-12 17:35 PDT, Build Bot 		no flags
Details
Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 (294.72 KB, application/zip)
2013-08-13 01:49 PDT, Build Bot 		no flags
Details
another (44.64 KB, patch)
2013-08-13 07:42 PDT, Antti Koivisto 		eflews.bot: commit-queue-
DetailsFormatted DiffDiff
yet another (44.91 KB, patch)
2013-08-13 10:45 PDT, Antti Koivisto 		gtk-ews: commit-queue-
DetailsFormatted DiffDiff
Archive of layout-test-results from webkit-ews-04 for mac-mountainlion (572.22 KB, application/zip)
2013-08-13 13:06 PDT, Build Bot 		no flags
Details
Archive of layout-test-results from webkit-ews-07 for mac-mountainlion (576.69 KB, application/zip)
2013-08-13 14:18 PDT, Build Bot 		no flags
Details
Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2 (596.04 KB, application/zip)
2013-08-13 20:06 PDT, Build Bot 		no flags
Details
more (45.28 KB, patch)
2013-08-14 05:54 PDT, Antti Koivisto 		kling: review+
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Antti Koivisto
Comment 1 2013-08-12 14:22:38 PDT
Created attachment 208563 [details] possibly patch
Build Bot
Comment 2 2013-08-12 15:02:42 PDT
Comment on attachment 208563 [details] possibly patch Attachment 208563 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/1445487
EFL EWS Bot
Comment 3 2013-08-12 15:03:40 PDT
Comment on attachment 208563 [details] possibly patch Attachment 208563 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/1455188
Build Bot
Comment 4 2013-08-12 15:20:37 PDT
Comment on attachment 208563 [details] possibly patch Attachment 208563 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/1449294
Antti Koivisto
Comment 5 2013-08-12 16:16:21 PDT
Created attachment 208570 [details] another
Build Bot
Comment 6 2013-08-12 17:35:26 PDT
Comment on attachment 208570 [details] another Attachment 208570 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/1459173 New failing tests: editing/style/remove-underline-across-paragraph-in-bold.html editing/inserting/editing-empty-divs.html editing/selection/caret-rtl.html fullscreen/full-screen-render-inline.html editing/style/remove-underline-from-stylesheet.html editing/selection/caret-ltr-right.html fast/repaint/caret-jump-between-nodes.html http/tests/inspector/inspect-element.html editing/selection/caret-ltr-2.html fast/multicol/span/clone-summary.html fast/html/details-remove-child-2.html editing/pasteboard/paste-match-style-001.html editing/selection/caret-rtl-2-left.html fast/multicol/span/clone-anonymous-block-non-inline-child-crash.html editing/style/remove-underline-across-paragraph.html editing/selection/caret-ltr-2-left.html fast/html/details-add-child-2.html editing/style/remove-underline.html fast/html/details-open2.html editing/selection/4983858.html editing/selection/caret-rtl-2.html editing/style/remove-underline-after-paragraph.html editing/selection/caret-rtl-right.html editing/inserting/editable-html-element.html editing/style/remove-underline-in-bold.html editing/style/remove-underline-after-paragraph-in-bold.html editing/style/typing-style-003.html editing/selection/caret-ltr.html fast/loader/javascript-url-in-object.html editing/style/underline.html
Build Bot
Comment 7 2013-08-12 17:35:28 PDT
Created attachment 208576 [details] Archive of layout-test-results from webkit-ews-08 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-08 Port: mac-mountainlion Platform: Mac OS X 10.8.4
EFL EWS Bot
Comment 8 2013-08-12 17:46:37 PDT
Comment on attachment 208570 [details] another Attachment 208570 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/1459183
Build Bot
Comment 9 2013-08-13 01:49:44 PDT
Comment on attachment 208570 [details] another Attachment 208570 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/1378424 New failing tests: editing/style/remove-underline-across-paragraph-in-bold.html editing/inserting/editing-empty-divs.html editing/selection/caret-rtl.html editing/style/typing-style-003.html editing/selection/caret-ltr-right.html http/tests/inspector/inspect-element.html editing/pasteboard/copy-image-with-alt-text.html editing/pasteboard/paste-match-style-001.html editing/selection/caret-rtl-2-left.html editing/style/remove-underline-across-paragraph.html editing/selection/caret-ltr-2-left.html fast/html/details-remove-child-2.html fast/html/details-add-child-2.html editing/style/remove-underline.html fast/html/details-open2.html editing/selection/4983858.html editing/selection/caret-rtl-2.html editing/style/remove-underline-after-paragraph.html editing/selection/caret-rtl-right.html editing/inserting/editable-html-element.html editing/style/remove-underline-in-bold.html editing/style/remove-underline-after-paragraph-in-bold.html editing/style/remove-underline-from-stylesheet.html editing/selection/caret-ltr.html editing/selection/caret-ltr-2.html editing/style/underline.html
Build Bot
Comment 10 2013-08-13 01:49:47 PDT
Created attachment 208609 [details] Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-15 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.4
Antti Koivisto
Comment 11 2013-08-13 07:42:17 PDT
Created attachment 208629 [details] another
EFL EWS Bot
Comment 12 2013-08-13 08:37:06 PDT
Comment on attachment 208629 [details] another Attachment 208629 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/1451473
Antti Koivisto
Comment 13 2013-08-13 10:45:34 PDT
Created attachment 208651 [details] yet another
kov's GTK+ EWS bot
Comment 14 2013-08-13 12:24:14 PDT
Comment on attachment 208651 [details] yet another Attachment 208651 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/1446748
Build Bot
Comment 15 2013-08-13 13:06:18 PDT
Comment on attachment 208651 [details] yet another Attachment 208651 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/1458429 New failing tests: fullscreen/video-controls-override.html
Build Bot
Comment 16 2013-08-13 13:06:23 PDT
Created attachment 208666 [details] Archive of layout-test-results from webkit-ews-04 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-04 Port: mac-mountainlion Platform: Mac OS X 10.8.4
Build Bot
Comment 17 2013-08-13 14:18:26 PDT
Comment on attachment 208651 [details] yet another Attachment 208651 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/1456540 New failing tests: fullscreen/video-controls-override.html
Build Bot
Comment 18 2013-08-13 14:18:30 PDT
Created attachment 208675 [details] Archive of layout-test-results from webkit-ews-07 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-07 Port: mac-mountainlion Platform: Mac OS X 10.8.4
Build Bot
Comment 19 2013-08-13 20:06:51 PDT
Comment on attachment 208651 [details] yet another Attachment 208651 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/1453665 New failing tests: fullscreen/video-controls-override.html
Build Bot
Comment 20 2013-08-13 20:06:56 PDT
Created attachment 208697 [details] Archive of layout-test-results from webkit-ews-16 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-16 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.4
Antti Koivisto
Comment 21 2013-08-14 05:54:00 PDT
Created attachment 208720 [details] more
Andreas Kling
Comment 22 2013-08-14 06:49:17 PDT
Comment on attachment 208720 [details] more r=me, this is very cool. Like I mentioned on IRC, I think we can micro-optimize some things here but now is hardly the time. :)
Antti Koivisto
Comment 23 2013-08-14 07:02:45 PDT
http://trac.webkit.org/changeset/154047
Ryosuke Niwa
Comment 24 2013-08-14 11:36:25 PDT
It seems like you've already figured out what was wrong with the editing test? Let me know if you still needed my help. I assume you've figured it out for now.
Note You need to log in before you can comment on or make changes to this bug.